Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a19718bd authored by Ben Hutchings's avatar Ben Hutchings Committed by Greg Kroah-Hartman
Browse files

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: default avatarGSR <gsr.bugs@infernal-iceberg.com>
Tested-by: default avatarGSR <gsr.bugs@infernal-iceberg.com>
References: https://bugs.debian.org/859978


Fixes: 8fb2e440 (cpupower: Show Intel turbo ratio support via ...)
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 4f39fcce

tools/power/cpupower/utils/helpers/cpuid.c

+1 −0
Original line number Diff line number Diff line
@@ -156,6 +156,7 @@ int get_cpu_info(unsigned int cpu, struct cpupower_cpu_info *cpu_info) 
					 */

			case 0x2C:	/* Westmere EP - Gulftown */

				cpu_info->caps |= CPUPOWER_CAP_HAS_TURBO_RATIO;

				break;

			case 0x2A:	/* SNB */

			case 0x2D:	/* SNB Xeon */

			case 0x3A:	/* IVB */