Date: Sun, 07 Mar 2021 13:43:50 -0500 From: Chris Rees <crees@bayofrum.net> To: dev-commits-src-all@freebsd.org, Baptiste Daroussin <bapt@freebsd.org>, src-committers@freebsd.org, dev-commits-src-main@freebsd.org, Rick Parrish <unitrunker@gmail.com> Subject: Re: git: 77e1ccbee3ed - main - rc: implement parallel boot Message-ID: <74fde23ead6719ac4e56dacb51bca6ed@bayofrum.net> In-Reply-To: <6D4FCE07-B996-430C-8EA8-6CB37A6DEEE8@bayofrum.net> References: <202102231027.11NARYYE041280@gitrepo.freebsd.org> <6D4FCE07-B996-430C-8EA8-6CB37A6DEEE8@bayofrum.net>
next in thread | previous in thread | raw e-mail | index | archive | help
Forgot to CC the actual author! On 2021-02-23 17:03, Chris Rees wrote: > Hi, > > On 23 February 2021 10:27:34 GMT, Baptiste Daroussin <bapt@FreeBSD.org> > wrote: >> The branch main has been updated by bapt: >> >> URL: >> https://cgit.FreeBSD.org/src/commit/?id=77e1ccbee3ed6c837929e4e232fd07f95bfc8294 >> >> commit 77e1ccbee3ed6c837929e4e232fd07f95bfc8294 >> Author: Rick Parrish <unitrunker@gmail.com> >> AuthorDate: 2021-02-07 06:15:21 +0000 >> Commit: Baptiste Daroussin <bapt@FreeBSD.org> >> CommitDate: 2021-02-23 10:16:53 +0000 >> >> rc: implement parallel boot >> >> take advantage of the rcorder -p argument to implement parallel >> booting in rc. >> >> According to the author non scientific tests: >> on a Core 2 Duo with spinning disk: >> >> | Services enabled | before | after | saving | >> | 0 | 8s | 8s | 0 | >> | 1 | 13s | 13s | 0 | >> | 2 | 17s | 13s | 5 | >> | 3 | 23s | 13s | 10 | >> | 4 | 28s | 13s | 15 | >> | 5 | 33s | 13s | 20 | >> >> PR: 249192 >> MFC after: 3 weeks >> --- >> libexec/rc/rc | 49 ++++++++++++++++++++++++++++++++++--------------- >> 1 file changed, 34 insertions(+), 15 deletions(-) >> >> diff --git a/libexec/rc/rc b/libexec/rc/rc >> index 35db4a850516..722d7fe35884 100644 >> --- a/libexec/rc/rc >> +++ b/libexec/rc/rc >> @@ -91,19 +91,31 @@ if ! [ -e ${firstboot_sentinel} ]; then >> skip_firstboot="-s firstboot" >> fi >> >> +# rc_parallel_start default is "NO" >> +rc_parallel_start=${rc_parallel_start:-NO} > > Should this go in defaults/rc.conf? I think this should be in defaults/rc.conf-- the load_rc_config line is above it. Incidentally, is the plan for this to go into 13 *and* 12, or just 13? It's an exciting step! Chris
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?74fde23ead6719ac4e56dacb51bca6ed>