Re: [PATCH] drm/amdgpu: Use drm_dbg_kms for reporting failure to get a GEM FB

[Christian König <ckoenig.leichtzumerken@xxxxxxxxx>]

Am 02.06.21 um 11:53 schrieb Michel Dänzer:
> From: Michel Dänzer <mdaenzer@xxxxxxxxxx>
>
> drm_err meant broken user space could spam dmesg.
>
> Fixes: f258907fdd835e "drm/amdgpu: Verify bo size can fit framebuffer
>                         size on init."
> Signed-off-by: Michel Dänzer <mdaenzer@xxxxxxxxxx>

Reviewed-by: Christian König <christian.koenig@xxxxxxx>

> ---
>   drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> index 8170e24ce37d..2a4cd7d377bf 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
> @@ -1058,7 +1058,7 @@ int amdgpu_display_gem_fb_init(struct drm_device *dev,
>   
>   	return 0;
>   err:
> -	drm_err(dev, "Failed to init gem fb: %d\n", ret);
> +	drm_dbg_kms(dev, "Failed to init gem fb: %d\n", ret);
>   	rfb->base.obj[0] = NULL;
>   	return ret;
>   }
> @@ -1096,7 +1096,7 @@ int amdgpu_display_gem_fb_verify_and_init(
>   
>   	return 0;
>   err:
> -	drm_err(dev, "Failed to verify and init gem fb: %d\n", ret);
> +	drm_dbg_kms(dev, "Failed to verify and init gem fb: %d\n", ret);
>   	rfb->base.obj[0] = NULL;
>   	return ret;
>   }

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx