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 19:47:04 +0300
Message-ID: <80b1a55a-056d-45b6-8aae-d2d2db9bf218@gmail.com> (raw)
In-Reply-To: <at9zgi1HzrDU_Krv@example.org>
On 10/26/26 6:36 PM, Alexey Gladkov wrote:
> On Mon, Oct 26, 2026 at 06:15:56AM +0300, Leonid Krivoshein wrote:
>> Привет!
>>
>>
>> On 10/21/26 1:18 PM, Alexey Gladkov wrote:
>>>>>> Другой момент с этими коммитами не связан, но давно напрашивался. В v2.36.0
>>>>>> была добавлена rootdelay_reset_timer(), здесь проскочило её
>>>>>> усовершенствование. На время диалогов ввода и на время заведомо длительных
>>>>>> или непредсказуемых по времени выполнения операций хотелось бы не просто
>>>>>> приостанавливать отчёт таймера rootdelay, но и возобновлять его отсчёт, а не
>>>>>> просто сбрасывать. Сейчас я его просто сбрасываю, так тоже работает, но это
>>>>>> конечно увеличивает суммарный (фактический) rootdelay.
>>>>> Почему на время диалогов ты не хочешь сделать rootdelay_pause ?
>>>>>
>>>> Как раз делаю.
>>>>
>>>>
>>>>> Поставить rootdelay на паузу для диалогов мне кажется правильной идеей. Я
>>>>> паузу как раз для этого использую в fsck и kickstart. Для последнего
>>>>> rootdelay отключается по сути вообще.
>>>>>
>>>> Раньше тоже так делал. Если что-то пошло не так, мы не увидим ни stage2, ни
>>>> отладочного rdshell. Поэтому сейчас сделаны две вещи: 1) таймаут отключается
>>>> только на время длительных операций, диалогов, а после rootdelay
>>>> перезапускается, 2) при фатальном выходе chaind устанавливает дедлайн на
>>>> now+2 секунды, чтобы поскорее попасть в rdshell.
>>
>> К сожалению, поздно вспомнил, что перетаскивание rootdelay_pause в коде было
>> связано ни с одной, а с двумя проблемами. Первую проблему обсудили и решили
>> выше. А вторая проблема была связана с периодическим выводом сообщений от
>> этого демона, что он ожидает корня. Когда мы работаем с диалогами на TTY2,
>> вперемешку с интерфейсом dialog иногда вылазили эти сообщения. Поэтому и
>> отключали этого демона на всё время работы интерактивщины.
>>
>> В идеале нужно:
>>
>> 1. Отключать учёт времени истечения загрузки только на определённые моменты:
>> это уже сделано.
>> 2. Отключать или перенаправлять сообщения от демона rootdelayd: этого пока
>> нет.
>>
>> Это две разные проблемы. Не совсем верно решать первым способом вторую
>> проблему, но сейчас у меня другого варианта просто нет. Возможно, я не прав
>> в том смысле, что в initrd, помимо rootdelayd, могут быть другие процессы,
>> которые могут что-то писать в активный TTY, просто мне таких пока не
>> попадалось.
>
> Новый код в rootdelayd теперь показывает это сообщение под блокировкой:
>
> run_console_locked show_waiting_message
>
> так что если твой диалог на время работы будет брать блокировку, то
> сообщение показано не будет.
>
С новой реализацией ещё не проверялось, но было видно, что сообщения
проскакивают между диалогами, хотя на время их вывода используется
rootdelay_pause. Запускать всю цепочку в интерактивном режиме под
блокировкой консоли кажется сомнительной идеей, потому как может
привести к взаимным блокировкам. Например, я жду настройки сети, а фича
network ждёт консоли.
Отдельная ручка для управления этими сообщениями прям очень нужна.
Демон rootdelayd мог бы создавать при запуске каталог
/.initrd/rootdelay/show-wait-msg или что-то вроде того, и выводить это
сообщение только пока данный каталог существует. Если его нет, то просто
не выводить. Тогда bootchain, pipeline, kickstart и другие, кому нужно
долго работать с *РАЗНЫМИ* диалогами, при этом продолжать учитывать
время загрузки, могли бы на время своей работы данный каталог удалять, а
по окончании работы восстанавливать.
Сейчас же приходится оборачивать всю цепочку в rootdelay_pause() /
rootdelay_unpause() только из-за этих сообщений.
--
WBR, Leonid Krivoshein.
next prev parent reply other threads:[~2026-10-26 16:47 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 [this message]
2026-10-26 17:15 ` Alexey Gladkov
2026-10-26 19:41 ` Leonid Krivoshein
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=80b1a55a-056d-45b6-8aae-d2d2db9bf218@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