Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

bug.h: Fix up CONFIG_BUG=n implicit function declarations.

Commit 2603efa31a03 ("bug.h: Fix up powerpc build regression") corrected
the powerpc build case and extended the __ASSEMBLY__ guards, but it also
got caught in pre-processor hell accidentally matching the else case of
CONFIG_BUG resulting in the BUG disabled case tripping up on
-Werror=implicit-function-declaration.

It's not possible to __ASSEMBLY__ guard the entire file as architecture
code needs to get at the BUGFLAG_WARNING definition in the GENERIC_BUG
case, but the rest of the CONFIG_BUG=y/n case needs to be guarded.

Rather than littering endless __ASSEMBLY__ checks in each of the if/else
cases we just move the BUGFLAG definitions up under their own
GENERIC_BUG test and then shove everything else under one big
__ASSEMBLY__ guard.

Build tested on all of x86 CONFIG_BUG=y, CONFIG_BUG=n, powerpc (due to
it's dependence on BUGFLAG definitions in assembly code), and sh (due to
not bringing in linux/kernel.h to satisfy the taint flag definitions used
by the generic bug code).

Hopefully that's the end of the corner cases and I can abstain from ever
having to touch this infernal header ever again.

Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Tested-by: Fengguang Wu <wfg@linux.intel.com>
Acked-by: Randy Dunlap <rdunlap@xenotime.net>
Cc: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

authored by

Paul Mundt and committed by
Linus Torvalds
09682c1d 6b16351a

+9 -10
+9 -10
include/asm-generic/bug.h
··· 3 3 4 4 #include <linux/compiler.h> 5 5 6 + #ifdef CONFIG_GENERIC_BUG 7 + #define BUGFLAG_WARNING (1 << 0) 8 + #define BUGFLAG_TAINT(taint) (BUGFLAG_WARNING | ((taint) << 8)) 9 + #define BUG_GET_TAINT(bug) ((bug)->flags >> 8) 10 + #endif 11 + 12 + #ifndef __ASSEMBLY__ 13 + #include <linux/kernel.h> 14 + 6 15 #ifdef CONFIG_BUG 7 16 8 17 #ifdef CONFIG_GENERIC_BUG 9 - #ifndef __ASSEMBLY__ 10 18 struct bug_entry { 11 19 #ifndef CONFIG_GENERIC_BUG_RELATIVE_POINTERS 12 20 unsigned long bug_addr; ··· 31 23 #endif 32 24 unsigned short flags; 33 25 }; 34 - #endif /* __ASSEMBLY__ */ 35 - 36 - #define BUGFLAG_WARNING (1 << 0) 37 - #define BUGFLAG_TAINT(taint) (BUGFLAG_WARNING | ((taint) << 8)) 38 - #define BUG_GET_TAINT(bug) ((bug)->flags >> 8) 39 - 40 26 #endif /* CONFIG_GENERIC_BUG */ 41 - 42 - #ifndef __ASSEMBLY__ 43 - #include <linux/kernel.h> 44 27 45 28 /* 46 29 * Don't use BUG() or BUG_ON() unless there's really no way out; one