Re: [PATCH i-g-t 1/2] tests/intel/xe_gt_freq: Fix freq_low_max test

["Belgaumkar, Vinay" <vinay.belgaumkar@intel.com>]

On 1/19/2024 9:04 AM, Badal Nilawar wrote:
> freq_low_max test is intended to validate GuC PC (SLPC) behaviour
> when max freq set to less than min freq. As workloads are not being
> run validating cur freq is enough.
>
> v2: Validate act freq as well when gt is not in C6 (Vinay)
>
> Fixes: acaaca0bf317 ("tests/xe: Add Xe IGT tests")
> Signed-off-by: Badal Nilawar <badal.nilawar@intel.com>
> ---
>   tests/intel/xe_gt_freq.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/tests/intel/xe_gt_freq.c b/tests/intel/xe_gt_freq.c
> index 4334a8a41..bf63e4298 100644
> --- a/tests/intel/xe_gt_freq.c
> +++ b/tests/intel/xe_gt_freq.c
> @@ -263,7 +263,9 @@ static void test_freq_low_max(int fd, int gt_id)
>   	igt_assert(set_freq(fd, gt_id, "max", rpn) > 0);
>   	usleep(ACT_FREQ_LATENCY_US);
>   	igt_assert(get_freq(fd, gt_id, "cur") == rpe);
> -	igt_assert(get_freq(fd, gt_id, "act") == rpe);
> +
> +	if (!xe_is_gt_in_c6(fd, gt_id))
> +		igt_assert(get_freq(fd, gt_id, "act") == rpe);
Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
>   }
>   
>   /**