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] Fwd: [#269003] TESTED make-initrd.git=2.14.1-alt1
Date: Thu, 27 May 2021 18:10:48 +0300
Message-ID: <6dd33811-5270-8869-7628-b026d56f23d7@gmail.com> (raw)
In-Reply-To: <20210527135347.s6e5rr7t7ipz6uc3@example.org>


27.05.2021 16:53, Alexey Gladkov пишет:
> On Thu, May 27, 2021 at 03:29:50PM +0300, Leonid Krivoshein wrote:
> [...]
>>> Я лишь могу посмотреть на разные куски кода и покомментировать.
>> Это безусловно очень полезно, твои советы особенно ценны для меня, хоть и
>> жаль твоего времени, т.к. код пока не финальный. За вчера добил liverw,
>> сейчас чиню checksum. Всё же я имел ввиду оценить идею, в целом. Например,
>> мне кажется, разделение на суб-фичи для bootchain само напрашивается. Но вот
>> стоит ли делить на под-пакеты -- вопрос?
> Если это упростит сопровождение, то почему бы нет. Вопрос в том, насколько
> будет сложно реализовать суб-фичи.
>

Глядя на другие проще. И ещё сделал в корне mix-altboot для 
разработчиков. Он не только смешивает все шаги в один каталог data 
подобно тому, как это делает make-initrd при создании образа, но и 
создаёт подкаталог hooks, в котором "склеивает" все пронумерованные 
хуки, чтобы легче было понять, куда вставлять свой новый код. Ищутся они 
через git grep use_hooks и будут описаны на ВиКи.


>> [...] Могу попробовать ещё раз.
> У тебя могло бы быть что-то вроде:
>
> save_image() { # right
> 	if [ -z "$dstreg" ]; then
> 		dd "of=$to" bs=32k 2>/dev/null
> 	else
> 		cat >"$to"
> 	fi
> }
>
> read_image() { # left
> 	if [ -n "$srcreg" ]; then
> 		pv -n -i 1 -- "$url"
> 		return
> 	fi
>
> 	opts="$opts --silent --no-buffer --connect-timeout 5"
> 	opts="$opts --max-redirs 5 --max-filesize $filesize"
> 	
> 	[ "$method" != ftp ] || [ -z "$user" ] || [ -z "$pass" ] ||
> 		opts="${opts:+$opts }-u \"$user:$pass\""
>
> 	curl $opts -- "$url" |
> 		pv -n -i 1 -s "$filesize"
> }
>
> { read_image | save_image } 2>&1 |
> 	IM_gauge "[ Downloading image... ]" "$text"

Спасибо, попробую.


> [...]
>>> Зачем ты пытаешься использовать ttyN ? Тебе не хватает /dev/console ?
>> Нужно собрать диск с таксом, чтобы было наглядней. Хотя уже скоро соберу
>> новый. Сейчас "новый пропагатор" (altboot) вообще как бы "невидим".
>> Переключение на tty2 происходит спустя некое время, либо сразу, если там
>> есть диалог ввода. Поскольку в большинстве случаев загрузка автоматическая,
>> ввода от пользователя не требуется и занимает считанные секунды, создаваемые
>> "новым пропагатором" и bootchain консоли tty2 и tty3 бесследно исчезают, не
>> создавая лишних мельканий на экране. Ещё красивее с плимутом -- всё
>> происходит под "червячком", а вот если tty2 активируется, то отключается ещё
>> и rootdelay.
> Ааааа ... я забыл про plymouth.

Если ты про BUG #40090 , то я тестирую на срезе Сизифа, где make-initrd 
ещё 2.16.0 без этой баги, но обход уже известен.


>>> [...]
>> Да я собственно из-за этого фрагмента и написал тебе, переделывал его раз
>> 20, но так и не понял. Выше ты пишешь, что да, можно запускать setsid второй
>> раз, это и есть как бы второй запуск (первый -- openvt). Если убрать тут
>> setsid, результат будет таким же. Если убрать не setsid, а амперсанд в
>> конце, тут будет задержка в $selay, чего я никак не мог победить. В обычной
>> консоли оно себя так не ведёт. Потому и стоит сейчас if, а вообще
>> предполагалось так: setsid activate-interactive-vt $delay безо всяких
>> проверок и условий.
> Там написано, что форк будет сделан только если просто sedsid отвалится.
> "But doesn't fail if shell is not interactive". У тебя он как раз не
> интерактивный. Поэтому setsid activate-interactive-vt просто висит.

Так вот оно в чём дело! Тогда понятно... Обойдусь одним амперсандом. И, 
судя по этой реализации setsid, за зомбоками там тоже предлагается самим 
присматривать. :-) У меня вызов wait уже стоит перед выходом.


-- 
Best regards,
Leonid Krivoshein.



  reply	other threads:[~2021-05-27 15:10 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-05 20:33 ` Leonid Krivoshein
2021-04-05 22:51   ` Leonid Krivoshein
2021-04-06  8:44     ` Alexey Gladkov
2021-04-06 17:38       ` Leonid Krivoshein
2021-04-07 13:13         ` Alexey Gladkov
2021-04-06  8:28   ` Alexey Gladkov
2021-04-06 16:38     ` Leonid Krivoshein
2021-04-06 19:05       ` Alexey Gladkov
2021-04-06 19:30         ` Alexey Gladkov
2021-04-06 23:13           ` Leonid Krivoshein
2021-04-07 12:28             ` Alexey Gladkov
2021-04-06 23:00         ` Leonid Krivoshein
2021-04-07 12:11           ` Alexey Gladkov
2021-04-06 23:59         ` Leonid Krivoshein
2021-04-07  1:51     ` Leonid Krivoshein
2021-04-07 12:57       ` Alexey Gladkov
2021-04-07 18:29         ` Leonid Krivoshein
2021-05-26 15:05         ` Leonid Krivoshein
2021-05-26 18:12           ` Alexey Gladkov
2021-05-26 19:25             ` Leonid Krivoshein
2021-05-27  8:37               ` Alexey Gladkov
2021-05-27 12:29                 ` Leonid Krivoshein
2021-05-27 13:53                   ` Alexey Gladkov
2021-05-27 15:10                     ` Leonid Krivoshein [this message]
2021-05-27 17:04                       ` Alexey Gladkov
2021-05-27 17:11                         ` Leonid Krivoshein
2021-05-30 20:34                     ` 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=6dd33811-5270-8869-7628-b026d56f23d7@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