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