From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Mon, 27 Sep 2021 11:09:07 +0200 From: Alexey Gladkov To: make-initrd@lists.altlinux.org Message-ID: <20210927090907.fbre22j7urxlnukt@example.org> References: <7636585b-202d-95e0-eb0a-d320f69cc77a@gmail.com> <20210926140917.4xky7nfnlemy4jh3@example.org> <94921a86-1043-8273-2c29-957cf53e14e1@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <94921a86-1043-8273-2c29-957cf53e14e1@gmail.com> Subject: Re: [make-initrd] [PATCH v1 21/41] fork pipeline: bootchain sysvinit script now use external hooks X-BeenThere: make-initrd@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: make-initrd@lists.altlinux.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 Sep 2021 09:09:08 -0000 Archived-At: List-Archive: On Sun, Sep 26, 2021 at 10:53:08PM +0300, Leonid Krivoshein wrote: > > 26.09.2021 17:09, Alexey Gladkov пишет: > > On Fri, Sep 24, 2021 at 06:57:45PM +0300, Leonid Krivoshein wrote: > > > > Что это за хуки и зачем они нужны ? > > Отлично, уже дошли до первого принципиального и важного изменения! > Полагаю, тут больше камень в огород отсутствия описаний, это я поправлю. > > Стартовый скрипт SysVinit ранее включал prepare() для предварительной > подготовки каталогов для компонента waitdev, отделённого теперь в другую > фичу. Когда ты говоришь про prepare, ты про выполнение скриптов перед сервисом (udev, ueventd, etc.) ? > Другим компонентам bootchain при расширении (а теперь это уже более > десятка фич и подпакетов) тоже может потребоваться подобная инициализация. > Здесь в одном коммите видно, куда переехал код функции prepare(). Хуки нужны > для того, чтобы вызывать подобную инициализацию из определённого места, так > как базовый пакет bootchain-core может не знать, с какими компонентами > bootchain собран initramfs. Соответственно, включаемые скрипты кладутся в > /lib/bootchain-prepare.d. Если потребуется, по аналогии можно добавить в > stop() хуки для деинициализации. https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L32-L38 https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L61 https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L109 https://github.com/osboot/make-initrd/blob/master/data/etc/rc.d/rc#L114 Ты имеешь в виду вот такие хуки ? > > > --- > > > .../data/etc/rc.d/init.d/bootchain | 33 ++++++++++--------- > > > .../data/lib/bootchain-prepare.d/300-waitdev | 12 +++++++ > > > 2 files changed, 29 insertions(+), 16 deletions(-) > > > create mode 100644 make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev > > > > > > diff --git > > > a/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain > > > b/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain > > > index 89d9de5..7bdb1cf 100755 > > > --- a/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain > > > +++ b/make-initrd/features/bootchain-core/data/etc/rc.d/init.d/bootchain > > > @@ -1,6 +1,6 @@ > > > #!/bin/bash > > > ### BEGIN INIT INFO > > > -# Provides: pipeline > > > +# Provides: bootchain > > > # Required-Start: uevent udev > > > # Should-Start: > > > # Required-Stop: > > > @@ -13,27 +13,28 @@ > > > . /.initrd/initenv > > > . /etc/init.d/template > > > -NAME=pipelined > > > +NAME=bootchained > > > PIDFILE="/var/run/$NAME.pid" > > > ARGS="--lockfile $LOCKFILE --pidfile $PIDFILE --name $NAME --displayname > > > $NAME" > > > -prepare() { > > > - local dir i n > > > - > > > - dir=/.initrd/pipeline/waitdev > > > - mkdir -p -- "$dir" > > > - > > > - i=0 > > > - while [ "$i" -lt "${WAITDEV:-0}" ]; do > > > - touch "$dir/$i" > > > - i=$(($i + 1)) > > > - done > > > +use_hooks() > > > +{ > > > + local hook hdir="$1" > > > + > > > + if [ -d "$hdir" ]; then > > > + # shellcheck disable=SC2012 > > > + for hook in $(ls -1 -- "$hdir"/* |sort) _; do > > > + [ -s "$hook" ] || > > > + continue > > > + . "$hook" > > > + done > > > + fi > > > } > > > start() { > > > RETVAL=0 > > > - if [ "${ROOT-}" = pipeline ]; then > > > - prepare > > > + if [ "${ROOT-}" = bootchain ] || [ "${ROOT-}" = pipeline ]; then > > > + use_hooks /lib/bootchain-prepare.d > > > start_daemon --background $ARGS "$NAME" > > > RETVAL=$? > > > fi > > > @@ -43,7 +44,7 @@ start() { > > > stop() { > > > stop_daemon $ARGS "$NAME" > > > RETVAL=$? > > > - [ ! -f "$PIDFILE" ] || rm -f -- "$PIDFILE" > > > + rm -f -- "$PIDFILE" > > > return $RETVAL > > > } > > > diff --git a/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev > > > new file mode 100644 > > > index 0000000..f7315a3 > > > --- /dev/null > > > +++ b/make-initrd/features/bootchain-waitdev/data/lib/bootchain-prepare.d/300-waitdev > > > @@ -0,0 +1,12 @@ > > > +#!/bin/bash -efu > > > + > > > +dir=/.initrd/bootchain/waitdev > > > +mkdir -p -- "$dir" > > > + > > > +[ -z "${WAITDEV_TIMEOUT-}" ] || > > > + printf '%s\n' "$WAITDEV_TIMEOUT" >"$dir/TIMECNT" > > > +i=0 > > > +while [ "$i" -lt "${WAITDEV:-0}" ]; do > > > + touch "$dir/$i" > > > + i=$((1 + $i)) > > > +done > > > -- > > > 2.21.0 > > > > > > > > > _______________________________________________ > > > Make-initrd mailing list > > > Make-initrd@lists.altlinux.org > > > https://lists.altlinux.org/mailman/listinfo/make-initrd > > > > > -- > Best regards, > Leonid Krivoshein. > > _______________________________________________ > Make-initrd mailing list > Make-initrd@lists.altlinux.org > https://lists.altlinux.org/mailman/listinfo/make-initrd -- Rgrds, legion