luck/tmp_suning_uos_patched
1
0
Fork 1
Code Issues Pull Requests Packages Projects Releases Wiki Activity

WARN_ONCE(): use bool for boolean flag

Browse Source 
Commit 7086745309 ("printk_once(): use bool
for boolean flag") changed printk_once() to use bool instead of int for
its guard variable.  Do the same change to WARN_ONCE() and WARN_ON_ONCE(),
for the same reasons.

This resulted in a reduction of 1462 bytes on a x86-64 defconfig:

   text    data     bss     dec     hex filename
8101271 1207116  992764 10301151         9d2edf vmlinux.before
8100553 1207148  991988 10299689         9d2929 vmlinux.after

Signed-off-by: Cesar Eduardo Barros <cesarb@cesarb.net>
Cc: Roland Dreier <rolandd@cisco.com>
Cc: Daniel Walker <dwalker@fifo99.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
Cesar Eduardo Barros 2009-12-14 18:00:13 -08:00 committed by Linus Torvalds
parent 6613c5e860
commit 42f247c83a
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
include/asm-generic/bug.h
View File

@ -113,22 +113,22 @@ extern void warn_slowpath_null(const char *file, const int line);
#endif #endif
#define WARN_ON_ONCE(condition) ({ \ #define WARN_ON_ONCE(condition) ({ \
static int __warned; \ static bool __warned; \
int __ret_warn_once = !!(condition); \ int __ret_warn_once = !!(condition); \
\ \
if (unlikely(__ret_warn_once)) \ if (unlikely(__ret_warn_once)) \
if (WARN_ON(!__warned)) \ if (WARN_ON(!__warned)) \
__warned = 1; \ __warned = true; \
unlikely(__ret_warn_once); \ unlikely(__ret_warn_once); \
}) })
#define WARN_ONCE(condition, format...) ({ \ #define WARN_ONCE(condition, format...) ({ \
static int __warned; \ static bool __warned; \
int __ret_warn_once = !!(condition); \ int __ret_warn_once = !!(condition); \
\ \
if (unlikely(__ret_warn_once)) \ if (unlikely(__ret_warn_once)) \
if (WARN(!__warned, format)) \ if (WARN(!__warned, format)) \
__warned = 1; \ __warned = true; \
unlikely(__ret_warn_once); \ unlikely(__ret_warn_once); \
}) })