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

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

On 10/21/26 10:30 PM, Alexey Gladkov wrote:
> On Wed, Oct 21, 2026 at 09:52:39PM +0300, Leonid Krivoshein wrote:
>>>>> Документация у нас в 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 секунд на загрузку, верно?
> 
> Кажется моя реализация может накинуть времени пользователю в 1-2 секунды.
> Нужно поправить.
> 

Я добавлял две секунды как раз с учётом того, что для расчёта 
выполнялось округление до целых.


>> Можно ещё глянуть эти пару функций (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.
>>
>> _______________________________________________
>> Make-initrd mailing list
>> Make-initrd@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/make-initrd
> 

-- 
WBR, Leonid Krivoshein.