From owner-dev-commits-src-all@freebsd.org Mon Apr 12 18:51:57 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 F31245DAF62; Mon, 12 Apr 2021 18:51:57 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (br1.CN84in.dnsmgr.net [69.59.192.140]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4FJyWx4P1Mz4mk2; Mon, 12 Apr 2021 18:51:57 +0000 (UTC) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: from gndrsh.dnsmgr.net (localhost [127.0.0.1]) by gndrsh.dnsmgr.net (8.13.3/8.13.3) with ESMTP id 13CIpmkZ034867; Mon, 12 Apr 2021 11:51:48 -0700 (PDT) (envelope-from freebsd@gndrsh.dnsmgr.net) Received: (from freebsd@localhost) by gndrsh.dnsmgr.net (8.13.3/8.13.3/Submit) id 13CIpmmQ034866; Mon, 12 Apr 2021 11:51:48 -0700 (PDT) (envelope-from freebsd) From: "Rodney W. Grimes" Message-Id: <202104121851.13CIpmmQ034866@gndrsh.dnsmgr.net> Subject: Re: git: 82431b7506d6 - main - rc: remove the 'archdep' script In-Reply-To: <202104121418.13CEISV0082707@gitrepo.freebsd.org> To: Edward Tomasz Napierala Date: Mon, 12 Apr 2021 11:51:48 -0700 (PDT) CC: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Reply-To: rgrimes@freebsd.org X-Mailer: ELM [version 2.4ME+ PL121h (25)] MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 4FJyWx4P1Mz4mk2 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, 12 Apr 2021 18:51:58 -0000 > The branch main has been updated by trasz: > > URL: https://cgit.FreeBSD.org/src/commit/?id=82431b7506d646165d52f11bc2958e8ce8ec5cce > > commit 82431b7506d646165d52f11bc2958e8ce8ec5cce > Author: Edward Tomasz Napierala > AuthorDate: 2021-04-12 13:38:16 +0000 > Commit: Edward Tomasz Napierala > CommitDate: 2021-04-12 14:18:17 +0000 > > rc: remove the 'archdep' script > > It was unused since 405c3050f10, which removed iBCS support. > > This also moves the 'linux' rc script slightly earlier, which > might help in some setups. The original version of this patch > moved it even more, before 'mountcritlocal', which would fixe > mount(8) errors due to missing /dev/shm in setups with entries > for /path/to/chroot/dev/shm without the "late" flag; however, > in the end 'kldxref' turned out to depend on 'mountcritlocal' > anyway. I would think that these hintts are built and kernel/module install time, not at the much later when the system is booting. Crazy idea, but first isnt it too late for hints by the time this runs? So... change this from a _start to a _shutdown action and build the hints files on the way down so that all things are ready next time the system boots. This would fix the /usr mount issue. > > Reported By: pstef > Reviewed By: dchagin > Sponsored By: EPSRC > Differential Revision: https://reviews.freebsd.org/D29590 > --- > ObsoleteFiles.inc | 3 +++ > libexec/rc/rc.d/Makefile | 1 - > libexec/rc/rc.d/archdep | 26 -------------------------- > libexec/rc/rc.d/linux | 2 +- > libexec/rc/rc.d/sysvipc | 2 +- > 5 files changed, 5 insertions(+), 29 deletions(-) > > diff --git a/ObsoleteFiles.inc b/ObsoleteFiles.inc > index 373e014ba074..1019436f1e28 100644 > --- a/ObsoleteFiles.inc > +++ b/ObsoleteFiles.inc > @@ -40,6 +40,9 @@ > # xargs -n1 | sort | uniq -d; > # done > > +# 20210410: remove unused libexec/rc.d/archdep > +OLD_FILES+=etc/rc.d/archdep > + > # 20210408: remove tcp_hostcache.h > OLD_FILES+=usr/include/netinet/tcp_hostcache.h > > diff --git a/libexec/rc/rc.d/Makefile b/libexec/rc/rc.d/Makefile > index 1a61e89aa215..7b15cab40bb8 100644 > --- a/libexec/rc/rc.d/Makefile > +++ b/libexec/rc/rc.d/Makefile > @@ -13,7 +13,6 @@ CONFS= DAEMON \ > SERVERS \ > addswap \ > adjkerntz \ > - archdep \ > bgfsck \ > ${_blacklistd} \ > bridge \ > diff --git a/libexec/rc/rc.d/archdep b/libexec/rc/rc.d/archdep > deleted file mode 100755 > index 52a7ac684eed..000000000000 > --- a/libexec/rc/rc.d/archdep > +++ /dev/null > @@ -1,26 +0,0 @@ > -#!/bin/sh > -# > -# $FreeBSD$ > -# > - > -# PROVIDE: archdep > -# REQUIRE: mountcritremote > -# KEYWORD: nojail > - > -. /etc/rc.subr > - > -name="archdep" > -start_cmd="archdep_start" > -stop_cmd=":" > - > -archdep_start() > -{ > - local _arch > - > - _arch=`${SYSCTL_N} hw.machine_arch` > - case $_arch in > - esac > -} > - > -load_rc_config $name > -run_rc_command "$1" > diff --git a/libexec/rc/rc.d/linux b/libexec/rc/rc.d/linux > index c9c45ba8b338..882cf4012495 100755 > --- a/libexec/rc/rc.d/linux > +++ b/libexec/rc/rc.d/linux > @@ -4,7 +4,7 @@ > # > > # PROVIDE: linux > -# REQUIRE: archdep > +# REQUIRE: kldxref > # KEYWORD: nojail > > . /etc/rc.subr > diff --git a/libexec/rc/rc.d/sysvipc b/libexec/rc/rc.d/sysvipc > index 0effbe7e22ca..bbec47d75e24 100755 > --- a/libexec/rc/rc.d/sysvipc > +++ b/libexec/rc/rc.d/sysvipc > @@ -4,7 +4,7 @@ > # > > # PROVIDE: sysvipc > -# REQUIRE: archdep > +# REQUIRE: kldxref > # KEYWORD: nojail > > . /etc/rc.subr > -- Rod Grimes rgrimes@freebsd.org