Messages in this thread Patch in this message | | | From | John Ogness <> | Subject | [PATCH printk v1 13/18] printk: nobkl: Add write context storage for atomic writes | Date | Thu, 2 Mar 2023 21:02:13 +0106 |
| |
From: Thomas Gleixner <tglx@linutronix.de>
The number of consoles is unknown at compile time and allocating write contexts on stack in emergency/panic situations is not desired either.
Allocate a write context array (one for each priority level) along with the per CPU output buffers, thus allowing atomic contexts on multiple CPUs and priority levels to execute simultaneously without clobbering each other's write context.
Co-developed-by: John Ogness <john.ogness@linutronix.de> Signed-off-by: John Ogness <john.ogness@linutronix.de> Signed-off-by: Thomas Gleixner (Intel) <tglx@linutronix.de> --- include/linux/console.h | 2 ++ kernel/printk/internal.h | 5 +++++ 2 files changed, 7 insertions(+)
diff --git a/include/linux/console.h b/include/linux/console.h index 710f1e72cd0f..089a94a3dd8d 100644 --- a/include/linux/console.h +++ b/include/linux/console.h @@ -222,6 +222,7 @@ struct cons_state { * @CONS_PRIO_NORMAL: Regular printk * @CONS_PRIO_EMERGENCY: Emergency output (WARN/OOPS...) * @CONS_PRIO_PANIC: Panic output + * @CONS_PRIO_MAX: The number of priority levels * * Emergency output can carefully takeover the console even without consent * of the owner, ideally only when @cons_state::unsafe is not set. Panic @@ -234,6 +235,7 @@ enum cons_prio { CONS_PRIO_NORMAL, CONS_PRIO_EMERGENCY, CONS_PRIO_PANIC, + CONS_PRIO_MAX, }; struct console; diff --git a/kernel/printk/internal.h b/kernel/printk/internal.h index a72402c1ac93..a417e3992b7a 100644 --- a/kernel/printk/internal.h +++ b/kernel/printk/internal.h @@ -181,11 +181,16 @@ struct printk_message { /** * struct cons_context_data - console context data + * @wctxt: Write context per priority level * @pbufs: Buffer for storing the text * * Used for early boot and for per CPU data. + * + * The write contexts are allocated to avoid having them on stack, e.g. in + * warn() or panic(). */ struct cons_context_data { + struct cons_write_context wctxt[CONS_PRIO_MAX]; struct printk_buffers pbufs; }; -- 2.30.2
| |