[PATCH i-g-t] tests/intel/xe_pm_residency: Add an assertion on MI_STORE execution time

[PATCH i-g-t] tests/intel/xe_pm_residency: Add an assertion on MI_STORE execution time

[Karthik Poosa]

MI_STORE_DWORD generally completes within couple of ms.
Assert if it takes more than 1.2 seconds, as it will cause
IGT test to timeout due to sleep of 120 seconds which is
the current per task timeout.

Signed-off-by: Karthik Poosa <karthik.poosa@intel.com>
---
 tests/intel/xe_pm_residency.c | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/tests/intel/xe_pm_residency.c b/tests/intel/xe_pm_residency.c
index 0e687558b..29c35f617 100644
--- a/tests/intel/xe_pm_residency.c
+++ b/tests/intel/xe_pm_residency.c
@@ -144,6 +144,15 @@ static void exec_load(int fd, struct drm_xe_engine_class_instance *hwe, unsigned
 			  1e-3 * submit,
 			  1e-3 * (elapsed - submit));
 
+		/*
+		 * MI_STORE_DWORD generally completes within couple of ms.
+		 * Assert if it takes more than 1.2 seconds, as it will cause
+		 * IGT test to timeout due to sleep of 120 seconds which is
+		 * the current per task timeout. Current there is no way to
+		 * read this timeout from IGT test.
+		 */
+		igt_assert((uint64_t)elapsed < (uint64_t)(1.2 * NSEC_PER_SEC));
+
 		syncobj_reset(fd, &syncobj, 1);
 
 		/*
-- 
2.25.1

Re: [PATCH i-g-t] tests/intel/xe_pm_residency: Add an assertion on MI_STORE execution time

[Riana Tauro]

Hi Karthik

On 9/23/2024 10:17 PM, Karthik Poosa wrote:
> MI_STORE_DWORD generally completes within couple of ms.
> Assert if it takes more than 1.2 seconds, as it will cause
> IGT test to timeout due to sleep of 120 seconds which is
> the current per task timeout.
s/task/test
> 
> Signed-off-by: Karthik Poosa <karthik.poosa@intel.com>
> ---
>   tests/intel/xe_pm_residency.c | 9 +++++++++
>   1 file changed, 9 insertions(+)
> 
> diff --git a/tests/intel/xe_pm_residency.c b/tests/intel/xe_pm_residency.c
> index 0e687558b..29c35f617 100644
> --- a/tests/intel/xe_pm_residency.c
> +++ b/tests/intel/xe_pm_residency.c
> @@ -144,6 +144,15 @@ static void exec_load(int fd, struct drm_xe_engine_class_instance *hwe, unsigned
>   			  1e-3 * submit,
>   			  1e-3 * (elapsed - submit));
>   
> +		/*
> +		 * MI_STORE_DWORD generally completes within couple of ms.
> +		 * Assert if it takes more than 1.2 seconds, as it will cause
> +		 * IGT test to timeout due to sleep of 120 seconds which is
> +		 * the current per task timeout. Current there is no way to
s/task/test
s/Current/Currently

With that
Reviewed-by: Riana Tauro <riana.tauro@intel.com>
> +		 * read this timeout from IGT test.
> +		 */
> +		igt_assert((uint64_t)elapsed < (uint64_t)(1.2 * NSEC_PER_SEC));
> +
>   		syncobj_reset(fd, &syncobj, 1);
>   
>   		/*