summaryrefslogtreecommitdiffstats
path: root/mm
diff options
context:
space:
mode:
authorMichal Hocko <mhocko@kernel.org>2018-04-05 16:25:30 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2018-04-05 21:36:27 -0700
commit91241681c62a5a690c88eb2aca027f094125eaac (patch)
treecad7912bd078096ca33893be54443fb9b97676f5 /mm
parent2c7452a075d4db2dc8e7a61369ede1afc05cd63e (diff)
downloadlinux-91241681c62a5a690c88eb2aca027f094125eaac.tar.gz
linux-91241681c62a5a690c88eb2aca027f094125eaac.tar.bz2
linux-91241681c62a5a690c88eb2aca027f094125eaac.zip
include/linux/mmdebug.h: make VM_WARN* non-rvals
At present the construct if (VM_WARN(...)) will compile OK with CONFIG_DEBUG_VM=y and will fail with CONFIG_DEBUG_VM=n. The reason is that VM_{WARN,BUG}* have always been special wrt. {WARN/BUG}* and never generate any code when DEBUG_VM is disabled. So we cannot really use it in conditionals. We considered changing things so that this construct works in both cases but that might cause unwanted code generation with CONFIG_DEBUG_VM=n. It is safer and simpler to make the build fail in both cases. [akpm@linux-foundation.org: changelog] Signed-off-by: Michal Hocko <mhocko@suse.com> Reviewed-by: Andrew Morton <akpm@linux-foundation.org> Cc: Stephen Rothwell <sfr@canb.auug.org.au> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
0 files changed, 0 insertions, 0 deletions