enable forkserver on OpenBSD
Categories
(Core :: IPC, enhancement)
Tracking
()
Tracking | Status | |
---|---|---|
firefox91 | --- | fixed |
People
(Reporter: gaston, Assigned: gaston)
Details
Attachments
(1 file, 1 obsolete file)
it's been enabled on FreeBSD in bug #1607103, afaict it's not really used (?? should be used for fission ?) but enabling it on OpenBSD didnt produce more errors than before...
Assignee | ||
Comment 1•3 years ago
|
||
:sinker, any idea about the implications/interactoins with our sandboxing (added in bug #1457092 then improved in bug #1584839 and bug #1580268) ?
Comment 2•3 years ago
|
||
After enabling support you also need to flip the dom.ipc.forkserver.enable preference to use it. If it works the only difference you should see is a drop of several megabytes (we measured between 5 and 7 MiB) in the RSS of child processes.
Comment 3•3 years ago
|
||
The patch is fine but it needs to be submitted using Phabricator in order to land it. If your busy I can do it for you.
Assignee | ||
Comment 4•3 years ago
|
||
It seems to work fine, in my limited testing with 90.0b1. all the processes that were previously named 'tab' now have /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
in the process list, and are indeed consuming a bit less ressources:
landry 48553 11.4 4.6 372144 375608 p4 S+pU 7:45AM 0:07.25 firefox
landry 96300 4.2 1.9 123248 152900 p4 S+pU 7:45AM 0:02.25 /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
landry 7762 2.6 1.4 55172 111604 p4 S+pU 7:45AM 0:01.30 /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
landry 25373 0.0 0.4 2128 31344 p4 S+ 7:45AM 0:00.05 /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
landry 81008 0.0 0.5 4904 40332 p4 S+ 7:45AM 0:00.06 /usr/local/lib/firefox/firefox -contentproc -parentBuildID 20210601230611 -prefsLen 1 -prefMapSize 248765 -appdir /usr/local/lib/firefox/browser 48553 socket
landry 18885 0.0 0.7 18928 59824 p4 S+pU 7:45AM 0:00.21 /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
landry 66644 0.8 1.1 35844 93748 p4 S+pU 7:45AM 0:00.59 /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
landry 70099 0.0 0.7 19468 54236 p4 S+pU 7:45AM 0:00.12 /usr/local/lib/firefox/firefox -contentproc -appdir /usr/local/lib/firefox/browser 48553 forkserver
without forkserver enabled:
landry 96915 25.2 4.8 403628 393260 p4 S+pU 7:48AM 0:11.61 firefox
landry 99597 6.4 2.0 106308 161024 p4 S+pU 7:48AM 0:03.07 /usr/local/lib/firefox/firefox -contentproc -childID 4 -isForBrowser -prefsLen 5254 -prefMapSize 248765 -jsInit 285440 -parentBuildID 20210601230611 -appdir /usr/local/lib/firefox/browser 96915 tab
landry 80805 3.1 1.6 57308 129760 p4 S+pU 7:48AM 0:01.44 /usr/local/lib/firefox/firefox -contentproc -childID 1 -isForBrowser -prefsLen 65 -prefMapSize 248765 -jsInit 285440 -parentBuildID 20210601230611 -appdir /usr/local/lib/firefox/browser 96915 tab
landry 69369 2.5 1.5 55280 126428 p4 S+pU 7:48AM 0:01.20 /usr/local/lib/firefox/firefox -contentproc -childID 2 -isForBrowser -prefsLen 388 -prefMapSize 248765 -jsInit 285440 -parentBuildID 20210601230611 -appdir /usr/local/lib/firefox/browser 96915 tab
landry 51711 3.0 1.6 56056 129092 p4 S+pU 7:48AM 0:01.27 /usr/local/lib/firefox/firefox -contentproc -childID 5 -isForBrowser -prefsLen 5914 -prefMapSize 248765 -jsInit 285440 -parentBuildID 20210601230611 -appdir /usr/local/lib/firefox/browser 96915 tab
landry 24641 0.0 0.5 4636 40016 p4 S+ 7:48AM 0:00.07 /usr/local/lib/firefox/firefox -contentproc -parentBuildID 20210601230611 -prefsLen 1 -prefMapSize 248765 -appdir /usr/local/lib/firefox/browser 96915 socket
landry 74573 0.8 1.3 35824 108064 p4 S+pU 7:48AM 0:00.56 /usr/local/lib/firefox/firefox -contentproc -childID 3 -isForBrowser -prefsLen 482 -prefMapSize 248765 -jsInit 285440 -parentBuildID 20210601230611 -appdir /usr/local/lib/firefox/browser 96915 tab
landry 20276 0.2 0.8 17088 68992 p4 S+pU 7:48AM 0:00.17 /usr/local/lib/firefox/firefox -contentproc -childID 7 -isForBrowser -prefsLen 5980 -prefMapSize 248765 -jsInit 285440 -parentBuildID 20210601230611 -appdir /usr/local/lib/firefox/browser 96915 tab
will run with the setting enabled for a while to see if it doesnt conflict with the sandboxing work i started in bug #1713745 and bug #1713999.
as for submitting the patch via Phabricator, my moz-phab fu is rusty, but since i have a bunch of patches to upstream, guess that's an incentive for me to try and submit it...
Assignee | ||
Comment 5•3 years ago
|
||
i guess there's another bug tracking dom.ipc.forkserver.enable
being turned on for all platforms ?
Assignee | ||
Comment 6•3 years ago
|
||
Comment 7•3 years ago
•
|
||
(In reply to Landry Breuil (:gaston) from comment #5)
i guess there's another bug tracking
dom.ipc.forkserver.enable
being turned on for all platforms ?
We don't have one. I think we can file one per platform as we go. ATM the only platform that has it enabled and turned on by default is KaiOS but we hoped to be able to turn it on on Linux soon-ish.
Updated•3 years ago
|
Comment 8•3 years ago
|
||
Alright, this is queued for landing.
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/743a7a76c203 Build forkserver on OpenBSD r=gsvelto
Comment 11•3 years ago
|
||
bugherder |
Description
•