Date: Sun, 07 Mar 2021 23:16:26 -0800 From: Cy Schubert <Cy.Schubert@cschubert.com> To: Chris Rees <crees@bayofrum.net> Cc: 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: <202103080716.1287GQWB055131@slippy.cwsent.com> In-Reply-To: <74fde23ead6719ac4e56dacb51bca6ed@bayofrum.net> References: <202102231027.11NARYYE041280@gitrepo.freebsd.org> <6D4FCE07-B996-430C-8EA8-6CB37A6DEEE8@bayofrum.net> <74fde23ead6719ac4e56dacb51bca6ed@bayofrum.net>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <74fde23ead6719ac4e56dacb51bca6ed@bayofrum.net>, Chris Rees writes: > 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=77e1ccbee3ed6c837929e4e232fd07f95b > fc8294 > >> > >> 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! If this does go into 13 or 12, it should be tested in -CURRENT for longer than three weeks. Probably six weeks. -- Cheers, Cy Schubert <Cy.Schubert@cschubert.com> FreeBSD UNIX: <cy@FreeBSD.org> Web: https://FreeBSD.org NTP: <cy@nwtime.org> Web: https://nwtime.org The need of the many outweighs the greed of the few.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202103080716.1287GQWB055131>