[PATCH] tests/xe_pmu: Fix frequency stashing issue

[Koppuravuri, Ravi Kishore]

Hi,

On 28-04-2025 11:06, Anirban, Sk wrote:
>
>
> On 25-04-2025 23:36, sk.anirban at intel.com wrote:
>> From: Sk Anirban <sk.anirban at intel.com>
>>
>> Resolved the frequency stashing problem by accurately
>> dereferencing the pointers, ensuring the minimum and
>> maximum frequencies are correctly restored.
>>
>> Signed-off-by: Sk Anirban <sk.anirban at intel.com>
>> ---
>>   tests/intel/xe_pmu.c | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/tests/intel/xe_pmu.c b/tests/intel/xe_pmu.c
>> index 589c7cbde..094dc0c6c 100644
>> --- a/tests/intel/xe_pmu.c
>> +++ b/tests/intel/xe_pmu.c
>> @@ -527,8 +527,8 @@ static void stash_gt_freq(int fd, uint32_t 
>> **stash_min, uint32_t **stash_max)
>>       igt_skip_on(*stash_min == NULL || *stash_max == NULL);
>>         xe_for_each_gt(fd, gt) {
>> -        *stash_min[gt] = xe_gt_get_freq(fd, gt, "min");
>> -        *stash_max[gt] = xe_gt_get_freq(fd, gt, "max");
>> +        (*stash_min)[gt] = xe_gt_get_freq(fd, gt, "min");
>> +        (*stash_max)[gt] = xe_gt_get_freq(fd, gt, "max");
>>       }
>>   }
>
It looks good to me.

Reviewed-by: Ravi Kishore Koppuravuri <ravi.kishore.koppuravuri at intel.com>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20250428/869e1602/attachment.htm>