From owner-dev-commits-src-all@freebsd.org Mon Mar 8 07:16:33 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 71651567BD0; Mon, 8 Mar 2021 07:16:33 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from smtp-out-no.shaw.ca (smtp-out-no.shaw.ca [64.59.134.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Dv8lj0GvCz3lvn; Mon, 8 Mar 2021 07:16:32 +0000 (UTC) (envelope-from cy.schubert@cschubert.com) Received: from spqr.komquats.com ([70.67.229.168]) by shaw.ca with ESMTPA id JA7wlTpJX2SWTJA7xlCS0f; Mon, 08 Mar 2021 00:16:30 -0700 X-Authority-Analysis: v=2.4 cv=fdJod2cF c=1 sm=1 tr=0 ts=6045cf4e a=7AlCcx2GqMg+lh9P3BclKA==:117 a=7AlCcx2GqMg+lh9P3BclKA==:17 a=xqWC_Br6kY4A:10 a=kj9zAlcOel0A:10 a=dESyimp9J3IA:10 a=ZB5LerlCAAAA:8 a=6I5d2MoRAAAA:8 a=pGLkceISAAAA:8 a=YxBL1-UpAAAA:8 a=EkcXrb_YAAAA:8 a=IN2qHnOaUhcsauUY1eUA:9 a=CjuIK1q_8ugA:10 a=YKPTzOroS2oaEK2QgPcx:22 a=IjZwj45LgO3ly-622nXo:22 a=Ia-lj3WSrqcvXOmTRaiG:22 a=LK5xJRSDVpKd5WXXoEvA:22 a=BPzZvq435JnGatEyYwdK:22 Received: from slippy.cwsent.com (slippy [IPv6:fc00:1:1:1::5b]) by spqr.komquats.com (Postfix) with ESMTPS id 10D5D2AB2; Sun, 7 Mar 2021 23:16:27 -0800 (PST) Received: from slippy (localhost [127.0.0.1]) by slippy.cwsent.com (8.16.1/8.16.1) with ESMTP id 1287GQWB055131; Sun, 7 Mar 2021 23:16:26 -0800 (PST) (envelope-from Cy.Schubert@cschubert.com) Message-Id: <202103080716.1287GQWB055131@slippy.cwsent.com> X-Mailer: exmh version 2.9.0 11/07/2018 with nmh-1.7.1 Reply-to: Cy Schubert From: Cy Schubert X-os: FreeBSD X-Sender: cy@cwsent.com X-URL: http://www.cschubert.com/ To: Chris Rees cc: dev-commits-src-all@freebsd.org, Baptiste Daroussin , src-committers@freebsd.org, dev-commits-src-main@freebsd.org, Rick Parrish Subject: Re: git: 77e1ccbee3ed - main - rc: implement parallel boot In-reply-to: <74fde23ead6719ac4e56dacb51bca6ed@bayofrum.net> References: <202102231027.11NARYYE041280@gitrepo.freebsd.org> <6D4FCE07-B996-430C-8EA8-6CB37A6DEEE8@bayofrum.net> <74fde23ead6719ac4e56dacb51bca6ed@bayofrum.net> Comments: In-reply-to Chris Rees message dated "Sun, 07 Mar 2021 13:43:50 -0500." Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Sun, 07 Mar 2021 23:16:26 -0800 X-CMAE-Envelope: MS4xfFQ/DyGcyBpZWDprn+Ze9YBd5CmU2P5GKf9kcMvUDGO0faejt4XfFT4h941Ya2OkxzSMBRVFCyt2RoDwUWXLee9C5rqYagPAMTVZM716ntLLyAH48EsK WQ9FossCkxfGSZnVAJEBNW2pK0qY+oXYyq4pZ1FcW7I8S/NlHZ43BzENcEvZCQe8MIFxYfRFLgxKjyp565ehpTaptvNhnKKp9BUcSQwZJZI2rGf5kW9meB9a YLzQ/O3LabwQfm9MddzIqtgy6z0CGOpHnFbMd+MrZF0X7fMkmR9bMjFkIYM8zCUBU6PSY6HUf/58CYlzuXSzAJ1+0cOIsBtZMLpYEPbPUXKHmSIQh31RkbQ5 5iMkdIPKFlEXhqWuSDt+B0y/w8CzLKGq/N9i12I2kUUZHgDgVh0= X-Rspamd-Queue-Id: 4Dv8lj0GvCz3lvn X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 08 Mar 2021 07:16:33 -0000 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 > > wrote: > >> The branch main has been updated by bapt: > >> > >> URL: > >> https://cgit.FreeBSD.org/src/commit/?id=77e1ccbee3ed6c837929e4e232fd07f95b > fc8294 > >> > >> commit 77e1ccbee3ed6c837929e4e232fd07f95bfc8294 > >> Author: Rick Parrish > >> AuthorDate: 2021-02-07 06:15:21 +0000 > >> Commit: Baptiste Daroussin > >> 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 FreeBSD UNIX: Web: https://FreeBSD.org NTP: Web: https://nwtime.org The need of the many outweighs the greed of the few.