Consider to move back to the polling timer model to detect a low-memory situation
Categories
(Core :: XPCOM, enhancement)
Tracking
()
People
(Reporter: toshi, Unassigned)
References
(Blocks 1 open bug)
Details
Bug 1586236 changed the memory detection model of our memory watcher for Windows from the polling timer to memory resource notification. The current model is excellent from the performance perspective, but we miss more potential OOM cases because OOM crash on Windows is caused by a shortage of commit space, not physical memory.
Depending on the result of the tab unloding experiment we conduct soon, we may move back to the polling timer to reach out to more OOM cases. If we do, we'll implement at least the following optimization techniques to minimize the performance impact:
- Dynamic polling interval depending on the available commit space
- Background timer thread
Reporter | ||
Updated•3 years ago
|
Comment 1•3 years ago
|
||
(In reply to Toshihito Kikuchi [:toshi] from comment #0)
Bug 1586236 changed the memory detection model of our memory watcher for Windows from the polling timer to memory resource notification. The current model is excellent from the performance perspective, but we miss more potential OOM cases because OOM crash on Windows is caused by a shortage of commit space, not physical memory.
Depending on the result of the tab unloding experiment we conduct soon, we may move back to the polling timer to reach out to more OOM cases. If we do, we'll implement at least the following optimization techniques to minimize the performance impact:
- Dynamic polling interval depending on the available commit space
- Background timer thread
This sounds great. Pushing these through sooner than later will benefit all. Looking forward to testing new builds with this improved function.
Updated•3 years ago
|
Comment hidden (off-topic) |
Comment 3•2 years ago
|
||
Sorry, there was a problem with the detection of inactive users. I'm reverting the change.
Comment 4•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Description
•