x86: Sanity-check static_cpu_has usage
authorBorislav Petkov <bp@suse.de>
Sun, 9 Jun 2013 10:07:32 +0000 (12:07 +0200)
committerH. Peter Anvin <hpa@linux.intel.com>
Fri, 21 Jun 2013 00:37:19 +0000 (17:37 -0700)
commit5700f743b597951743da9c7d891d3989aac0486e
treee0482b8bb6ec72bc1db19bcd853922efb97b36bc
parentc3b83598c1eeb1507603b461f5843ec2a49e3033
x86: Sanity-check static_cpu_has usage

static_cpu_has may be used only after alternatives have run. Before that
it always returns false if constant folding with __builtin_constant_p()
doesn't happen. And you don't want that.

This patch is the result of me debugging an issue where I overzealously
put static_cpu_has in code which executed before alternatives have run
and had to spend some time with scratching head and cursing at the
monitor.

So add a jump to a warning which screams loudly when we use this
function too early. The alternatives patch that check away in
conjunction with patching the rest of the kernel image.

[ hpa: factored this into its own configuration option.  If we want to
  have an overarching option, it should be an option which selects
  other options, not as a group option in the source code. ]

Signed-off-by: Borislav Petkov <bp@suse.de>
Link: http://lkml.kernel.org/r/1370772454-6106-4-git-send-email-bp@alien8.de
Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
arch/x86/Kconfig.debug
arch/x86/include/asm/cpufeature.h
arch/x86/kernel/cpu/common.c