From: John Ogness <john.ogness@linutronix.de> To: Petr Mladek <pmladek@suse.com> Cc: Juergen Gross <jgross@suse.com>, linux-fbdev@vger.kernel.org, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Helge Deller <deller@gmx.de>, linux-kernel@vger.kernel.org, Steven Rostedt <rostedt@goodmis.org>, Javier Martinez Canillas <javierm@redhat.com>, Sergey Senozhatsky <senozhatsky@chromium.org>, dri-devel@lists.freedesktop.org, Thomas Zimmermann <tzimmermann@suse.de>, Tom Rix <trix@redhat.com>, Thomas Gleixner <tglx@linutronix.de>, Boris Ostrovsky <boris.ostrovsky@oracle.com> Subject: [PATCH printk v3 33/40] printk, xen: fbfront: create/use safe function for forcing preferred Date: Mon, 7 Nov 2022 15:22:31 +0106 [thread overview] Message-ID: <20221107141638.3790965-34-john.ogness@linutronix.de> (raw) In-Reply-To: <20221107141638.3790965-1-john.ogness@linutronix.de> With commit 9e124fe16ff2("xen: Enable console tty by default in domU if it's not a dummy") a hack was implemented to make sure that the tty console remains the console behind the /dev/console device. The main problem with the hack is that, after getting the console pointer to the tty console, it is assumed the pointer is still valid after releasing the console_sem. This assumption is incorrect and unsafe. Make the hack safe by introducing a new function console_force_preferred_locked() and perform the full operation under the console_list_lock. Signed-off-by: John Ogness <john.ogness@linutronix.de> --- drivers/video/fbdev/xen-fbfront.c | 12 +++------ include/linux/console.h | 1 + kernel/printk/printk.c | 44 ++++++++++++++++++++++++++++--- 3 files changed, 46 insertions(+), 11 deletions(-) diff --git a/drivers/video/fbdev/xen-fbfront.c b/drivers/video/fbdev/xen-fbfront.c index 4d2694d904aa..8752d389e382 100644 --- a/drivers/video/fbdev/xen-fbfront.c +++ b/drivers/video/fbdev/xen-fbfront.c @@ -504,18 +504,14 @@ static void xenfb_make_preferred_console(void) if (console_set_on_cmdline) return; - console_lock(); + console_list_lock(); for_each_console(c) { if (!strcmp(c->name, "tty") && c->index == 0) break; } - console_unlock(); - if (c) { - unregister_console(c); - c->flags |= CON_CONSDEV; - c->flags &= ~CON_PRINTBUFFER; /* don't print again */ - register_console(c); - } + if (c) + console_force_preferred_locked(c); + console_list_unlock(); } static int xenfb_resume(struct xenbus_device *dev) diff --git a/include/linux/console.h b/include/linux/console.h index cdae70e27377..b6b5d796d15c 100644 --- a/include/linux/console.h +++ b/include/linux/console.h @@ -273,6 +273,7 @@ enum con_flush_mode { }; extern int add_preferred_console(char *name, int idx, char *options); +extern void console_force_preferred_locked(struct console *con); extern void register_console(struct console *); extern int unregister_console(struct console *); extern void console_lock(void); diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index be40a9688403..d74e6e609f7d 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -247,9 +247,10 @@ int devkmsg_sysctl_set_loglvl(struct ctl_table *table, int write, void console_list_lock(void) { /* - * In unregister_console(), synchronize_srcu() is called with the - * console_list_lock held. Therefore it is not allowed that the - * console_list_lock is taken with the srcu_lock held. + * In unregister_console() and console_force_preferred_locked(), + * synchronize_srcu() is called with the console_list_lock held. + * Therefore it is not allowed that the console_list_lock is taken + * with the srcu_lock held. * * Whether or not this context is in the read-side critical section * can only be detected if the appropriate debug options are enabled. @@ -3457,6 +3458,43 @@ int unregister_console(struct console *console) } EXPORT_SYMBOL(unregister_console); +/** + * console_force_preferred_locked - force a registered console preferred + * @con: The registered console to force preferred. + * + * Must be called under console_list_lock(). + */ +void console_force_preferred_locked(struct console *con) +{ + struct console *cur_pref_con; + + if (!console_is_registered_locked(con)) + return; + + cur_pref_con = console_first(); + + /* Already preferred? */ + if (cur_pref_con == con) + return; + + hlist_del_init_rcu(&con->node); + + /* + * Ensure that all SRCU list walks have completed so that the console + * can be added to the beginning of the console list and its forward + * list pointer can be re-initialized. + */ + synchronize_srcu(&console_srcu); + + con->flags |= CON_CONSDEV; + WARN_ON(!con->device); + + /* Only the new head can have CON_CONSDEV set. */ + WRITE_ONCE(cur_pref_con->flags, cur_pref_con->flags & ~CON_CONSDEV); + hlist_add_behind_rcu(&con->node, console_list.first); +} +EXPORT_SYMBOL(console_force_preferred_locked); + /* * Initialize the console device. This is called *early*, so * we can't necessarily depend on lots of kernel help here. -- 2.30.2
WARNING: multiple messages have this Message-ID (diff)
From: John Ogness <john.ogness@linutronix.de> To: Petr Mladek <pmladek@suse.com> Cc: Sergey Senozhatsky <senozhatsky@chromium.org>, Steven Rostedt <rostedt@goodmis.org>, Thomas Gleixner <tglx@linutronix.de>, linux-kernel@vger.kernel.org, Helge Deller <deller@gmx.de>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Javier Martinez Canillas <javierm@redhat.com>, Thomas Zimmermann <tzimmermann@suse.de>, Juergen Gross <jgross@suse.com>, Boris Ostrovsky <boris.ostrovsky@oracle.com>, Tom Rix <trix@redhat.com>, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: [PATCH printk v3 33/40] printk, xen: fbfront: create/use safe function for forcing preferred Date: Mon, 7 Nov 2022 15:22:31 +0106 [thread overview] Message-ID: <20221107141638.3790965-34-john.ogness@linutronix.de> (raw) In-Reply-To: <20221107141638.3790965-1-john.ogness@linutronix.de> With commit 9e124fe16ff2("xen: Enable console tty by default in domU if it's not a dummy") a hack was implemented to make sure that the tty console remains the console behind the /dev/console device. The main problem with the hack is that, after getting the console pointer to the tty console, it is assumed the pointer is still valid after releasing the console_sem. This assumption is incorrect and unsafe. Make the hack safe by introducing a new function console_force_preferred_locked() and perform the full operation under the console_list_lock. Signed-off-by: John Ogness <john.ogness@linutronix.de> --- drivers/video/fbdev/xen-fbfront.c | 12 +++------ include/linux/console.h | 1 + kernel/printk/printk.c | 44 ++++++++++++++++++++++++++++--- 3 files changed, 46 insertions(+), 11 deletions(-) diff --git a/drivers/video/fbdev/xen-fbfront.c b/drivers/video/fbdev/xen-fbfront.c index 4d2694d904aa..8752d389e382 100644 --- a/drivers/video/fbdev/xen-fbfront.c +++ b/drivers/video/fbdev/xen-fbfront.c @@ -504,18 +504,14 @@ static void xenfb_make_preferred_console(void) if (console_set_on_cmdline) return; - console_lock(); + console_list_lock(); for_each_console(c) { if (!strcmp(c->name, "tty") && c->index == 0) break; } - console_unlock(); - if (c) { - unregister_console(c); - c->flags |= CON_CONSDEV; - c->flags &= ~CON_PRINTBUFFER; /* don't print again */ - register_console(c); - } + if (c) + console_force_preferred_locked(c); + console_list_unlock(); } static int xenfb_resume(struct xenbus_device *dev) diff --git a/include/linux/console.h b/include/linux/console.h index cdae70e27377..b6b5d796d15c 100644 --- a/include/linux/console.h +++ b/include/linux/console.h @@ -273,6 +273,7 @@ enum con_flush_mode { }; extern int add_preferred_console(char *name, int idx, char *options); +extern void console_force_preferred_locked(struct console *con); extern void register_console(struct console *); extern int unregister_console(struct console *); extern void console_lock(void); diff --git a/kernel/printk/printk.c b/kernel/printk/printk.c index be40a9688403..d74e6e609f7d 100644 --- a/kernel/printk/printk.c +++ b/kernel/printk/printk.c @@ -247,9 +247,10 @@ int devkmsg_sysctl_set_loglvl(struct ctl_table *table, int write, void console_list_lock(void) { /* - * In unregister_console(), synchronize_srcu() is called with the - * console_list_lock held. Therefore it is not allowed that the - * console_list_lock is taken with the srcu_lock held. + * In unregister_console() and console_force_preferred_locked(), + * synchronize_srcu() is called with the console_list_lock held. + * Therefore it is not allowed that the console_list_lock is taken + * with the srcu_lock held. * * Whether or not this context is in the read-side critical section * can only be detected if the appropriate debug options are enabled. @@ -3457,6 +3458,43 @@ int unregister_console(struct console *console) } EXPORT_SYMBOL(unregister_console); +/** + * console_force_preferred_locked - force a registered console preferred + * @con: The registered console to force preferred. + * + * Must be called under console_list_lock(). + */ +void console_force_preferred_locked(struct console *con) +{ + struct console *cur_pref_con; + + if (!console_is_registered_locked(con)) + return; + + cur_pref_con = console_first(); + + /* Already preferred? */ + if (cur_pref_con == con) + return; + + hlist_del_init_rcu(&con->node); + + /* + * Ensure that all SRCU list walks have completed so that the console + * can be added to the beginning of the console list and its forward + * list pointer can be re-initialized. + */ + synchronize_srcu(&console_srcu); + + con->flags |= CON_CONSDEV; + WARN_ON(!con->device); + + /* Only the new head can have CON_CONSDEV set. */ + WRITE_ONCE(cur_pref_con->flags, cur_pref_con->flags & ~CON_CONSDEV); + hlist_add_behind_rcu(&con->node, console_list.first); +} +EXPORT_SYMBOL(console_force_preferred_locked); + /* * Initialize the console device. This is called *early*, so * we can't necessarily depend on lots of kernel help here. -- 2.30.2
next prev parent reply other threads:[~2022-11-07 14:17 UTC|newest] Thread overview: 129+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-11-07 14:15 [PATCH printk v3 00/40] reduce console_lock scope John Ogness 2022-11-07 14:15 ` John Ogness 2022-11-07 14:15 ` John Ogness 2022-11-07 14:15 ` John Ogness 2022-11-07 14:15 ` [PATCH printk v3 01/40] rcu: implement lockdep_rcu_enabled for !CONFIG_DEBUG_LOCK_ALLOC John Ogness 2022-11-07 18:01 ` Paul E. McKenney 2022-11-07 19:23 ` John Ogness 2022-11-08 19:27 ` Paul E. McKenney 2022-11-09 17:49 ` Paul E. McKenney 2022-11-08 10:29 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 02/40] serial: kgdboc: Lock console list in probe function John Ogness 2022-11-09 8:20 ` Daniel Thompson 2022-11-07 14:16 ` [PATCH printk v3 03/40] printk: Convert console_drivers list to hlist John Ogness 2022-11-07 14:16 ` [PATCH printk v3 04/40] printk: Prepare for SRCU console list protection John Ogness 2022-11-08 12:14 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 05/40] printk: fix setting first seq for consoles John Ogness 2022-11-08 12:20 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 06/40] um: kmsg_dump: only dump when no output console available John Ogness 2022-11-07 14:16 ` John Ogness 2022-11-08 12:41 ` Petr Mladek 2022-11-08 12:41 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 07/40] console: introduce console_is_enabled() wrapper John Ogness 2022-11-08 16:18 ` Petr Mladek 2022-11-10 15:05 ` John Ogness 2022-11-11 13:32 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 08/40] printk: use console_is_enabled() John Ogness 2022-11-08 17:40 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 09/40] um: kmsg_dump: " John Ogness 2022-11-07 14:16 ` John Ogness 2022-11-09 14:52 ` Petr Mladek 2022-11-09 14:52 ` Petr Mladek 2022-11-09 14:56 ` Petr Mladek 2022-11-09 14:56 ` Petr Mladek 2022-11-09 14:57 ` Petr Mladek 2022-11-09 14:57 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 10/40] kdb: kdb_io: " John Ogness 2022-11-09 8:22 ` Daniel Thompson 2022-11-09 14:59 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 11/40] um: kmsg_dumper: use srcu console list iterator John Ogness 2022-11-07 14:16 ` John Ogness 2022-11-09 15:05 ` Petr Mladek 2022-11-09 15:05 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 12/40] tty: serial: kgdboc: document console_lock usage John Ogness 2022-11-09 8:23 ` Daniel Thompson 2022-11-09 15:19 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 13/40] tty: tty_io: " John Ogness 2022-11-09 15:20 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 14/40] proc: consoles: " John Ogness 2022-11-09 15:22 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 15/40] kdb: use srcu console list iterator John Ogness 2022-11-09 8:53 ` Daniel Thompson 2022-11-09 9:27 ` John Ogness 2022-11-09 15:27 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 16/40] printk: console_flush_all: " John Ogness 2022-11-07 14:16 ` [PATCH printk v3 17/40] printk: console_unblank: " John Ogness 2022-11-07 14:16 ` [PATCH printk v3 18/40] printk: console_flush_on_panic: " John Ogness 2022-11-07 14:16 ` [PATCH printk v3 19/40] printk: console_device: " John Ogness 2022-11-09 15:58 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 20/40] printk: __pr_flush: " John Ogness 2022-11-07 14:16 ` [PATCH printk v3 21/40] printk: introduce console_list_lock John Ogness 2022-11-10 12:58 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 22/40] console: introduce console_is_registered() John Ogness 2022-11-10 13:00 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 23/40] serial_core: replace uart_console_enabled() with uart_console_registered() John Ogness 2022-11-08 8:46 ` Geert Uytterhoeven 2022-11-10 13:24 ` Petr Mladek 2022-11-10 13:46 ` John Ogness 2022-11-07 14:16 ` [PATCH printk v3 24/40] tty: nfcon: use console_is_registered() John Ogness 2022-11-08 8:39 ` Geert Uytterhoeven 2022-11-10 13:58 ` Petr Mladek 2022-11-10 14:19 ` John Ogness 2022-11-10 17:50 ` Eero Tamminen 2022-11-07 14:16 ` [PATCH printk v3 25/40] efi: earlycon: " John Ogness 2022-11-10 14:28 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 26/40] tty: hvc: " John Ogness 2022-11-07 14:16 ` John Ogness 2022-11-10 14:52 ` Petr Mladek 2022-11-10 14:52 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 27/40] tty: serial: earlycon: " John Ogness 2022-11-10 15:00 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 28/40] tty: serial: pic32_uart: " John Ogness 2022-11-10 15:01 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 29/40] tty: serial: samsung_tty: " John Ogness 2022-11-07 14:16 ` John Ogness 2022-11-10 15:01 ` Petr Mladek 2022-11-10 15:01 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 30/40] tty: serial: xilinx_uartps: " John Ogness 2022-11-07 14:16 ` John Ogness 2022-11-10 15:04 ` Petr Mladek 2022-11-10 15:04 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 31/40] usb: early: xhci-dbc: " John Ogness 2022-11-10 15:05 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 32/40] netconsole: avoid CON_ENABLED misuse to track registration John Ogness 2022-11-10 15:17 ` Petr Mladek 2022-11-07 14:16 ` John Ogness [this message] 2022-11-07 14:16 ` [PATCH printk v3 33/40] printk, xen: fbfront: create/use safe function for forcing preferred John Ogness 2022-11-10 15:34 ` Petr Mladek 2022-11-10 15:34 ` Petr Mladek 2022-11-10 16:03 ` John Ogness 2022-11-10 16:03 ` John Ogness 2022-11-10 17:26 ` Petr Mladek 2022-11-10 17:26 ` Petr Mladek 2022-11-10 22:37 ` John Ogness 2022-11-10 22:37 ` John Ogness 2022-11-11 10:48 ` Petr Mladek 2022-11-11 10:48 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 34/40] tty: tty_io: use console_list_lock for list synchronization John Ogness 2022-11-07 14:16 ` [PATCH printk v3 35/40] proc: consoles: use console_list_lock for list iteration John Ogness 2022-11-07 14:16 ` [PATCH printk v3 36/40] tty: serial: kgdboc: use console_list_lock for list traversal John Ogness 2022-11-09 9:06 ` Daniel Thompson 2022-11-09 9:44 ` John Ogness 2022-11-10 18:00 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 37/40] tty: serial: kgdboc: synchronize tty_find_polling_driver() and register_console() John Ogness 2022-11-10 15:13 ` Daniel Thompson 2022-11-10 18:03 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 38/40] tty: serial: kgdboc: use console_list_lock to trap exit John Ogness 2022-11-10 15:18 ` Daniel Thompson 2022-11-11 9:59 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 39/40] printk: relieve console_lock of list synchronization duties John Ogness 2022-11-07 16:30 ` John Ogness 2022-11-11 10:27 ` Petr Mladek 2022-11-11 13:06 ` Petr Mladek 2022-11-07 14:16 ` [PATCH printk v3 40/40] tty: serial: sh-sci: use setup() callback for early console John Ogness 2022-11-08 8:53 ` Geert Uytterhoeven 2022-11-11 17:15 ` Petr Mladek 2022-11-11 14:43 ` [PATCH printk v3 00/40] reduce console_lock scope Mathieu Desnoyers 2022-11-11 14:43 ` Mathieu Desnoyers 2022-11-11 14:43 ` Mathieu Desnoyers 2022-11-11 14:43 ` Mathieu Desnoyers
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20221107141638.3790965-34-john.ogness@linutronix.de \ --to=john.ogness@linutronix.de \ --cc=boris.ostrovsky@oracle.com \ --cc=deller@gmx.de \ --cc=dri-devel@lists.freedesktop.org \ --cc=gregkh@linuxfoundation.org \ --cc=javierm@redhat.com \ --cc=jgross@suse.com \ --cc=linux-fbdev@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=pmladek@suse.com \ --cc=rostedt@goodmis.org \ --cc=senozhatsky@chromium.org \ --cc=tglx@linutronix.de \ --cc=trix@redhat.com \ --cc=tzimmermann@suse.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.