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

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

On 10/21/26 1:18 PM, Alexey Gladkov wrote:
> On Wed, Oct 21, 2026 at 11:50:15AM +0300, Leonid Krivoshein wrote:
>>
>> 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/ или ты про какую-то другую ?
>>>
>>
>> Про неё, да. Про список вида ресурс (тип блокировки) и файл блокировки.
> 
> Кто-нибудь может сделать такую документацию ? :)
> 

Наверное, да. Только сейчас работаю над значительным улучшением и 
документированием bootchain/altboot.


>> И раз уж заглянул... по-моему, проще так:
>>
>> get_majmin()
>> {
>>      mountpoint -x -- "$1" 2>/dev/null
>> }
>>
>> результат будет аналогичным.
> 
> Этой функции очень много лет. Она появилась до того как в образе появился
> mountpoint. Но да, твой код проще.
> 

Можно использовать, раз уж появился, я давно использую.


>>
>>>> Другой момент с этими коммитами не связан, но давно напрашивался. В 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() можно
>> сразу схлопотать таймаут. Обойти можно сбросом счётчика, но чтобы считалось
>> правильно, необходимо добавлять арифметику для расчёта нового дедлайна, чего
>> бы не хотелось делать на стороне клиента.
> 
> Предлагаемый патч в соседней ветке. Посмотри пожалуйста.
> 

Посмотрел. Да, что-то вроде того. Потому что всегда по факту были 
rootdelayd_pause() и rootdelayd_unpause(), а теперь как надо, без буквы 
d на конце. :-) Приостанавливалась реакция демона, а не отсчёт таймера.

 > If the timer is reset while paused, advance the recorded pause start 
timestamp to prevent adding duplicate time.

А как теперь будет работать следующий код если до окончания загрузки 
перед его вызовом оставалось 30 секунд?

rootdelay_reset_timer
rootdelay_unpause

По идее, пользователю должны добавить 180 секунд на загрузку, верно?

Можно ещё глянуть эти пару функций (longtermop_start и longtermop_end) 
[1] и весь патч [2] на предмет взаимодействия с make-initrd rootdelay 
из-вне -- здесь учитывается вложенность вызовов. Такая обёртка меня 
вполне устраивает, т.е. на стороне make-initrd вложенность можно не 
учитывать. Позже переделаю их с учётом атомарности.


1. 
https://github.com/klark973/make-initrd-bootchain/blob/sisyphus/bootchain-core/data/bin/bootchain-sh-functions#L200
2. 
https://github.com/klark973/make-initrd-bootchain/commit/8d3407240c214eee86b5421210b46ba4577da559


-- 
WBR, Leonid Krivoshein.