Re: [make-initrd] udhcpc script в фиче network

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

24.09.2021 22:00, Alexey Gladkov пишет:
> On Fri, Sep 24, 2021 at 08:31:50PM +0300, Leonid Krivoshein wrote:
>> [...]
>>> Также я уже говорил, что я думаю про дёргание /lib/initrd/cmdline.d/network.
>>>
>>> http://git.altlinux.org/tasks/index/sisyphus/tested/285722/gears/100/git?p=git;a=blob;f=bootchain-waitnet/data/lib/bootchain/waitnet;h=9f555ab3a44c83e2c54c949f2d9401d6f6995ed2;hb=0b2522e81ea6279295bae02a8aa77e84c86d3918#l56
>>>
>>> Этот шаг можно выполнить лишь один раз.
>> Не согласен! :-)
> Ты делаешь внутри условия запись:
>
> if [ ! -s "$NETBOOT_IFNAME" ]; then
> 	...
> 	 echo "$netdev" >"$NETBOOT_IFNAME"
> fi
>
> Больше ты этот файл нигде не пишешь:
>
> http://git.altlinux.org/tasks/index/sisyphus/tested/285722/gears/100/git?p=git&a=search&h=0b2522e81ea6279295bae02a8aa77e84c86d3918&st=grep&s=NETBOOT_IFNAME
>
> то есть после записи в этот файл этот if никогда больше не выполнится.
> Этот шаг можно выполнить лишь один раз.

В коде два if'а -- первый препятствует повторному дёрганью фичи network, 
второй, про который ты теперь говоришь, чтобы не проверять заново сеть, 
если ранее эта проверка уже была выполнена. Но и внутри второго if'а 
есть ещё один if -- если сеть поднялась, только в этом случае создаётся 
файл, а диалог возникает, если этот if не выполнен. И да, мы можем 
оказаться в этом месте снова благодаря вызову alboot_restart() -- он 
заставляет bootchain перезапустить цепочку с самого начала, с шага altboot.

Таким образом возникает ситуация, при которой если даже сеть попытались 
поднять, но она не поднялась, мы можем повторять эту попытку, не дёргая 
повторно network. На моей памяти только один случай, когда сразу после 
подключения сети настройки выдавались не менее 20 секунд (битый ethernet 
кабель). Фактически это место возможности возраста для повторения 
попытки. Диалог -- о том, что может стоит переткнуть провод в другой 
штекер или проверить соединение.


>> [...]
>>> http://git.altlinux.org/tasks/index/sisyphus/tested/285722/gears/100/git?p=git;a=blob;f=bootchain-waitnet/data/lib/bootchain/waitnet;h=9f555ab3a44c83e2c54c949f2d9401d6f6995ed2;hb=0b2522e81ea6279295bae02a8aa77e84c86d3918#l65
>>>
>>> Хардкод имени интерфейса. is_loopback().
>> Не только, это ещё и break loop / защита от пустого каталога. Обычно я тут
>> ставлю "_".
> Тогда так и нужно писать:
>
> for d in /sys/class/net/*; do
> 	[ -d "$d" ] || continue
> 	...
> done
>
> И не нужно отдельно вписывать lo. Он всегда там есть.

В принципе, вариант, но у меня везде шебанг с set -f. ОК, подумаем над 
улучшением этого места. Всё равно waitnet не останется в нынешнем виде.



-- 
Best regards,
Leonid Krivoshein.