Re: [make-initrd] [PATCH v2 00/11] Improve support for atomic actions

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

On 10/21/26 10:18 AM, Alexey Gladkov wrote:
> On Tue, Oct 20, 2026 at 10:04:56PM +0300, Leonid Krivoshein wrote:
>> Было бы здорово иметь одно место в документации, где будут перечислены все
>> файлы, доступ к которым требует блокировок. Например, из этих коммитов я
>> узнал, что следует блокировать доступ на запись в /.initrd/initenv.
>> Инклюдить его можно по-прежнему без блокировки благодаря атомарному
>> обновлению, если я правильно понял.
> 
> Сейчас с твоей подачи такой список стал организовываться в
> features/runtime/data/bin/initrd-sh-functions вместе с API для блокировок.
> 

Отлично, спасибо!


> Документация у нас в Documentation/ или ты про какую-то другую ?
> 

Про неё, да. Про список вида ресурс (тип блокировки) и файл блокировки.


И раз уж заглянул... по-моему, проще так:

get_majmin()
{
     mountpoint -x -- "$1" 2>/dev/null
}

результат будет аналогичным.


>> Другой момент с этими коммитами не связан, но давно напрашивался. В v2.36.0
>> была добавлена rootdelay_reset_timer(), здесь проскочило её
>> усовершенствование. На время диалогов ввода и на время заведомо длительных
>> или непредсказуемых по времени выполнения операций хотелось бы не просто
>> приостанавливать отчёт таймера rootdelay, но и возобновлять его отсчёт, а не
>> просто сбрасывать. Сейчас я его просто сбрасываю, так тоже работает, но это
>> конечно увеличивает суммарный (фактический) rootdelay.
> 
> Почему на время диалогов ты не хочешь сделать rootdelay_pause ?
> 

Как раз делаю.


> Поставить rootdelay на паузу для диалогов мне кажется правильной идеей. Я
> паузу как раз для этого использую в fsck и kickstart. Для последнего
> rootdelay отключается по сути вообще.
> 

Раньше тоже так делал. Если что-то пошло не так, мы не увидим ни stage2, 
ни отладочного rdshell. Поэтому сейчас сделаны две вещи: 1) таймаут 
отключается только на время длительных операций, диалогов, а после 
rootdelay перезапускается, 2) при фатальном выходе chaind устанавливает 
дедлайн на now+2 секунды, чтобы поскорее попасть в rdshell.

Cпич о том, что текущая реализация отталкивается от сохранённого 
дедлайна, а не учёта времени, потраченного на загрузку. В моём 
понимании, rootdelay_pause() -- перестать расходовать rootdelay на 
неопределённое время. Сейчас же получается, что после вызова 
rootdelay_unpause() можно сразу схлопотать таймаут. Обойти можно сбросом 
счётчика, но чтобы считалось правильно, необходимо добавлять арифметику 
для расчёта нового дедлайна, чего бы не хотелось делать на стороне клиента.


>>
>>>>
>>>>>
>>>>> Alexey Gladkov (10):
>>>>>      runtime: Fix console lock shadowing by /dev mount
>>>>>      runtime: Avoid leaking lock descriptor 90 to child processes
>>>>>      runtime: Replace /.initrd/initenv locking with atomic file update
>>>>>      runtime: Centralize file and console locking helpers
>>>>>      create-rootfs: Require bash 4.1 or newer for initramfs
>>>>>      network: Share DHCP hook lock between IPv4 and IPv6 on the same
>>>>>        interface
>>>>>      luks: Serialize /etc/luks.keys updates and avoid duplicate entries
>>>>>      runtime: Synchronize concurrent /etc/fstab access with shared and
>>>>>        exclusive locks
>>>>>      runtime: Update rootdelay deadline and failure message atomically
>>>>>      runtime: Centralize atomic file replacement in write_atomic helper
>>>>>
>>>>> Egor Shestakov (1):
>>>>>      runtime: Fix a lock leak in rootdelayd
>>>>>
>>>>>     .../uevent/handlers/btrfs-member/100-timeout  |   4 +-
>>>>>     .../data/lib/uevent/handlers/040-mountdev     |  19 +--
>>>>>     .../luks/data/lib/uevent/filters/lukskeys     |  49 +++---
>>>>>     .../luks/data/lib/uevent/handlers/085-luks    |  33 ++--
>>>>>     .../network/data/bin/network-sh-functions     |   1 +
>>>>>     .../network/data/etc/rc.d/init.d/network-up   |   3 -
>>>>>     features/network/data/lib/network/hostname    |  18 ++-
>>>>>     features/network/data/lib/network/resolvconf  |  84 +++++-----
>>>>>     .../network/data/lib/network/udhcpc4.script   |  90 +++++------
>>>>>     features/runtime/data/bin/initrd-sh-functions | 123 ++++++++++++++-
>>>>>     features/runtime/data/etc/rc.d/init.d/cmdline |   6 +-
>>>>>     features/runtime/data/etc/rc.d/init.d/fstab   |   4 +-
>>>>>     features/runtime/data/etc/rc.d/rc             | 148 +++++++++---------
>>>>>     features/runtime/data/init                    |   2 +
>>>>>     .../data/lib/initrd/post/fstab/mountsrc       |  61 ++++----
>>>>>     .../runtime/data/lib/uevent/filters/mountdev  |  71 +++++----
>>>>>     .../lib/uevent/handlers/mount/900-localdev    |  39 +++--
>>>>>     features/runtime/data/sbin/rootdelayd         |  74 ++++-----
>>>>>     .../data/lib/uevent/handlers/040-sshfsroot    |  26 ++-
>>>>>     features/zfs/data/lib/initrd/post/fstab/zfs   | 105 +++++++------
>>>>>     features/zfs/data/sbin/ueventd-zfs            |  21 ++-
>>>>>     .../tests/ts0044-member-timeout-queue-api/run |   2 +
>>>>>     tools/create-rootfs                           |   2 +
>>>>>     23 files changed, 566 insertions(+), 419 deletions(-)
>>>>>
>>>>
>>>> -- 
>>>> WBR, Leonid Krivoshein.
>>>>
>>>> _______________________________________________
>>>> Make-initrd mailing list
>>>> Make-initrd@lists.altlinux.org
>>>> https://lists.altlinux.org/mailman/listinfo/make-initrd
>>>
>>
>> -- 
>> WBR, Leonid Krivoshein.
>>
>> _______________________________________________
>> Make-initrd mailing list
>> Make-initrd@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/make-initrd
> 

-- 
WBR, Leonid Krivoshein.