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

From: Leonid Krivoshein <klark.devel@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: Re: [make-initrd] [PATCH v2 00/11] Improve support for atomic actions
Date: Tue, 20 Oct 2026 22:04:56 +0300
Message-ID: <bf06d3f1-55d0-4eeb-af71-f48706be8e30@gmail.com> (raw)
In-Reply-To: <ateB1ChMNdgCs7xk@example.org>

On 10/20/26 5:59 PM, Alexey Gladkov wrote:
> On Mon, Oct 19, 2026 at 10:30:38PM +0300, Leonid Krivoshein wrote:
>> Привет!
>>
>>
>> On 10/19/26 12:57 PM, Alexey Gladkov wrote:
>>> Нашлось ещё ряд мест, где потенциально могут возникнуть проблемы из-за
>>> параллельного доступа к файлам на чтение/запись.
>>>
>>> Я добавил новый API не только для flock-блокировок, но и для более атомарного
>>> обновления файлов.
>>>
>>> Вроде исправил всё что нашёл.
>>>
>>
>> Грандиозно! Выходит, иногда стоит задавать тут такие вопросы.
> 
> Всегда нужно было делать.
> 
> Других вопросов нет ? Тогда я их завтра применяю.
> 

Было бы здорово иметь одно место в документации, где будут перечислены 
все файлы, доступ к которым требует блокировок. Например, из этих 
коммитов я узнал, что следует блокировать доступ на запись в 
/.initrd/initenv. Инклюдить его можно по-прежнему без блокировки 
благодаря атомарному обновлению, если я правильно понял.

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

>>
>>>
>>> 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.