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

[Riana Tauro <riana.tauro@intel.com>]

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);
>   
>   		/*