summaryrefslogtreecommitdiffstats
path: root/fs
diff options
context:
space:
mode:
authorPatrick Palka <patrick@parcs.ath.cx>2014-02-17 20:50:59 -0500
committerTheodore Ts'o <tytso@mit.edu>2014-02-17 20:50:59 -0500
commit024949ec8fc165bfac8eb051e537bc303adb365f (patch)
treeecea44873e977784d5cbd3003299b3081f4dc5ad /fs
parent7747e6d028b891f3bd02d93295d80f230ba43f6a (diff)
downloadop-kernel-dev-024949ec8fc165bfac8eb051e537bc303adb365f.zip
op-kernel-dev-024949ec8fc165bfac8eb051e537bc303adb365f.tar.gz
ext4: address a benign compiler warning
When !defined(CONFIG_EXT4_DEBUG), mb_debug() should be defined as a no_printk() statement instead of an empty statement in order to suppress the following compiler warning: fs/ext4/mballoc.c: In function ‘ext4_mb_cleanup_pa’: fs/ext4/mballoc.c:2659:47: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body] mb_debug(1, "mballoc: %u PAs left\n", count); Signed-off-by: Patrick Palka <patrick@parcs.ath.cx> Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs')
-rw-r--r--fs/ext4/mballoc.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/ext4/mballoc.h b/fs/ext4/mballoc.h
index 08481ee..9347328 100644
--- a/fs/ext4/mballoc.h
+++ b/fs/ext4/mballoc.h
@@ -48,7 +48,7 @@ extern ushort ext4_mballoc_debug;
} \
} while (0)
#else
-#define mb_debug(n, fmt, a...)
+#define mb_debug(n, fmt, a...) no_printk(fmt, ## a)
#endif
#define EXT4_MB_HISTORY_ALLOC 1 /* allocation */
OpenPOWER on IntegriCloud