Message ID | 1352844821-18952-5-git-send-email-daniel.santos@pobox.com (mailing list archive) |
---|---|
State | Not Applicable, archived |
Headers | show |
On Tue, Nov 13, 2012 at 04:13:37PM -0600, danielfsantos@att.net wrote: > When __CHECKER__ is defined, we disable all of the BUILD_BUG.* macros. > However, both BUILD_BUG_ON_NOT_POWER_OF_2 and BUILD_BUG_ON was > evaluating to nothing in this case, and we want (0) since this is a > function-like macro that will be followed by a semicolon. > > Signed-off-by: Daniel Santos <daniel.santos@pobox.com> Acked-by: Borislav Petkov <bp@alien8.de>
diff --git a/include/linux/bug.h b/include/linux/bug.h index 298a916..1b2465d 100644 --- a/include/linux/bug.h +++ b/include/linux/bug.h @@ -12,10 +12,10 @@ enum bug_trap_type { struct pt_regs; #ifdef __CHECKER__ -#define BUILD_BUG_ON_NOT_POWER_OF_2(n) +#define BUILD_BUG_ON_NOT_POWER_OF_2(n) (0) #define BUILD_BUG_ON_ZERO(e) (0) #define BUILD_BUG_ON_NULL(e) ((void*)0) -#define BUILD_BUG_ON(condition) +#define BUILD_BUG_ON(condition) (0) #define BUILD_BUG() (0) #else /* __CHECKER__ */
When __CHECKER__ is defined, we disable all of the BUILD_BUG.* macros. However, both BUILD_BUG_ON_NOT_POWER_OF_2 and BUILD_BUG_ON was evaluating to nothing in this case, and we want (0) since this is a function-like macro that will be followed by a semicolon. Signed-off-by: Daniel Santos <daniel.santos@pobox.com> --- include/linux/bug.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)