[meta] Turn on dom.ipc.processPriorityManager.enabled by default and let it ride out to release (macOS and Linux)
Categories
(Core :: DOM: Content Processes, enhancement, P3)
Tracking
()
People
(Reporter: mconley, Unassigned)
References
(Depends on 2 open bugs)
Details
(Keywords: meta)
This bug will track the work required to both evaluate the background process priority manager as appropriate to release, and (if so), to ship it to the release channel audience.
Comment 1•5 years ago
|
||
Mike, is there a plan to let processPriorityManager ride the trains? How will it impact Fission when one page may have many iframe processes?
Reporter | ||
Comment 2•5 years ago
|
||
(In reply to Chris Peterson [:cpeterson] from comment #1)
Mike, is there a plan to let processPriorityManager ride the trains?
The processPriorityManager rode the trains out for Windows in Firefox 69. Since Fission spun up, I don't think there's been much attention paid to the processPriorityManager (which currently is tab-centric, as opposed to frame-centric).
How will it impact Fission when one page may have many iframe processes?
I believe on Windows, subframes in the foreground tab without a top-level frame also in the foreground will run with lower OS priority.
We should make the processPriorityManager care about when frames are in the foreground tab, or we should turn off the process priority manager.
I've filed bug 1618547 for that.
Comment 3•5 years ago
|
||
(In reply to Mike Conley (:mconley) (:⚙️) from comment #2)
The processPriorityManager rode the trains out for Windows in Firefox 69. Since Fission spun up, I don't think there's been much attention paid to the processPriorityManager (which currently is tab-centric, as opposed to frame-centric).
macOS and Linux remain
Comment 5•3 years ago
|
||
(In reply to Chris Peterson [:cpeterson] from comment #3)
(In reply to Mike Conley (:mconley) (:⚙️) from comment #2)
The processPriorityManager rode the trains out for Windows in Firefox 69. Since Fission spun up, I don't think there's been much attention paid to the processPriorityManager (which currently is tab-centric, as opposed to frame-centric).
macOS and Linux remain
Mike, do you know which work remains to do to enable the process priority manager on all tier1 platforms?
Comment 6•3 years ago
|
||
(In reply to Florian Quèze [:florian] from comment #5)
Mike, do you know which work remains to do to enable the process priority manager on all tier1 platforms?
SetProcessPriority() (located in hal/Hal.h) needs to be implemented on platforms we want to support, which basically means figuring out the platform integration part of it, so this might be a better question for gcp. (The priority manager can be enabled on all platforms right now, it just doesn't actually change the OS priority.)
Comment 7•3 years ago
|
||
I believe this is bug https://bugzilla.mozilla.org/show_bug.cgi?id=1372543 which seems to be complicated on Linux at least.
Updated•3 years ago
|
Updated•2 years ago
|
Description
•