Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 12 Apr 2021 11:51:48 -0700 (PDT)
From:      "Rodney W. Grimes" <freebsd@gndrsh.dnsmgr.net>
To:        Edward Tomasz Napierala <trasz@freebsd.org>
Cc:        src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org
Subject:   Re: git: 82431b7506d6 - main - rc: remove the 'archdep' script
Message-ID:  <202104121851.13CIpmmQ034866@gndrsh.dnsmgr.net>
In-Reply-To: <202104121418.13CEISV0082707@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
> The branch main has been updated by trasz:
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=82431b7506d646165d52f11bc2958e8ce8ec5cce
> 
> commit 82431b7506d646165d52f11bc2958e8ce8ec5cce
> Author:     Edward Tomasz Napierala <trasz@FreeBSD.org>
> AuthorDate: 2021-04-12 13:38:16 +0000
> Commit:     Edward Tomasz Napierala <trasz@FreeBSD.org>
> 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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202104121851.13CIpmmQ034866>