diff options
| author | Thomas Renninger <trenn@suse.de> | 2011-01-21 15:11:19 +0100 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2011-02-10 23:58:11 -0500 |
| commit | 8209e054b6bd0805a25b8c4af971160fa146bf22 (patch) | |
| tree | 29706b992f75b98266a1f12ffcc760ec7b00ec06 /tools/power/x86/turbostat/turbostat.c | |
| parent | 6148a47ac3872092d4bc4888838bec6dff16654d (diff) | |
| download | talos-op-linux-8209e054b6bd0805a25b8c4af971160fa146bf22.tar.gz talos-op-linux-8209e054b6bd0805a25b8c4af971160fa146bf22.zip | |
tools: turbostat: fix bitwise and operand
bug could cause false positive on indicating
presence of invarient TSC or APERF support.
Signed-off-by: Thomas Renninger <trenn@suse.de>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'tools/power/x86/turbostat/turbostat.c')
| -rw-r--r-- | tools/power/x86/turbostat/turbostat.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c index 4c6983de6fd9..a96248f19264 100644 --- a/tools/power/x86/turbostat/turbostat.c +++ b/tools/power/x86/turbostat/turbostat.c @@ -892,7 +892,7 @@ void check_cpuid() * this check is valid for both Intel and AMD */ asm("cpuid" : "=a" (eax), "=b" (ebx), "=c" (ecx), "=d" (edx) : "a" (0x80000007)); - has_invariant_tsc = edx && (1 << 8); + has_invariant_tsc = edx & (1 << 8); if (!has_invariant_tsc) { fprintf(stderr, "No invariant TSC\n"); @@ -905,7 +905,7 @@ void check_cpuid() */ asm("cpuid" : "=a" (eax), "=b" (ebx), "=c" (ecx), "=d" (edx) : "a" (0x6)); - has_aperf = ecx && (1 << 0); + has_aperf = ecx & (1 << 0); if (!has_aperf) { fprintf(stderr, "No APERF MSR\n"); exit(1); |
