Skip to content

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

Open
asurdej-comcast wants to merge 1 commit intoWebPlatformForEmbedded:wpe-2.38from
asurdej-comcast:memory_service_worker_gpu
Open

MemoryPressureHandler/glib: Disable GPU memory accounting for service…#1651
asurdej-comcast wants to merge 1 commit intoWebPlatformForEmbedded:wpe-2.38from
asurdej-comcast:memory_service_worker_gpu

Conversation

@asurdej-comcast
Copy link
Copy Markdown

@asurdej-comcast asurdej-comcast commented Apr 28, 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
116f85f

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

@asurdej-comcast asurdej-comcast marked this pull request as ready for review April 28, 2026 10:08
@pgorszkowski-igalia pgorszkowski-igalia self-assigned this Apr 28, 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
@asurdej-comcast asurdej-comcast force-pushed the memory_service_worker_gpu branch from 0d98656 to 116f85f Compare April 28, 2026 10:28
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