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



  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