Re: [make-initrd] Fwd: [#269003] TESTED make-initrd.git=2.14.1-alt1

[Leonid Krivoshein <klark.devel@gmail.com>]

26.05.2021 21:12, Alexey Gladkov пишет:
> On Wed, May 26, 2021 at 06:05:29PM +0300, Leonid Krivoshein wrote:
>> [...]
>> Учёл практически все твои замечания. Всё ещё делаю обещанное, однако дело
>> уже близится к концу. В коде ещё остаются захардкореными некоторые пути и
>> имена, это потом уберётся в альт-специфичный конфиг.
>>
>> #271420 -- предыдущая версия "всё в одном", около 3 тыс. строк. По ней легче
>> понять задумку. Тут предварительно отлажена локальная загрузка. altboot
>> сделан шагом pipeline, который выполняет роль вложенной pipeline.
>>
>> #272587 -- текущая версия разбита на десяток подпакетов, уже более 4200
>> строк, устранены первичные замечания, но понять проще предыдущий вариант.
>> Тут тоже работает пока только локальная загрузка, но уже с плимутом, liverw
>> надо переделывать, ещё не проверялись: nfs, cifs и overlayroot. Однако
>> оценить задумку можно уже сейчас. Документации пока тоже нет. Здесь
>> отказался от вложенной pipeline, сделал форк в bootchain и значительное
>> расширение его функционала, обратная совместимость с pipeline сохранена.
> Это невозможно отревьювить в виде одного коммита.

Что ты имеешь ввиду? Сделать так, чтобы сначала были отдельно видны 
изменения в коде pipeline несколькими коммитами, остальное доложить ещё 
одним коммитом? Или что слишком большой объём просто нет возможности 
ревьювить? Второе, конечно, понятно. А первое... в принципе, pipeline 
отделён сейчас в bootchain-core и два отдельных метода загрузки 
bootchain-getimage и bootchain-waitdev. Для последнего добавлен 
опциональный общий тайм-аут, и теперь его можно использовать до altboot 
(localdev), что позволит использовать подход пропагатора с диалогами и 
сканированием устройств как fallback, если заданное в /proc/cmdline не 
будет найдено, зато waitdev позволяет более тонко задавать спецификацию 
искомого и искать несколько устройств, включая символьные. То есть, не 
внося изменений в код altboot, можно красиво пристроить слева всю 
цепочку pipeline. А так по дефолту мы собираем образы с root=bootchain 
bootchain=fg,altboot и всё, что было в пропагаторе, сохранено для 
совместимости. Разве что добавили UUID к методам disk и cdrom (в 
automatic=...).


> Верю тебе на слово :)

Спасибо за доверие. :-) Но я "работаю" не один, за мной проверяет на 
образах Антон Мидюков, некоторые коллеги тоже на это посматривают пока 
издалека, надеюсь, скоро присоединятся тестировщики. Не хочется, чтобы 
вышел факап в p10 с заменой пропагатора.


>> Хотел посоветоваться насчёт setsid, слишком неожиданно для меня его
>> поведение в теле скрипта, запущенного через openvt. Он ведёт себя так, будто
>> бы его нет, приходится использовать амперсанд (&). При этом, когда я делаю
>> то же самое руками в обычном терминале, setsid ведёт себя ожидаемым образом.
>> Т.е. одна и та же команда работает по-разному:
> openvt сам сделает setsid, если только ты не делаешь openvt --exec .

Нет, exec не делаю. Но разве запущенный таким образом скрипт не может 
запускать setsid ещё раз? В какой-то момент мне нужно от него снова 
"отделиться" (форкнуться). У тебя в make-initrd setsid используется 
всего в нескольких местах, но очевидно работает ожидаемым образом.


>> setsid sleep 10
>>
>> работает как:
>>
>> sleep 10
>>
>> при открытии через openvt, а в консоли работает как:
>>
>> sleep 10 &
> если ты имеешь в виду, что процесс уходит в бэкграунд, то это делает
> openvt. Он может так не делать, если выполнять его с опцией --exec.

Вопрос терминологии -- у меня всё наоборот. pipeline и bootchain -- 
демоны, они уже работают в фоне, а мне в какой-то момент необходимо 
продолжить выполнение на переднем плане, на конкретном терминале. Это 
отделение происходит в /sbin/bootchain-loop, конкретная реализация в 
/bin/interactive-sh-functions, см. IM_exec() и IM_activate(). В общем, 
тут удивляет, почему форкнутый процесс нельзя снова форкнуть таким 
образом. Потому что не хочется отслеживать потомков и иметь дело с 
зомби, не люблю эти ужастики. ))


>> речь об этом фрагменте:
>> bootchain-interactive/data/bin/interactive-sh-functions
>>
>> ...
>> setsid /bin/bash -c "/bin/activate-interactive-vt $delay" &
>> message "TTY${_IM_VT_number} will be activated after $((1 + delay)) seconds"
>> ...
>>
>> (в нынешнем варианте задумка работает, но я считаю это неправильно).


-- 
Best regards,
Leonid Krivoshein.