On Fri 2019-04-26 14:32:59, Sergey Senozhatsky wrote:
> The following pattern in register_console() is not completely safe:
>
> for_each_console(bcon)
> if (bcon->flags & CON_BOOT)
> unregister_console(bcon);
>
> Because, in theory, console drivers list and console drivers
> can be modified concurrently from another CPU. We need to grab
> console_sem lock, which protects console drivers list and console
> drivers, before we start iterating console drivers list.
>
> Factor out __unregister_console(), which will be called from
> unregister_console() and register_console(), in both cases
> under console_sem lock.
>
> diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c
> index 17102fd4c136..b0e361ca1bea 100644
> --- a/kernel/printk/printk.c
> +++ b/kernel/printk/printk.c
> @@ -2605,6 +2605,48 @@ static int __init keep_bootcon_setup(char *str)
>
> early_param("keep_bootcon", keep_bootcon_setup);
>
> +static int __unregister_console(struct console *console)
> +{
> + struct console *a, *b;
> + int res;
> +
> + pr_info("%sconsole [%s%d] disabled\n",
> + (console->flags & CON_BOOT) ? "boot" : "",
> + console->name, console->index);
> +
> + res = _braille_unregister_console(console);
It looks safe to call _braille_unregister_console() under
console_lock().
Therefore this patch looks fine and make sense, so:
Reviewed-by: Petr Mladek <[email protected]>
Best Regards,
Petr
