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

Das, Nirmoy nirmoy.das at amd.com
Fri May 28 14:49:10 UTC 2021


On 5/28/2021 4:36 PM, Christian König wrote:
> 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.
>

No worries, I will push it now with CC:stable.


Thanks,

Nirmoy


> 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 dri-devel mailing list