[make-initrd] [PATCH 0/2] runtime: Fix console locking

[Alexey Gladkov <gladkov.alexey@gmail.com>]

Кажется я нашёл причину почему rdshell стартует до завершения всех сервисов.
Причина этого ещё один баг с fd_lock 90 /dev/console.

По задумке при старте runlevel 3 rc захватывает fd_lock 90 /dev/console и держит
его до конца запуска всех S*-сервисов. Поэтому rootdelayd, запущенный в
S01:rootdelay, должен был ждать на fd_lock 90 /dev/console, пока `rc 3` не
запустит все сервисы.

Но в момент старта /etc/rc.d/rc 3 сервис S04:mountfs ещё не отработал, и /dev
это обычная директория на ramfs (/). Поэтому `rc 3` лочит inode /dev/console на
ramfs.

Потом S04:mountfs монтирует поверх /dev файловую систему devtmpfs (или tmpfs).
Теперь /dev/console это новый inode на devtmpfs, а старый inode, залоченный
процессом `rc 3`, оказывается перекрыт точкой монтирования /dev.

Когда rootdelayd просыпается после `sleep 0.2` как раз во время работы
udev/uevent, он вызывает fd_lock 90 /dev/console уже на новом inode /dev/console
в devtmpfs, который никем не залочен.

В итоге rootdelayd не ждёт окончания `rc 3`, а сразу запускает rdshell
параллельно с `rc 3`.

То есть у нас тут две пересекающиеся проблемы с блокировками. Я не хотел плодить
lock-файлы, но использовать /dev/console оказалось плохой идеей.

Я запушил в for-master патч Егора и свой фикс. Тесты на altlinux уже успешно
прошли.

https://github.com/osboot/make-initrd/compare/for-master?expand=1


Alexey Gladkov (1):
  runtime: Fix console lock shadowing by /dev mount

Egor Shestakov (1):
  runtime: Fix a lock leak in rootdelayd

 features/fsck/data/lib/uevent/handlers/040-mountdev |  2 +-
 features/luks/data/lib/uevent/handlers/085-luks     |  2 +-
 features/network/data/etc/rc.d/init.d/network-up    |  3 ---
 features/runtime/data/etc/rc.d/rc                   |  2 +-
 features/runtime/data/init                          |  1 +
 .../data/lib/uevent/handlers/mount/900-localdev     |  2 +-
 features/runtime/data/sbin/rootdelayd               | 13 ++++++++-----
 .../data/lib/uevent/handlers/040-sshfsroot          |  2 +-
 features/zfs/data/sbin/ueventd-zfs                  |  2 +-
 9 files changed, 15 insertions(+), 14 deletions(-)

-- 
2.56.0