mirror of
https://github.com/torvalds/linux.git
synced 2024-09-23 08:23:44 +00:00
9ce02f0fc6
The macro __WARN_FLAGS() uses a local variable named "f". This being a
common name, there is a risk of shadowing other variables.
For example, GCC would yield:
| In file included from ./include/linux/bug.h:5,
| from ./include/linux/cpumask.h:14,
| from ./arch/x86/include/asm/cpumask.h:5,
| from ./arch/x86/include/asm/msr.h:11,
| from ./arch/x86/include/asm/processor.h:22,
| from ./arch/x86/include/asm/timex.h:5,
| from ./include/linux/timex.h:65,
| from ./include/linux/time32.h:13,
| from ./include/linux/time.h:60,
| from ./include/linux/stat.h:19,
| from ./include/linux/module.h:13,
| from virt/lib/irqbypass.mod.c:1:
| ./include/linux/rcupdate.h: In function 'rcu_head_after_call_rcu':
| ./arch/x86/include/asm/bug.h:80:21: warning: declaration of 'f' shadows a parameter [-Wshadow]
| 80 | __auto_type f = BUGFLAG_WARNING|(flags); \
| | ^
| ./include/asm-generic/bug.h:106:17: note: in expansion of macro '__WARN_FLAGS'
| 106 | __WARN_FLAGS(BUGFLAG_ONCE | \
| | ^~~~~~~~~~~~
| ./include/linux/rcupdate.h:1007:9: note: in expansion of macro 'WARN_ON_ONCE'
| 1007 | WARN_ON_ONCE(func != (rcu_callback_t)~0L);
| | ^~~~~~~~~~~~
| In file included from ./include/linux/rbtree.h:24,
| from ./include/linux/mm_types.h:11,
| from ./include/linux/buildid.h:5,
| from ./include/linux/module.h:14,
| from virt/lib/irqbypass.mod.c:1:
| ./include/linux/rcupdate.h:1001:62: note: shadowed declaration is here
| 1001 | rcu_head_after_call_rcu(struct rcu_head *rhp, rcu_callback_t f)
| | ~~~~~~~~~~~~~~~^
For reference, sparse also warns about it, c.f. [1].
This patch renames the variable from f to __flags (with two underscore
prefixes as suggested in the Linux kernel coding style [2]) in order
to prevent collisions.
[1] https://lore.kernel.org/all/CAFGhKbyifH1a+nAMCvWM88TK6fpNPdzFtUXPmRGnnQeePV+1sw@mail.gmail.com/
[2] Linux kernel coding style, section 12) Macros, Enums and RTL,
paragraph 5) namespace collisions when defining local variables in
macros resembling functions
https://www.kernel.org/doc/html/latest/process/coding-style.html#macros-enums-and-rtl
Fixes: bfb1a7c91f
("x86/bug: Merge annotate_reachable() into_BUG_FLAGS() asm")
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
Acked-by: Josh Poimboeuf <jpoimboe@redhat.com>
Link: https://lkml.kernel.org/r/20220324023742.106546-1-mailhol.vincent@wanadoo.fr
90 lines
2.4 KiB
C
90 lines
2.4 KiB
C
/* SPDX-License-Identifier: GPL-2.0 */
|
|
#ifndef _ASM_X86_BUG_H
|
|
#define _ASM_X86_BUG_H
|
|
|
|
#include <linux/stringify.h>
|
|
#include <linux/instrumentation.h>
|
|
#include <linux/objtool.h>
|
|
|
|
/*
|
|
* Despite that some emulators terminate on UD2, we use it for WARN().
|
|
*/
|
|
#define ASM_UD2 ".byte 0x0f, 0x0b"
|
|
#define INSN_UD2 0x0b0f
|
|
#define LEN_UD2 2
|
|
|
|
#ifdef CONFIG_GENERIC_BUG
|
|
|
|
#ifdef CONFIG_X86_32
|
|
# define __BUG_REL(val) ".long " __stringify(val)
|
|
#else
|
|
# define __BUG_REL(val) ".long " __stringify(val) " - 2b"
|
|
#endif
|
|
|
|
#ifdef CONFIG_DEBUG_BUGVERBOSE
|
|
|
|
#define _BUG_FLAGS(ins, flags, extra) \
|
|
do { \
|
|
asm_inline volatile("1:\t" ins "\n" \
|
|
".pushsection __bug_table,\"aw\"\n" \
|
|
"2:\t" __BUG_REL(1b) "\t# bug_entry::bug_addr\n" \
|
|
"\t" __BUG_REL(%c0) "\t# bug_entry::file\n" \
|
|
"\t.word %c1" "\t# bug_entry::line\n" \
|
|
"\t.word %c2" "\t# bug_entry::flags\n" \
|
|
"\t.org 2b+%c3\n" \
|
|
".popsection\n" \
|
|
extra \
|
|
: : "i" (__FILE__), "i" (__LINE__), \
|
|
"i" (flags), \
|
|
"i" (sizeof(struct bug_entry))); \
|
|
} while (0)
|
|
|
|
#else /* !CONFIG_DEBUG_BUGVERBOSE */
|
|
|
|
#define _BUG_FLAGS(ins, flags, extra) \
|
|
do { \
|
|
asm_inline volatile("1:\t" ins "\n" \
|
|
".pushsection __bug_table,\"aw\"\n" \
|
|
"2:\t" __BUG_REL(1b) "\t# bug_entry::bug_addr\n" \
|
|
"\t.word %c0" "\t# bug_entry::flags\n" \
|
|
"\t.org 2b+%c1\n" \
|
|
".popsection\n" \
|
|
extra \
|
|
: : "i" (flags), \
|
|
"i" (sizeof(struct bug_entry))); \
|
|
} while (0)
|
|
|
|
#endif /* CONFIG_DEBUG_BUGVERBOSE */
|
|
|
|
#else
|
|
|
|
#define _BUG_FLAGS(ins, flags, extra) asm volatile(ins)
|
|
|
|
#endif /* CONFIG_GENERIC_BUG */
|
|
|
|
#define HAVE_ARCH_BUG
|
|
#define BUG() \
|
|
do { \
|
|
instrumentation_begin(); \
|
|
_BUG_FLAGS(ASM_UD2, 0, ""); \
|
|
__builtin_unreachable(); \
|
|
} while (0)
|
|
|
|
/*
|
|
* This instrumentation_begin() is strictly speaking incorrect; but it
|
|
* suppresses the complaints from WARN()s in noinstr code. If such a WARN()
|
|
* were to trigger, we'd rather wreck the machine in an attempt to get the
|
|
* message out than not know about it.
|
|
*/
|
|
#define __WARN_FLAGS(flags) \
|
|
do { \
|
|
__auto_type __flags = BUGFLAG_WARNING|(flags); \
|
|
instrumentation_begin(); \
|
|
_BUG_FLAGS(ASM_UD2, __flags, ASM_REACHABLE); \
|
|
instrumentation_end(); \
|
|
} while (0)
|
|
|
|
#include <asm-generic/bug.h>
|
|
|
|
#endif /* _ASM_X86_BUG_H */
|