Re: [PATCH v1 2/2] tests/device_reset: Add Xe healthcheck to post-reset validation

["Bernatowicz, Marcin" <marcin.bernatowicz@linux.intel.com>]

On 8/11/2026 4:14 PM, Lukasz Laguna wrote:
> Extend the healthcheck for Xe devices to validate basic command
> submission after reset.
>
> Signed-off-by: Lukasz Laguna <lukasz.laguna@intel.com>
> ---
>   tests/device_reset.c | 13 ++++++++++---
>   1 file changed, 10 insertions(+), 3 deletions(-)
>
> diff --git a/tests/device_reset.c b/tests/device_reset.c
> index b281f4ece..70ca511e0 100644
> --- a/tests/device_reset.c
> +++ b/tests/device_reset.c
> @@ -14,6 +14,8 @@
>   #include "igt_pci.h"
>   #include "igt_sysfs.h"
>   #include "igt_kmod.h"
> +#include "xe/xe_device.h"
> +
>   /**
>    * TEST: device reset
>    * Description: Examine behavior of a driver on device sysfs reset
> @@ -392,6 +394,8 @@ static bool is_i915_wedged(int i915)
>    */
>   static void healthcheck(struct device_fds *dev)
>   {
> +	int fd;
> +
>   	if (dev->fds.dev == -1) {
>   		/* give the kernel a breath for re-creating device nodes in devtmpfs */
>   		sleep(1);
> @@ -400,10 +404,13 @@ static void healthcheck(struct device_fds *dev)
>   		igt_debug("reopen the device\n");
>   		dev->fds.dev = __drm_open_driver(DRIVER_ANY);
>   	}
> -	igt_assert_fd(dev->fds.dev);
> +	fd = dev->fds.dev;
> +	igt_assert_fd(fd);
>   
> -	if (is_i915_device(dev->fds.dev))
> -		igt_assert(!is_i915_wedged(dev->fds.dev));
> +	if (is_i915_device(fd))
> +		igt_assert(!is_i915_wedged(fd));
> +	else if (is_xe_device(fd))
> +		xe_healthcheck(fd);
>   }
>   

LGTM,

Reviewed-by: Marcin Bernatowicz <marcin.bernatowicz@linux.intel.com>

>   /**