[PATCH 1/1] drm/amdgpu: make sure we unpin the UVD BO

Christian König christian.koenig at amd.com
Fri May 28 14:36:42 UTC 2021


Am 20.04.21 um 12:50 schrieb Nirmoy Das:
> Releasing pinned BOs is illegal now.
> UVD 6 was missing from:
> commit 2f40801dc553 ("drm/amdgpu: make sure we unpin the UVD BO")
>
> Signed-off-by: Nirmoy Das <nirmoy.das at amd.com>

Reviewed-by: Christian König <christian.koenig at amd.com>

Maybe add a CC:stable tag as well.

Sorry I somehow overlooked this patch. Can you make sure that it lands 
in amd-staging-drm-next ASAP? We have an user complaining.

Thanks,
Christian.

> ---
>   drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
> index 760859880c1e..4eebf973a065 100644
> --- a/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/uvd_v6_0.c
> @@ -357,6 +357,7 @@ static int uvd_v6_0_enc_ring_test_ib(struct amdgpu_ring *ring, long timeout)
>   
>   error:
>   	dma_fence_put(fence);
> +	amdgpu_bo_unpin(bo);
>   	amdgpu_bo_unreserve(bo);
>   	amdgpu_bo_unref(&bo);
>   	return r;



More information about the amd-gfx mailing list