[PATCH i-g-t 1/3] tests/intel/xe_oa: Move Xe2+ check before init_sys_info

Kamil Konieczny kamil.konieczny at linux.intel.com
Tue Dec 10 11:15:02 UTC 2024


Hi Ashutosh,
On 2024-12-09 at 18:08:10 -0800, Ashutosh Dixit wrote:
> In order to avoid failures on unsupported Xe1 platforms, move Xe2+ check
> before init_sys_info.
> 
> Closes: https://gitlab.freedesktop.org/drm/xe/kernel/-/issues/3645
> Signed-off-by: Ashutosh Dixit <ashutosh.dixit at intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/intel/xe_oa.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/intel/xe_oa.c b/tests/intel/xe_oa.c
> index 8a88945a2b..9e7c2ae1e6 100644
> --- a/tests/intel/xe_oa.c
> +++ b/tests/intel/xe_oa.c
> @@ -4855,14 +4855,14 @@ igt_main
>  		devid = intel_get_drm_devid(drm_fd);
>  		sysfs = igt_sysfs_open(drm_fd);
>  
> +		/* Currently only run on Xe2+ */
> +		igt_require(intel_graphics_ver(devid) >= IP_VER(20, 0));
> +
>  		igt_require(init_sys_info());
>  
>  		write_u64_file("/proc/sys/dev/xe/observation_paranoid", 1);
>  
>  		render_copy = igt_get_render_copyfunc(devid);
> -
> -		/* Currently only run on Xe2+ */
> -		igt_require(intel_graphics_ver(devid) >= IP_VER(20, 0));
>  	}
>  
>  	igt_subtest("non-system-wide-paranoid")
> -- 
> 2.47.1
> 


More information about the igt-dev mailing list