[PATCH] drm/amd/amdgpu: Print out ring name in dev_info

Christian König deathsimple at vodafone.de
Wed Apr 19 15:07:13 UTC 2017


Am 19.04.2017 um 17:03 schrieb Tom St Denis:
> So it's more obvious which rings are using which INV engines.
>
> Signed-off-by: Tom St Denis <tom.stdenis at amd.com>

I wonder if we shouldn't stop printing the ring numbers completely in 
the ring and IB tests as well and always use the ring name.

But anyway for now the patch is Reviewed-by: Christian König 
<christian.koenig at amd.com>.

Regards,
Christian.

> ---
>   drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> index a71521e10763..edf43769ae70 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v9_0.c
> @@ -394,8 +394,9 @@ static int gmc_v9_0_late_init(void *handle)
>   		unsigned vmhub = ring->funcs->vmhub;
>   
>   		ring->vm_inv_eng = vm_inv_eng[vmhub]++;
> -		dev_info(adev->dev, "ring %u uses VM inv eng %u on hub %u\n",
> -			 ring->idx, ring->vm_inv_eng, ring->funcs->vmhub);
> +		dev_info(adev->dev, "ring %u(%s) uses VM inv eng %u on hub %u\n",
> +			 ring->idx, ring->name, ring->vm_inv_eng,
> +			 ring->funcs->vmhub);
>   	}
>   
>   	/* Engine 17 is used for GART flushes */




More information about the amd-gfx mailing list