Skip to content

MemoryPressureHandler/glib: Disable GPU memory accounting for service…#1653

Open
pgorszkowski-igalia wants to merge 1 commit intowpe-2.46from
pgorszkowski/2.46/dont_account_gpu_memory_by_service_worker_process_in_memory_pressure
Open

MemoryPressureHandler/glib: Disable GPU memory accounting for service…#1653
pgorszkowski-igalia wants to merge 1 commit intowpe-2.46from
pgorszkowski/2.46/dont_account_gpu_memory_by_service_worker_process_in_memory_pressure

Conversation

@pgorszkowski-igalia
Copy link
Copy Markdown

@pgorszkowski-igalia pgorszkowski-igalia commented Apr 30, 2026

… workers.

MemoryPressureHandler accounts for GPU memory in every WPEWebProcess and has no way to distinguish whether it is the main web process or a service worker. In conjunction with s_videoMemoryInFootprint, footprint calculations subtract high GPU memory usage of the main web process from the low RSS memory value of a service worker, which results in an overflow and triggers critical memory pressure for the service worker.

Resolve this by disabling GPU memory accounting for service workers

Original author: Andrzej Surdej (https://github.com/asurdej-comcast)
a000eb1

Build-Tests Layout-Tests
✅ 🛠 wpe-246-amd64-build ✅ 🧪 wpe-246-amd64-layout
✅ 🛠 wpe-246-arm32-build ✅ 🧪 wpe-246-arm32-layout

… workers.

MemoryPressureHandler accounts for GPU memory in every WPEWebProcess
and has no way to distinguish whether it is the main web process
or a service worker. In conjunction with s_videoMemoryInFootprint,
footprint calculations subtract high GPU memory usage of the main web process
from the low RSS memory value of a service worker, which results in an overflow
and triggers critical memory pressure for the service worker.

Resolve this by disabling GPU memory accounting for service workers

Original author: Andrzej Surdej (https://github.com/asurdej-comcast)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants