On Mon 2022-11-07 15:22:08, John Ogness wrote: > Replace (console->flags & CON_ENABLED) usage with console_is_enabled(). > > Signed-off-by: John Ogness <[email protected]> > Reviewed-by: Petr Mladek <[email protected]> > Reviewed-by: Douglas Anderson <[email protected]> > --- > kernel/debug/kdb/kdb_io.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/kernel/debug/kdb/kdb_io.c b/kernel/debug/kdb/kdb_io.c > index 67d3c48a1522..550fe8b456ec 100644 > --- a/kernel/debug/kdb/kdb_io.c > +++ b/kernel/debug/kdb/kdb_io.c > @@ -559,7 +559,7 @@ static void kdb_msg_write(const char *msg, int msg_len) > } > > for_each_console(c) { > - if (!(c->flags & CON_ENABLED)) > + if (!console_is_enabled(c))
Same as with the 9th patch. I would prefer to merge this with the patch switching to the srcu console list iterator. It will explain why the racy check is needed here. This change does not make sense without the other. Best Regards, Petr _______________________________________________ Kgdb-bugreport mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
