diff mbox series

[i-g-t] tests/intel/xe_gt_freq: Check for RPe freq updates

Message ID 20240425195657.3638599-1-vinay.belgaumkar@intel.com (mailing list archive)
State New, archived
Headers show
Series [i-g-t] tests/intel/xe_gt_freq: Check for RPe freq updates | expand

Commit Message

Vinay Belgaumkar April 25, 2024, 7:56 p.m. UTC
Pcode can dynamically update RPe frequency. Use the latest value
in tests that check it.

Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
---
 tests/intel/xe_gt_freq.c | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Rodrigo Vivi April 29, 2024, 12:44 p.m. UTC | #1
On Thu, Apr 25, 2024 at 12:56:57PM -0700, Vinay Belgaumkar wrote:
> Pcode can dynamically update RPe frequency. Use the latest value
> in tests that check it.
> 
> Signed-off-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  tests/intel/xe_gt_freq.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/tests/intel/xe_gt_freq.c b/tests/intel/xe_gt_freq.c
> index c12f6c52c..5c6373016 100644
> --- a/tests/intel/xe_gt_freq.c
> +++ b/tests/intel/xe_gt_freq.c
> @@ -186,6 +186,9 @@ static void test_freq_fixed(int fd, int gt_id, bool gt_idle)
>  		igt_assert(get_freq(fd, gt_id, "act") == rpn);
>  	}
>  
> +	/* Refresh value of rpe, pcode could have adjusted it */
> +	rpe = get_freq(fd, gt_id, "rpe");
> +
>  	igt_assert(set_freq(fd, gt_id, "min", rpe) > 0);
>  	igt_assert(set_freq(fd, gt_id, "max", rpe) > 0);
>  	usleep(ACT_FREQ_LATENCY_US);
> @@ -268,6 +271,10 @@ static void test_freq_low_max(int fd, int gt_id)
>  	igt_assert(set_freq(fd, gt_id, "min", rpe) > 0);
>  	igt_assert(set_freq(fd, gt_id, "max", rpn) > 0);
>  	usleep(ACT_FREQ_LATENCY_US);
> +
> +	/* Refresh value of rpe, pcode could have adjusted it */
> +	rpe = get_freq(fd, gt_id, "rpe");
> +
>  	igt_assert(get_freq(fd, gt_id, "cur") == rpe);
>  
>  	if (!xe_is_gt_in_c6(fd, gt_id))
> -- 
> 2.38.1
>
diff mbox series

Patch

diff --git a/tests/intel/xe_gt_freq.c b/tests/intel/xe_gt_freq.c
index c12f6c52c..5c6373016 100644
--- a/tests/intel/xe_gt_freq.c
+++ b/tests/intel/xe_gt_freq.c
@@ -186,6 +186,9 @@  static void test_freq_fixed(int fd, int gt_id, bool gt_idle)
 		igt_assert(get_freq(fd, gt_id, "act") == rpn);
 	}
 
+	/* Refresh value of rpe, pcode could have adjusted it */
+	rpe = get_freq(fd, gt_id, "rpe");
+
 	igt_assert(set_freq(fd, gt_id, "min", rpe) > 0);
 	igt_assert(set_freq(fd, gt_id, "max", rpe) > 0);
 	usleep(ACT_FREQ_LATENCY_US);
@@ -268,6 +271,10 @@  static void test_freq_low_max(int fd, int gt_id)
 	igt_assert(set_freq(fd, gt_id, "min", rpe) > 0);
 	igt_assert(set_freq(fd, gt_id, "max", rpn) > 0);
 	usleep(ACT_FREQ_LATENCY_US);
+
+	/* Refresh value of rpe, pcode could have adjusted it */
+	rpe = get_freq(fd, gt_id, "rpe");
+
 	igt_assert(get_freq(fd, gt_id, "cur") == rpe);
 
 	if (!xe_is_gt_in_c6(fd, gt_id))