Re: [PATCH v1] drm/xe/guc_submit: Log wedged mode correctly

[Matthew Brost <matthew.brost@intel.com>]

On Sun, Mar 01, 2026 at 12:35:02AM +0530, Raag Jadav wrote:
> The device can be declared wedged in basically any wedged mode. Log the
> correct one that is set in device struct instead of hardcoding it.
> 
> Fixes: 17d3c3365ba9 ("drm/xe: Validate wedged_mode parameter and define enum for modes")
> Signed-off-by: Raag Jadav <raag.jadav@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_guc_submit.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_guc_submit.c b/drivers/gpu/drm/xe/xe_guc_submit.c
> index de716c1fb18e..4c761087c830 100644
> --- a/drivers/gpu/drm/xe/xe_guc_submit.c
> +++ b/drivers/gpu/drm/xe/xe_guc_submit.c
> @@ -1319,7 +1319,7 @@ void xe_guc_submit_wedge(struct xe_guc *guc)
>  	if (err) {
>  		xe_gt_err(gt, "Failed to register clean-up in wedged.mode=%s; "
>  			  "Although device is wedged.\n",
> -			  xe_wedged_mode_to_string(XE_WEDGED_MODE_UPON_ANY_HANG_NO_RESET));
> +			  xe_wedged_mode_to_string(guc_to_xe(guc)->wedged.mode));

It is actually a bug to register this in any mode aside from
XE_WEDGED_MODE_UPON_ANY_HANG_NO_RESET. We have a patch / series floating
around to fix this [1].

Matt

[1] https://patchwork.freedesktop.org/patch/706751/?series=162074&rev=2

>  		return;
>  	}
>  
> -- 
> 2.43.0
>