powerpc: Remove initialisation of readpos
authorMatthew Wilcox (Oracle) <willy@infradead.org>
Tue, 24 Oct 2023 14:55:59 +0000 (15:55 +0100)
committerSteven Rostedt (Google) <rostedt@goodmis.org>
Thu, 26 Oct 2023 01:25:51 +0000 (21:25 -0400)
While powerpc doesn't use the seq_buf readpos, it did explicitly
initialise it for no good reason.

Link: https://lore.kernel.org/linux-trace-kernel/20231024145600.739451-1-willy@infradead.org
Cc: Christoph Hellwig <hch@lst.de>
Cc: Justin Stitt <justinstitt@google.com>
Cc: Kent Overstreet <kent.overstreet@linux.dev>
Cc: Petr Mladek <pmladek@suse.com>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Cc: Sergey Senozhatsky <senozhatsky@chromium.org>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Reviewed-by: Kees Cook <keescook@chromium.org>
Fixes: d0ed46b60396 ("tracing: Move readpos from seq_buf to trace_seq")
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org>
arch/powerpc/kernel/setup-common.c

index 2f1026f..3497553 100644 (file)
@@ -601,7 +601,6 @@ struct seq_buf ppc_hw_desc __initdata = {
        .buffer = ppc_hw_desc_buf,
        .size = sizeof(ppc_hw_desc_buf),
        .len = 0,
-       .readpos = 0,
 };
 
 static __init void probe_machine(void)