Re: [make-initrd] [RFC PATCH] runtime: fix a suspended boot after rdshell

[Alexey Gladkov <legion@kernel.org>]

On Thu, Oct 15, 2026 at 02:48:48PM +0000, Egor Shestakov wrote:
> Привет, я похоже нашел причину почему все зависает. Применил патч у себя и
> все теперь работает нормально.

Простите за задержку. Уехал на openssl конференцию и думал, что никто не
заметит. И пропустил всю движуху :)

Да, ты прав. Это бага из-за рефакторинга. Исторически эта ошибка появилась
в два шага:

1. В коммите 598a4b241a ("Rewrite console locking") блокировка через
директорию /.initrd/rdshell.lock была заменена на `fd_lock 90 /dev/console`.
На тот момент логика жила в короткоживущем uevent-хендлере 400-rootdelay,
который делал:

  fd_lock 90 /dev/console
  rdshell "Emergency shell" &
  exit

Сам хендлер тут же завершался (exit), закрывая у себя fd 90, а фоновый
процесс rdshell наследовал fd 90 и удерживал flock на /dev/console, пока
пользователь не выйдет из шелла.

2. В коммите 5bba8d357a ("Convert rootdelay to service") одноразовый
хендлер превратился в постоянный демон rootdelayd с бесконечным циклом, а
exit механически заменили на continue.

В результате rootdelayd берёт эксклюзивный лок fd_lock 90 /dev/console,
запускает rdshell & и уходит на continue, где бесконечно крутится в:

   if [ -e "$disabled" ] || rootdelay_paused; then
      sleep $period                                                                                                                                                                                                                                      
      continue                                                                                                                                                                                                                                           
   fi

Когда пользователь нажимает Ctrl-D в rdshell, процесс rdshell завершается,
но демон rootdelayd остаётся жить с открытым дескриптором 90 и никогда не
вызывает fd_unlock 90.


> -- >8 --
> 
> Rdshell takes a write lock to /dev/console but never release it. Rdshell
> can run in foreground, because the rootdelayd doesn't wait for anything
> else and fails in an idle loop after rdshell has been invoked.
> 
> Signed-off-by: Egor Shestakov <ved@altlinux.org>
> ---
>  features/runtime/data/sbin/rootdelayd | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/features/runtime/data/sbin/rootdelayd b/features/runtime/data/sbin/rootdelayd
> index 7ea82afcf..85060cefd 100755
> --- a/features/runtime/data/sbin/rootdelayd
> +++ b/features/runtime/data/sbin/rootdelayd
> @@ -37,7 +37,8 @@ while :; do
>  		mkdir -p -- "$disabled"
>  
>  		fd_lock 90 /dev/console
> -		rdshell "Emergency shell" &
> +		rdshell "Emergency shell"
> +		fd_unlock 90
>  
>  		continue
>  	fi
> @@ -91,7 +92,9 @@ while :; do
>  
>  		mkdir -p -- "$disabled"
>  
> -		rdshell "$msg" &
> +		rdshell "$msg"
> +
> +		fd_unlock 90
>  
>  		continue
>  	fi

Патч правильный, но неполный. В твоём пвтче дескриптор 90 унаследуется
rdshell и остальными утилитами из него. Правильнее закрыть дескриптор:

  fd_lock 90 /dev/console
  rdshell "Emergency shell" 90<&-
  fd_unlock 90

и во втором месте тоже.

-- 
Rgrds, legion