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 <b...@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 <unitrun...@gmail.com>
AuthorDate: 2021-02-07 06:15:21 +0000
Commit:     Baptiste Daroussin <b...@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
_______________________________________________
dev-commits-src-main@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main
To unsubscribe, send any mail to "dev-commits-src-main-unsubscr...@freebsd.org"

Reply via email to