Hi, On Fri, Sep 23, 2022 at 5:05 PM John Ogness <[email protected]> wrote: > > From: Thomas Gleixner <[email protected]> > > Unprotected list walks are not necessarily safe. > > Signed-off-by: Thomas Gleixner <[email protected]> > Signed-off-by: John Ogness <[email protected]> > Reviewed-by: Sergey Senozhatsky <[email protected]> > --- > drivers/tty/serial/kgdboc.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/tty/serial/kgdboc.c b/drivers/tty/serial/kgdboc.c > index 79b7db8580e0..af2aa76bae15 100644 > --- a/drivers/tty/serial/kgdboc.c > +++ b/drivers/tty/serial/kgdboc.c > @@ -193,7 +193,8 @@ static int configure_kgdboc(void) > if (!p) > goto noconfig; > > - for_each_console(cons) { > + console_list_lock(); > + for_each_registered_console(cons) { > int idx; > if (cons->device && cons->device(cons, &idx) == p && > idx == tty_line) { > @@ -201,6 +202,7 @@ static int configure_kgdboc(void) > break; > } > } > + console_list_unlock();
Seems right to me, thanks! Reviewed-by: Douglas Anderson <[email protected]> _______________________________________________ Kgdb-bugreport mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
