turbostat: Use GCC's CPUID functions to support PIC
authorJosh Triplett <josh@joshtriplett.org>
Wed, 21 Aug 2013 00:20:14 +0000 (17:20 -0700)
committerBen Hutchings <ben@decadent.org.uk>
Tue, 1 Apr 2014 23:58:44 +0000 (00:58 +0100)
commitc9ae63f0c5fe163f200e6517c0a3e0d8721145bf
tree633d8d981ae642a8a049a3991f96a34586d7a423
parent4fab53434e082b3835f5a89d5af15983c2be664d
turbostat: Use GCC's CPUID functions to support PIC

commit 2b92865e648ce04a39fda4f903784a5d01ecb0dc upstream.

turbostat uses inline assembly to call cpuid.  On 32-bit x86, on systems
that have certain security features enabled by default that make -fPIC
the default, this causes a build error:

turbostat.c: In function ‘check_cpuid’:
turbostat.c:1906:2: error: PIC register clobbered by ‘ebx’ in ‘asm’
  asm("cpuid" : "=a" (fms), "=c" (ecx), "=d" (edx) : "a" (1) : "ebx");
  ^

GCC provides a header cpuid.h, containing a __get_cpuid function that
works with both PIC and non-PIC.  (On PIC, it saves and restores ebx
around the cpuid instruction.)  Use that instead.

Signed-off-by: Josh Triplett <josh@joshtriplett.org>
Signed-off-by: Len Brown <len.brown@intel.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
tools/power/x86/turbostat/turbostat.c