ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Leonid Krivoshein <klark.devel@gmail.com>
To: devel@lists.altlinux.org
Subject: Re: [devel] [#214195] DONE installer.git=1.8.43-alt1
Date: Sat, 6 Oct 2018 17:40:31 +0300
Message-ID: <865dd49a-2ac4-50a6-a4a8-344b394c254f@gmail.com> (raw)
In-Reply-To: <20181006141907.GA18842@imap.altlinux.org>


06.10.2018 17:19, Michael Shigorin пишет:
> On Sat, Oct 06, 2018 at 04:57:51PM +0300, Leonid Krivoshein wrote:
>> Про юмор:
>>   > still broken for me: catches the keyword but reboots
>> Не удивительно, ведь чтобы need_poweroff включился, у тебя
>> /proc _не_должен_ быть смонтирован (!) и ещё много чего...
> Почему?  Это место как раз Глеб посмотрел (и нашёл там пару
> ошибок, ага :) -- если /proc уже был смонтирован, что сейчас
> _не_ так, то просто получим ещё один "на миллисекундочку"
> смонтированный поверх него.  Т.е. инвариант соблюдён.

Не, у меня так именно с /proc не получается:

mount -n -t proc none /proc
mount: /proc: none already mounted or mount point busy.

WIFEXITED(system(...)) - да, тоже непривычно и вопреки логике.
Ну да, оно всегда будет завершаться "за миллискундочку". :)


>
>> А если need_poweroff таки включился, /proc здесь же
>> отмонтируется (!), после чего запись в /proc/sysrq-trigger
>> не работает и много чего ещё, например, больше ничего уже
>> нельзя отмонтировать. :)
> По факту на момент ребута /proc остаётся смонтированным,
> могу прислать вещдок.  О, или сразу к баге его приложу, вот.

Я его уже видел:

Remains mounted: /dev
Remains mounted: /proc
Remains mounted: /destination
Remains mounted: /destination/dev


>> А по существу:
>>
>> Не похоже, что /etc/rc.d/init.d/halt из startup определяет
>> нынче логику выключения.
> Там всё врукопашную в этом же install2-init.c.

Отлично. Уже намного проще.
Вызвать /etc/rc.d/init.d/halt не вариант? В нём побольше нюансов учтено.


>
>> А с /destination так вообще костыли для бага --
>> в desktop-инсталляторе у меня ещё и рекурсивно раз 150 это всё
>> монтировалось.
> Это про livecd-install?

Угу.


>
>> То есть не со следствием бороться надо, а искать источник ошибки.
> ...и не смешивать совсем разные в одно. :)

Размонтировать корень не получится, пока не размонтируется всё остальное.
А здесь ещё может быть куча надстроек: LVM, RAID, шифрование... И не 
только размонтирование.
Если они не деактивируются при выключении, отсутствие fsck в initrd 
будет не самой страшной бедой.

Здесь же надо сначала понять, кто и когда съедает префикс "/mnt"...


-- 
Best regards,
Leonid Krivoshein.



  reply	other threads:[~2018-10-06 14:40 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-06  8:14 ` Michael Shigorin
2018-10-06 13:57   ` Leonid Krivoshein
2018-10-06 14:19     ` Michael Shigorin
2018-10-06 14:40       ` Leonid Krivoshein [this message]
2018-10-07  0:12         ` Alexey V. Vissarionov
2018-10-07  0:49     ` Leonid Krivoshein
2018-10-07  8:48       ` Leonid Krivoshein
2018-10-15 22:54   ` 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=865dd49a-2ac4-50a6-a4a8-344b394c254f@gmail.com \
    --to=klark.devel@gmail.com \
    --cc=devel@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

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/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 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

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


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