On 10/21/26 04:47, Raag Jadav wrote:
> Currently, drm_dev_wedged_event() logs a basic message of device being
> wedged, but doesn't print the recovery method used by the caller. Log it
> for easier bug report analysis.
> 
> Signed-off-by: Raag Jadav <[email protected]>

Reviewed-by: Christian König <[email protected]>

> ---
> v2: Distinguish between recovery and no recovery cases (Christian, Rodrigo)
> v3: Retain drm_info() (Christian)
> ---
>  drivers/gpu/drm/drm_drv.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
> index cb53baa70995..43125d655f64 100644
> --- a/drivers/gpu/drm/drm_drv.c
> +++ b/drivers/gpu/drm/drm_drv.c
> @@ -598,8 +598,8 @@ int drm_dev_wedged_event(struct drm_device *dev, unsigned 
> long method,
>               /* Caller is unsure about recovery, do the best we can at this 
> point. */
>               snprintf(event_string, sizeof(event_string), "%s", 
> "WEDGED=unknown");
>  
> -     drm_info(dev, "device wedged, %s\n", method == DRM_WEDGE_RECOVERY_NONE ?
> -              "but no recovery needed" : "needs recovery");
> +     drm_info(dev, "device wedged with method %#lx, %s\n", method,
> +              method == DRM_WEDGE_RECOVERY_NONE ? "no recovery needed" : 
> "needs recovery");
>  
>       if (info && (info->comm[0] != '\0') && (info->pid >= 0)) {
>               snprintf(pid_string, sizeof(pid_string), "PID=%u", info->pid);

Reply via email to