[igt-dev] [PATCH i-g-t] tests/core_hotunplug: Add XE driver test support

[Dandamudi, Priyanka]

> -----Original Message-----
> From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On Behalf Of
> janga.rahul.kumar at intel.com
> Sent: 29 May 2023 03:47 PM
> To: igt-dev at lists.freedesktop.org; Gandi, Ramadevi
> <ramadevi.gandi at intel.com>; Kumar, Janga Rahul
> <janga.rahul.kumar at intel.com>
> Subject: [igt-dev] [PATCH i-g-t] tests/core_hotunplug: Add XE driver test
> support
> 
> From: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> 
> Enable test for DRIVER_XE
> 
> Signed-off-by: Janga Rahul Kumar<janga.rahul.kumar at intel.com>
> ---
>  tests/core_hotunplug.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/core_hotunplug.c b/tests/core_hotunplug.c index
> 4d6a721f..2b97e86e 100644
> --- a/tests/core_hotunplug.c
> +++ b/tests/core_hotunplug.c
> @@ -445,7 +445,7 @@ static void node_healthcheck(struct hotunplug *priv,
> unsigned flags)
>  	if (closed)	/* store fd for cleanup if not dirty */
>  		priv->fd.drm_hc = fd_drm;
> 
> -	if (priv->chipset == DRIVER_INTEL) {
> +	if (priv->chipset == DRIVER_INTEL || priv->chipset == DRIVER_XE) {
You need to add more support to make it work for XE similar to local_i915_healthcheck as it is just assigning NULL value which is not expected.
Thanks,
Priyanka
>  		/* don't report library failed asserts as healthcheck failure */
>  		priv->failure = "Unrecoverable test failure";
>  		if (local_i915_healthcheck(fd_drm, "") && @@ -694,6
> +694,10 @@ igt_main
>  			igt_require_gem(fd_drm);
> 
>  			priv.has_intel_perf =
> local_i915_perf_healthcheck(fd_drm);
> +		} else if (is_xe_device(fd_drm)) {
> +			priv.chipset = DRIVER_XE;
> +
> +			gem_quiescent_gpu(fd_drm);
>  		}
> 
>  		/* Make sure subtests always reopen the same device */
> --
> 2.25.1