| From 3ab6e1edc9fa00026424466e20e4e6be8f04e3c1 Mon Sep 17 00:00:00 2001 |
| From: Ben Hutchings <ben@decadent.org.uk> |
| Date: Tue, 11 Apr 2017 00:29:44 +0100 |
| Subject: [PATCH] cpupower: Fix turbo frequency reporting for pre-Sandy Bridge |
| cores |
| |
| commit 4cca0457686e4ee1677d69469e4ddfd94d389a80 upstream. |
| |
| The switch that conditionally sets CPUPOWER_CAP_HAS_TURBO_RATIO and |
| CPUPOWER_CAP_IS_SNB flags is missing a break, so all cores get both |
| flags set and an assumed base clock of 100 MHz for turbo values. |
| |
| Reported-by: GSR <gsr.bugs@infernal-iceberg.com> |
| Tested-by: GSR <gsr.bugs@infernal-iceberg.com> |
| References: https://bugs.debian.org/859978 |
| Fixes: 8fb2e440b223 (cpupower: Show Intel turbo ratio support via ...) |
| Signed-off-by: Ben Hutchings <ben@decadent.org.uk> |
| Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/tools/power/cpupower/utils/helpers/cpuid.c b/tools/power/cpupower/utils/helpers/cpuid.c |
| index 93b0aa74ca03..39c2c7d067bb 100644 |
| --- a/tools/power/cpupower/utils/helpers/cpuid.c |
| +++ b/tools/power/cpupower/utils/helpers/cpuid.c |
| @@ -156,6 +156,7 @@ out: |
| */ |
| case 0x2C: /* Westmere EP - Gulftown */ |
| cpu_info->caps |= CPUPOWER_CAP_HAS_TURBO_RATIO; |
| + break; |
| case 0x2A: /* SNB */ |
| case 0x2D: /* SNB Xeon */ |
| case 0x3A: /* IVB */ |
| -- |
| 2.12.0 |
| |