Messages in this thread Patch in this message | | | Date | Mon, 01 Sep 2008 16:07:01 +0300 | From | Boaz Harrosh <> | Subject | [PATCH] debug, x86: move BUILD_BUG_ON() ARRAY_SIZE and __FUNCTION__ |
| |
From: Ingo Molnar <mingo@elte.hu>
move BUILD_BUG_ON variants, ARRAY_SIZE and __FUNCTION__ definitions from kernel.h to compiler.h.
Besides being the correct location for such trivial wrappers around compiler functionality, this also allows the removal of duplicate definitions from arch/x86/boot/boot.h.
[ boot.h cannot just include kernel.h to pick up the new definitions, as it is also built into user-space utilities on the host system. ]
Signed-off-by: Ingo Molnar <mingo@elte.hu> Signed-off-by: Boaz Harrosh <bharrosh@panasas.com> --- arch/x86/boot/boot.h | 5 ----- include/linux/compiler.h | 14 ++++++++++++++ include/linux/kernel.h | 14 -------------- 3 files changed, 14 insertions(+), 19 deletions(-)
diff --git a/arch/x86/boot/boot.h b/arch/x86/boot/boot.h index cc0ef13..f09b79a 100644 --- a/arch/x86/boot/boot.h +++ b/arch/x86/boot/boot.h @@ -27,11 +27,6 @@ #include "bitops.h" #include <asm/cpufeature.h> -/* Useful macros */ -#define BUILD_BUG_ON(condition) ((void)sizeof(char[1 - 2*!!(condition)])) - -#define ARRAY_SIZE(x) (sizeof(x) / sizeof(*(x))) - extern struct setup_header hdr; extern struct boot_params boot_params; diff --git a/include/linux/compiler.h b/include/linux/compiler.h index c8bd2da..90fa975 100644 --- a/include/linux/compiler.h +++ b/include/linux/compiler.h @@ -194,4 +194,18 @@ extern void __chk_io_ptr(const volatile void __iomem *); */ #define ACCESS_ONCE(x) (*(volatile typeof(x) *)&(x)) +/* Force a compilation error if condition is true */ +#define BUILD_BUG_ON(condition) ((void)sizeof(char[1 - 2*!!(condition)])) + +/* Force a compilation error if condition is true, but also produce a + result (of value 0 and type size_t), so the expression can be used + e.g. in a structure initializer (or where-ever else comma expressions + aren't permitted). */ +#define BUILD_BUG_ON_ZERO(e) (sizeof(char[1 - 2 * !!(e)]) - 1) + +/* Trap pasters of __FUNCTION__ at compile-time */ +#define __FUNCTION__ (__func__) + +#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr)) + #endif /* __LINUX_COMPILER_H */ diff --git a/include/linux/kernel.h b/include/linux/kernel.h index 2651f80..29ca10d 100644 --- a/include/linux/kernel.h +++ b/include/linux/kernel.h @@ -42,8 +42,6 @@ extern const char linux_proc_banner[]; #define PTR_ALIGN(p, a) ((typeof(p))ALIGN((unsigned long)(p), (a))) #define IS_ALIGNED(x, a) (((x) & ((typeof(x))(a) - 1)) == 0) -#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + __must_be_array(arr)) - #define FIELD_SIZEOF(t, f) (sizeof(((t*)0)->f)) #define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d)) #define roundup(x, y) ((((x) + ((y) - 1)) / (y)) * (y)) @@ -467,18 +465,6 @@ struct sysinfo { char _f[20-2*sizeof(long)-sizeof(int)]; /* Padding: libc5 uses this.. */ }; -/* Force a compilation error if condition is true */ -#define BUILD_BUG_ON(condition) ((void)sizeof(char[1 - 2*!!(condition)])) - -/* Force a compilation error if condition is true, but also produce a - result (of value 0 and type size_t), so the expression can be used - e.g. in a structure initializer (or where-ever else comma expressions - aren't permitted). */ -#define BUILD_BUG_ON_ZERO(e) (sizeof(char[1 - 2 * !!(e)]) - 1) - -/* Trap pasters of __FUNCTION__ at compile-time */ -#define __FUNCTION__ (__func__) - /* This helps us to avoid #ifdef CONFIG_NUMA */ #ifdef CONFIG_NUMA #define NUMA_BUILD 1
| |