Re: [make-initrd] [PATCH v1 21/41] fork pipeline: bootchain sysvinit script now use external hooks

[Alexey Gladkov <gladkov.alexey@gmail.com>]

On Mon, Sep 27, 2021 at 04:11:43PM +0300, Leonid Krivoshein wrote:
> 
> 27.09.2021 12:09, Alexey Gladkov пишет:
> > 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.) ?
> 
> Речь о коде, перенесённом из одного места в другой. Как раз я не думаю, что
> тут можно как-то влиять на запуск до udev, ueventd. Но если мы вынесли из
> bootchain-core код, который до запуска демона должен что-то предварительно
> подготовить, нужен же механизм, который позволит его дёрнуть, для этого и
> нужны хуки. Другое дело, что по факту сейчас пока такой хук оказался лишь
> один. Но мне конструкция всё равно нравится больше, она делает стартовый
> скрипт более модульным, не зависящим от других фич bootchain-*.
> 
> 
> > > Другим компонентам 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
> > 
> > Ты имеешь в виду вот такие хуки ?
> 
> Ну очень похожие, хотя use_hooks() явно проще -- без блокировки консоли, без
> бита выполнения, просто соурсим исходники:

Эти хуки уже давно есть и можно использовать. Не нужно дублировать этот же
функционал.

> > > > > +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
> > > > >    }
> и:
> > > > > --- /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
> 
> 
> -- 
> Best regards,
> Leonid Krivoshein.
> 
> _______________________________________________
> Make-initrd mailing list
> Make-initrd@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Rgrds, legion