Skip to content

Commit 54cb728

Browse files
sanirban-999Andi Shyti
authored andcommitted
drm/i915/guc/slpc: Add helper function slpc_measure_power
Previously, the RPS function was being used, which utilizes raw frequency to calculate measured power. This commit introduces a dedicated function specifically for measuring power in SLPC, ensuring more accurate and reliable power measurements. Signed-off-by: Sk Anirban <sk.anirban@intel.com> Reviewed-by: Badal Nilawar <badal.nilawar@intel.com> Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com> Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20250113095912.356147-3-sk.anirban@intel.com
1 parent d7364b8 commit 54cb728

File tree

1 file changed

+16
-1
lines changed

1 file changed

+16
-1
lines changed

drivers/gpu/drm/i915/gt/selftest_slpc.c

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,21 @@ static int slpc_restore_freq(struct intel_guc_slpc *slpc, u32 min, u32 max)
9595
return 0;
9696
}
9797

98+
static u64 slpc_measure_power(struct intel_rps *rps, int *freq)
99+
{
100+
u64 x[5];
101+
int i;
102+
103+
for (i = 0; i < 5; i++)
104+
x[i] = __measure_power(5);
105+
106+
*freq = (*freq + intel_rps_read_actual_frequency(rps)) / 2;
107+
108+
/* A simple triangle filter for better result stability */
109+
sort(x, 5, sizeof(*x), cmp_u64, NULL);
110+
return div_u64(x[1] + 2 * x[2] + x[3], 4);
111+
}
112+
98113
static u64 measure_power_at_freq(struct intel_gt *gt, int *freq, u64 *power)
99114
{
100115
int err = 0;
@@ -103,7 +118,7 @@ static u64 measure_power_at_freq(struct intel_gt *gt, int *freq, u64 *power)
103118
if (err)
104119
return err;
105120
*freq = intel_rps_read_actual_frequency(&gt->rps);
106-
*power = measure_power(&gt->rps, freq);
121+
*power = slpc_measure_power(&gt->rps, freq);
107122

108123
return err;
109124
}

0 commit comments

Comments
 (0)