Re: [devel] installer: init: Killing all remaining processes

Re: [devel] installer: init: Killing all remaining processes

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 891 bytes --]

On Wed, Apr 16, 2008 at 06:01:27AM +0400, Evgeny Sinelnikov wrote:
[...]
> >  ECHILD может не случиться, если в системе остался хотя бы один процесс
> >  с ppid==1, который не собирается завершаться.  На нашем стенде это
> >  происходит всегда.
> 
> Даже когда ему SIGKILL отправили не отрывается... ? Хм... Ну, всякое
> бывает, но это какие-то проблемы ядра. Сизифный вариант на 2.6.24
> такого не давал ни разу.... Теперь понятно почему ранее не выявлялся
> баг с killall()...

Поведение сизифных ядер 2.6.18 и 2.6.24 заметно различаются.

2.6.18: SIGKILL не приводит у завершению всех процессов, ECHILD не
происходит, LOOP_CHANGE_FD на файл-дырку проходит безболезненно.

2.6.24: SIGKILL приводит к завершению всех процессов, ECHILD происходит,
LOOP_CHANGE_FD на файл-дырку приводит к зависанию.

installer-0.4-alt14 должен работать на обоих типах ядер.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] installer: init: Killing all remaining processes

[Stanislav Ievlev]

On Wed, Apr 16, 2008 at 07:40:13PM +0400, Dmitry V. Levin wrote:
> On Wed, Apr 16, 2008 at 06:01:27AM +0400, Evgeny Sinelnikov wrote:
> [...]
> > >  ECHILD может не случиться, если в системе остался хотя бы один процесс
> > >  с ppid==1, который не собирается завершаться.  На нашем стенде это
> > >  происходит всегда.
> > 
> > Даже когда ему SIGKILL отправили не отрывается... ? Хм... Ну, всякое
> > бывает, но это какие-то проблемы ядра. Сизифный вариант на 2.6.24
> > такого не давал ни разу.... Теперь понятно почему ранее не выявлялся
> > баг с killall()...
> 
> Поведение сизифных ядер 2.6.18 и 2.6.24 заметно различаются.
> 
> 2.6.18: SIGKILL не приводит у завершению всех процессов, ECHILD не
> происходит, LOOP_CHANGE_FD на файл-дырку проходит безболезненно.
Иногда тоже зависает.

Re: [devel] installer: init: Killing all remaining processes

[Anton V. Boyarshinov]

On Wed, 16 Apr 2008 19:40:13 +0400 Dmitry V. Levin
 wrote:

> installer-0.4-alt14 должен работать на обоих типах ядер.
не подтверждаю: ftp://ftp.altlinux.ru/pvt/people/boyarsh/DSC_6506.JPG

Если кратко, то Ooops на 2.6.24-std-def

Антон

Re: [devel] installer: init: Killing all remaining processes

[Evgeny Sinelnikov]

2008/4/17 Anton V. Boyarshinov <boyarsh@altlinux.org>:
> On Wed, 16 Apr 2008 19:40:13 +0400 Dmitry V. Levin
>   wrote:
>
>  > installer-0.4-alt14 должен работать на обоих типах ядер.
>  не подтверждаю: ftp://ftp.altlinux.ru/pvt/people/boyarsh/DSC_6506.JPG
>
>  Если кратко, то Ooops на 2.6.24-std-def
>

Странно, у меня на 2.6.24-std-def-alt7 сработало...

-- 
Sin (Sinelnikov Evgeny)

Re: [devel] installer: init: Killing all remaining processes

[Anton V. Boyarshinov]

> >  Если кратко, то Ooops на 2.6.24-std-def
> Странно, у меня на 2.6.24-std-def-alt7 сработало...
Если это race, то может быть всё что угодно..

Re: [devel] installer: init: Killing all remaining processes

[Stanislav Ievlev]

On Thu, Apr 17, 2008 at 01:48:43PM +0400, Evgeny Sinelnikov wrote:
> 2008/4/17 Anton V. Boyarshinov <boyarsh@altlinux.org>:
> > On Wed, 16 Apr 2008 19:40:13 +0400 Dmitry V. Levin
> >   wrote:
> >
> >  > installer-0.4-alt14 должен работать на обоих типах ядер.
> >  не подтверждаю: ftp://ftp.altlinux.ru/pvt/people/boyarsh/DSC_6506.JPG
> >
> >  Если кратко, то Ooops на 2.6.24-std-def
> >
> 
> Странно, у меня на 2.6.24-std-def-alt7 сработало...
У нас машинка SMP.

Валилось два раза, оба после Umount. При этом второй список неотмонтированных систем даже не выводится.
Один раз был молчаливый упс, второй раз kernel panic из-за выхода init'a и дальше backtrace с упоминанием read.
Есть подозрение что работа с файлом /proc/mounts  (а больше ни с чем в
этот момент работы и не велось) на 2.6.24  происходит совсем другим способом и он на пол-пути теряется.

Re: [devel] installer: init: Killing all remaining processes

[Evgeny Sinelnikov]

2008/4/17 Anton V. Boyarshinov <boyarsh@altlinux.org>:
> > >  Если кратко, то Ooops на 2.6.24-std-def
>  > Странно, у меня на 2.6.24-std-def-alt7 сработало...
>  Если это race, то может быть всё что угодно..
>

Это похоже не на race, a на то, что копирование файла init в память не
является полным решением проблем на 2.6.24. Если это и race, то
вероятно ядерный.

Если проблема повторяется предлагаю проверить инсталятор, в котором
вместо пустого файла в память копируется оригинальный образ, и
loop_change_fd() делается на оригинальный файл.

Кроме того интересно было бы увидеть проявиться ли этот баг на
insatller-0.4-alt13.eter1:
http://git.etersoft.ru/people/sin/packages/installer.git/
Сборку можно взять здесь:
http://ftp.etersoft.ru/pub/download/installer/i586/RPMS.hasher/

-- 
Sin (Sinelnikov Evgeny)

Re: [devel] installer: init: Killing all remaining processes

[Anton V. Boyarshinov]

> Странно, у меня на 2.6.24-std-def-alt7 сработало...
Всё ещё любопытственнее. Сборка на 2.6.24-std-def-alt7 сервера работает нормально, а gnome-cd стабильно выдаёт в этом месте Ooops

Re: [devel] installer: init: Killing all remaining processes

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 1020 bytes --]

On Thu, Apr 17, 2008 at 02:54:19PM +0400, Stanislav Ievlev wrote:
> On Thu, Apr 17, 2008 at 01:48:43PM +0400, Evgeny Sinelnikov wrote:
> > 2008/4/17 Anton V. Boyarshinov <boyarsh@altlinux.org>:
> > > On Wed, 16 Apr 2008 19:40:13 +0400 Dmitry V. Levin wrote:
> > >
> > >  > installer-0.4-alt14 должен работать на обоих типах ядер.
> > >  не подтверждаю: ftp://ftp.altlinux.ru/pvt/people/boyarsh/DSC_6506.JPG
> > >
> > >  Если кратко, то Ooops на 2.6.24-std-def
> > 
> > Странно, у меня на 2.6.24-std-def-alt7 сработало...
> У нас машинка SMP.
> 
> Валилось два раза, оба после Umount. При этом второй список неотмонтированных систем даже не выводится.
> Один раз был молчаливый упс, второй раз kernel panic из-за выхода init'a и дальше backtrace с упоминанием read.
> Есть подозрение что работа с файлом /proc/mounts  (а больше ни с чем в
> этот момент работы и не велось) на 2.6.24  происходит совсем другим способом и он на пол-пути теряется.

Я там ещё немного накоммитил, попробуйте.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] installer: init: Killing all remaining processes

[Anton V. Boyarshinov]

> Я там ещё немного накоммитил, попробуйте.
Визуальные изменения: появились надписи unmounting /oldroot/....
Потом всё равно Oops, снимок сделан.

А

[devel] installer: M40 ack

[Michael Shigorin]

On Thu, Apr 17, 2008 at 11:48:03PM +0400, Dmitry V. Levin wrote:
> > > >  > installer-0.4-alt14 должен работать на обоих типах ядер.
> Я там ещё немного накоммитил, попробуйте.

На 2.6.18 отработало, вроде всё хорошо.

Забрасываю бэкпорт с revert'нутым 0.4-alt9 в /i/u/4.0.

2 inger: угораздило же тебя сделать то изменение как раз
перед этими разборками и школьными выпусками... ;-)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] installer: M40 ack

[Dmitry V. Levin]

[-- Attachment #1: Type: text/plain, Size: 580 bytes --]

On Mon, Apr 21, 2008 at 06:46:38PM +0300, Michael Shigorin wrote:
> On Thu, Apr 17, 2008 at 11:48:03PM +0400, Dmitry V. Levin wrote:
> > > > >  > installer-0.4-alt14 должен работать на обоих типах ядер.
> > Я там ещё немного накоммитил, попробуйте.
> 
> На 2.6.18 отработало, вроде всё хорошо.

На десктопе, собранном на kernel-image-std-def-2.6.24-alt7, kernel panic
во время чтения /proc/mounts воспроизводится с вероятностью, близкой к 1,
так что не всё ещё ясно.

> Забрасываю бэкпорт с revert'нутым 0.4-alt9 в /i/u/4.0.

Подождал бы лучше -alt15.


-- 
ldv

[-- Attachment #2: Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] installer: M40 ack

[Evgeny Sinelnikov]

[...]

>  > Забрасываю бэкпорт с revert'нутым 0.4-alt9 в /i/u/4.0.
>
>  Подождал бы лучше -alt15.

+1

Мои тесты показывают, что даже текущие изменения решают определённые
проблемы - падать перестаёт...

-- 
Sin (Sinelnikov Evgeny)

Re: [devel] installer: M40 ack

[Michael Shigorin]

On Mon, Apr 21, 2008 at 07:54:32PM +0400, Evgeny Sinelnikov wrote:
> >  > Забрасываю бэкпорт с revert'нутым 0.4-alt9 в /i/u/4.0.
> >  Подождал бы лучше -alt15.
> +1

Хорошо, жду -- пока в репо для сборки Линукс Терминал будет.

> Мои тесты показывают, что даже текущие изменения решают
> определённые проблемы - падать перестаёт...

И завершается будто нормально.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] installer: M40 ack

[Stanislav Ievlev]

On Mon, Apr 21, 2008 at 06:46:38PM +0300, Michael Shigorin wrote:
> On Thu, Apr 17, 2008 at 11:48:03PM +0400, Dmitry V. Levin wrote:
> > > > >  > installer-0.4-alt14 должен работать на обоих типах ядер.
> > Я там ещё немного накоммитил, попробуйте.
> 
> На 2.6.18 отработало, вроде всё хорошо.
> 
> Забрасываю бэкпорт с revert'нутым 0.4-alt9 в /i/u/4.0.
> 
> 2 inger: угораздило же тебя сделать то изменение как раз
> перед этими разборками и школьными выпусками... ;-)
Я делаю всё для 4.1 ;)

Re: [devel] installer: M40 ack

[Michael Shigorin]

On Tue, Apr 22, 2008 at 12:01:43PM +0400, Stanislav Ievlev wrote:
> > > > > >  > installer-0.4-alt14 должен работать на обоих типах ядер.
> > > Я там ещё немного накоммитил, попробуйте.
> > На 2.6.18 отработало, вроде всё хорошо.
> > Забрасываю бэкпорт с revert'нутым 0.4-alt9 в /i/u/4.0.
> > 2 inger: угораздило же тебя сделать то изменение как раз
> > перед этими разборками и школьными выпусками... ;-)
> Я делаю всё для 4.1 ;)

На будущее просьба: делая однострочные изменения, ломающие
совместимость -- стоит подумать о том, есть ли вещи, которые
всё равно придётся фиксить и для старой ветки.

По науке это называется "список приоритетов", в котором изменения
насчёт install2-init.c получаются более важными, чем новые идеи.

Уж больно досадно ревертить ровно один махонький коммит.
(может, сделаешь там какое условие для совместимости и с M40, 
и с M41?)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/