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

[Alexey Gladkov <legion@kernel.org>]

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/ или ты про какую-то другую ?
> > 
> 
> Про неё, да. Про список вида ресурс (тип блокировки) и файл блокировки.

Кто-нибудь может сделать такую документацию ? :)

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

Предлагаемый патч в соседней ветке. Посмотри пожалуйста.

-- 
Rgrds, legion