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

[Leonid Krivoshein <klark.devel@gmail.com>]

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.