Make-initrd development discussion
 help / color / mirror / Atom feed
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: Mon, 26 Oct 2026 22:41:46 +0300
Message-ID: <f5b8945a-029b-47f9-855f-ff071e12f7ab@gmail.com> (raw)
In-Reply-To: <at-KtPijaWcETyfI@example.org>



On 10/26/26 8:15 PM, Alexey Gladkov wrote:
> On Mon, Oct 26, 2026 at 07:47:04PM +0300, Leonid Krivoshein wrote:
>>>> К сожалению, поздно вспомнил, что перетаскивание rootdelay_pause в коде было
>>>> связано ни с одной, а с двумя проблемами. Первую проблему обсудили и решили
>>>> выше. А вторая проблема была связана с периодическим выводом сообщений от
>>>> этого демона, что он ожидает корня. Когда мы работаем с диалогами на TTY2,
>>>> вперемешку с интерфейсом dialog иногда вылазили эти сообщения. Поэтому и
>>>> отключали этого демона на всё время работы интерактивщины.
>>>>
>>>> В идеале нужно:
>>>>
>>>> 1. Отключать учёт времени истечения загрузки только на определённые моменты:
>>>> это уже сделано.
>>>> 2. Отключать или перенаправлять сообщения от демона rootdelayd: этого пока
>>>> нет.
>>>>
>>>> Это две разные проблемы. Не совсем верно решать первым способом вторую
>>>> проблему, но сейчас у меня другого варианта просто нет. Возможно, я не прав
>>>> в том смысле, что в initrd, помимо rootdelayd, могут быть другие процессы,
>>>> которые могут что-то писать в активный TTY, просто мне таких пока не
>>>> попадалось.
>>>
>>> Новый код в rootdelayd теперь показывает это сообщение под блокировкой:
>>>
>>>     run_console_locked show_waiting_message
>>>
>>> так что если твой диалог на время работы будет брать блокировку, то
>>> сообщение показано не будет.
>>>
>>
>> С новой реализацией ещё не проверялось, но было видно, что сообщения
>> проскакивают между диалогами, хотя на время их вывода используется
>> rootdelay_pause.
> 
> Я не знаю как ты ставишь паузу, но если пауза стоит, то rootdelayd не
> может ничего выводить.
> 
> https://github.com/osboot/make-initrd/blob/master/features/runtime/data/sbin/rootdelayd#L13-L16
> 

Верно, не может. Если вызывать roodelay_pause() на время одного диалога, 
то пока с ним работаем, сообщений нет. Чтобы не было мелькания на экране 
при переходе от шага к шагу, экран не очищается, отработанный диалог 
остаётся на экране (TTY2) и в этот момент крайне редко между диалогами 
сообщение может проскочить, так как к этому моменту уже отработал 
rootdelay_unpause(). О чём я и говорю выше: сообщения иногда 
проскакивают *МЕЖДУ* диалогами, но результат виден на экране как артефакт.


>> Запускать всю цепочку в интерактивном режиме под блокировкой консоли
>> кажется сомнительной идеей, потому как может привести к взаимным
>> блокировкам.
> 
> Я не считаю, это сомнительной идеей. Твои диалоги занимают консоль.
> Блокировка как раз делается, чтобы разделить доступ к ней.
> 
>> Например, я жду настройки сети, а фича network ждёт консоли.
> 
> Фича network не ждёт console.
> 

Мой пример с network может и неудачный, но фичи могут использовать API 
make-initrd, и, если для их нормальной работы нужно будет в какой-то 
момент дождаться освобождения блокировки консоли, а шаг моей цепочки 
будет ожидать результата работы этой фичи, получится взаимная 
блокировка, к тому же весьма плохо диагностируемая. Поэтому мне идея 
блокировать консоль на всё время работы цепочки не нравится.


>> Отдельная ручка для управления этими сообщениями прям очень нужна.
> 
> Пока для меня это не очевидно. Ты приводишь какие-то надуманные с моей
> точки зрения аргументы.
> 

Вариант с атомарным созданием и удалением каталога мне нравится тем, что 
он крайне прост в реализации и исключает возможность взаимной блокировки.


>> Демон rootdelayd мог бы создавать при запуске каталог
>> /.initrd/rootdelay/show-wait-msg или что-то вроде того, и выводить это
>> сообщение только пока данный каталог существует. Если его нет, то просто не
>> выводить. Тогда bootchain, pipeline, kickstart и другие, кому нужно долго
>> работать с *РАЗНЫМИ* диалогами, при этом продолжать учитывать время
>> загрузки, могли бы на время своей работы данный каталог удалять, а по
>> окончании работы восстанавливать.
>>
>> Сейчас же приходится оборачивать всю цепочку в rootdelay_pause() /
>> rootdelay_unpause() только из-за этих сообщений.
> 
> Я пока не очень понимаю зачем твоей интерактивной программе на базе initrd
> вообще запущенный rootdelayd, который нужен в первую очередь для
> отслеживания параметра ROOTDELAY. Тебе он вообще не нужен.
> 

Учёт таймаута нужен для всего процесса загрузки, за исключением тех 
участков, про которые время ожидания заранее нельзя предсказать: 
диалоги, загрузка или проверка больших файлов, проверка или 
форматирование разделов. Правильно отключать отслеживание только на 
время выполнения таких участков.

Демон rootdelayd мне нужен для выполнения в фоне только одной из его 
задач -- отслеживанию таймаута, когда это разрешено. Но rootdelayd по 
ходу выполняет и вторую задачу. Он сам занимается интерактивным выводом 
в консоль, периодически напоминая пользователю, что ожидается корень, 
который до сих пор не найден. И вот эта его итеративность идёт вразрез с 
моей интерактивностью.

Сейчас стало понятно, как ещё можно обойти проблему: написать 
собственный аналог rootdelayd, который будет приостанавливать твой 
rootdelayd на время своей работы. Но это сложнее, чем сделать 
каталог-ручку для управления выводом сообщений от rootdelayd.


-- 
WBR, Leonid Krivoshein.



      reply	other threads:[~2026-10-26 19:41 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-12 22:42 [make-initrd] Параметр rdshell Leonid Krivoshein
2026-10-14 10:04 ` Egor Shestakov
2026-10-14 16:01   ` Leonid Krivoshein
2026-10-15 14:48     ` [make-initrd] [RFC PATCH] runtime: fix a suspended boot after rdshell Egor Shestakov
2026-10-16 10:20       ` Alexey Gladkov
2026-10-16 13:39         ` Egor Shestakov
2026-10-16 14:09           ` Alexey Gladkov
2026-10-16 14:13             ` Egor Shestakov
2026-10-16 14:11           ` Egor Shestakov
2026-10-19 10:22   ` [make-initrd] Параметр rdshell Alexey Gladkov
2026-10-19 12:51     ` [make-initrd] kmscon (was: Параметр rdshell) Egor Shestakov
2026-10-19 13:38       ` Alexey Gladkov
2026-10-19 15:09         ` Egor Shestakov
2026-10-19 16:14           ` Alexey Gladkov
2026-10-16 12:06 ` [make-initrd] [PATCH 0/2] runtime: Fix console locking Alexey Gladkov
2026-10-16 12:06   ` [make-initrd] [PATCH 1/2] runtime: Fix a lock leak in rootdelayd Alexey Gladkov
2026-10-16 12:06   ` [make-initrd] [PATCH 2/2] runtime: Fix console lock shadowing by /dev mount Alexey Gladkov
2026-10-16 20:02     ` Leonid Krivoshein
2026-10-17  9:45       ` Alexey Gladkov
2026-10-17 16:19         ` Leonid Krivoshein
2026-10-17 16:41           ` Alexey Gladkov
2026-10-19  9:57 ` [make-initrd] [PATCH v2 00/11] Improve support for atomic actions Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 01/11] runtime: Fix a lock leak in rootdelayd Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 02/11] runtime: Fix console lock shadowing by /dev mount Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 03/11] runtime: Avoid leaking lock descriptor 90 to child processes Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 04/11] runtime: Replace /.initrd/initenv locking with atomic file update Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 05/11] runtime: Centralize file and console locking helpers Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 06/11] create-rootfs: Require bash 4.1 or newer for initramfs Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 07/11] network: Share DHCP hook lock between IPv4 and IPv6 on the same interface Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 08/11] luks: Serialize /etc/luks.keys updates and avoid duplicate entries Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 09/11] runtime: Synchronize concurrent /etc/fstab access with shared and exclusive locks Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 10/11] runtime: Update rootdelay deadline and failure message atomically Alexey Gladkov
2026-10-19  9:57   ` [make-initrd] [PATCH v2 11/11] runtime: Centralize atomic file replacement in write_atomic helper Alexey Gladkov
2026-10-19 19:30   ` [make-initrd] [PATCH v2 00/11] Improve support for atomic actions Leonid Krivoshein
2026-10-20 14:59     ` Alexey Gladkov
2026-10-20 19:04       ` Leonid Krivoshein
2026-10-20 19:23         ` [make-initrd] Уточнение по rootdelay_unpause() Leonid Krivoshein
2026-10-21 10:10           ` [make-initrd] [PATCH] runtime: Freeze rootdelay countdown while paused Alexey Gladkov
2026-10-22 15:15             ` [make-initrd] [PATCH v2] " Alexey Gladkov
2026-10-22 22:14               ` Leonid Krivoshein
2026-10-22 22:26               ` Leonid Krivoshein
2026-10-22 22:37                 ` Leonid Krivoshein
2026-10-23  7:55                   ` Alexey Gladkov
2026-10-21  7:18         ` [make-initrd] [PATCH v2 00/11] Improve support for atomic actions Alexey Gladkov
2026-10-21  8:50           ` Leonid Krivoshein
2026-10-21 10:18             ` Alexey Gladkov
2026-10-21 18:52               ` Leonid Krivoshein
2026-10-21 19:30                 ` Alexey Gladkov
2026-10-21 19:33                   ` Leonid Krivoshein
2026-10-21 19:31                 ` Leonid Krivoshein
2026-10-21 20:03                   ` Alexey Gladkov
2026-10-21 20:48                     ` Leonid Krivoshein
2026-10-26  3:15               ` Leonid Krivoshein
2026-10-26 15:36                 ` Alexey Gladkov
2026-10-26 16:47                   ` Leonid Krivoshein
2026-10-26 17:15                     ` Alexey Gladkov
2026-10-26 19:41                       ` Leonid Krivoshein [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f5b8945a-029b-47f9-855f-ff071e12f7ab@gmail.com \
    --to=klark.devel@gmail.com \
    --cc=make-initrd@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Make-initrd development discussion

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
		make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
	public-inbox-index make-initrd

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.make-initrd


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git