36 lines
1.2 KiB
Diff
36 lines
1.2 KiB
Diff
|
From 6f10d066dce0f1781b514a0352f0b427a32b1bb2 Mon Sep 17 00:00:00 2001
|
||
|
From: Mario Limonciello <mario.limonciello@amd.com>
|
||
|
Date: Thu, 5 Sep 2024 11:30:00 -0500
|
||
|
Subject: ACPI: CPPC: Drop check for non zero perf ratio
|
||
|
|
||
|
perf_ratio is a u64 and SCHED_CAPACITY_SCALE is a large number.
|
||
|
Shifting by one will never have a zero value.
|
||
|
|
||
|
Drop the check.
|
||
|
|
||
|
Suggested-by: Gautham R. Shenoy <gautham.shenoy@amd.com>
|
||
|
Reviewed-by: Gautham R. Shenoy <gautham.sheoy@amd.com>
|
||
|
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
|
||
|
Reviewed-by: Gautham R. Shenoy <gautham.shenoy@amd.com>
|
||
|
---
|
||
|
arch/x86/kernel/acpi/cppc.c | 7 +------
|
||
|
1 file changed, 1 insertion(+), 6 deletions(-)
|
||
|
|
||
|
--- a/arch/x86/kernel/acpi/cppc.c
|
||
|
+++ b/arch/x86/kernel/acpi/cppc.c
|
||
|
@@ -91,13 +91,8 @@ static void amd_set_max_freq_ratio(void)
|
||
|
return;
|
||
|
}
|
||
|
|
||
|
- perf_ratio = div_u64(numerator * SCHED_CAPACITY_SCALE, nominal_perf);
|
||
|
/* midpoint between max_boost and max_P */
|
||
|
- perf_ratio = (perf_ratio + SCHED_CAPACITY_SCALE) >> 1;
|
||
|
- if (!perf_ratio) {
|
||
|
- pr_debug("Non-zero highest/nominal perf values led to a 0 ratio\n");
|
||
|
- return;
|
||
|
- }
|
||
|
+ perf_ratio = (div_u64(numerator * SCHED_CAPACITY_SCALE, nominal_perf) + SCHED_CAPACITY_SCALE) >> 1;
|
||
|
|
||
|
freq_invariance_set_perf_ratio(perf_ratio, false);
|
||
|
}
|