[PATCH] drm/amdgpu: don't skip runtime pm get on A+A config

Christian König ckoenig.leichtzumerken at gmail.com
Thu Dec 9 15:34:24 UTC 2021


Am 07.12.21 um 08:40 schrieb Quan, Evan:
> [AMD Official Use Only]
>> -----Original Message-----
>> From: Christian König <ckoenig.leichtzumerken at gmail.com>
>> Sent: Tuesday, December 7, 2021 3:03 PM
>> To: Quan, Evan <Evan.Quan at amd.com>; Deucher, Alexander
>> <Alexander.Deucher at amd.com>
>> Cc: amd-gfx at lists.freedesktop.org
>> Subject: Re: [PATCH] drm/amdgpu: don't skip runtime pm get on A+A config
>>
>> You are looking at outdated code, that stuff is gone by now.
>> amd-staging-drm-next probably needs a rebase.
> Yep, I can see it in the vanilla kernel.
> The patch is acked-by: Evan Quan <evan.quan at amd.com>

Thanks.

Alex any objections that I push this to drm-misc-next? It was found 
while working on changes already upstream in that function and would 
conflict if we push it through amd-staging-drm-next.

Regards,
Christian.

>
> BR
> Evan
>> And this code was what the check was initially good for. Just skipping the PM
>> stuff as well on A+A was unintentionally.
>>
>> Regards,
>> Christian.
>>
>> Am 07.12.21 um 02:58 schrieb Quan, Evan:
>>> [AMD Official Use Only]
>>>
>>> It seems more jobs(below) other than bumping the runpm counter are
>> performed.
>>> Are they desired also?
>>>
>>> 	r = __dma_resv_make_exclusive(bo->tbo.base.resv);
>>> 	if (r)
>>> 		goto out;
>>>
>>> 	bo->prime_shared_count++;
>>>
>>> BR
>>> Evan
>>>> -----Original Message-----
>>>> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of
>>>> Christian König
>>>> Sent: Monday, December 6, 2021 4:46 PM
>>>> To: Deucher, Alexander <Alexander.Deucher at amd.com>
>>>> Cc: amd-gfx at lists.freedesktop.org
>>>> Subject: [PATCH] drm/amdgpu: don't skip runtime pm get on A+A config
>>>>
>>>> The runtime PM get was incorrectly added after the check.
>>>>
>>>> Signed-off-by: Christian König <christian.koenig at amd.com>
>>>> ---
>>>>    drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c | 3 ---
>>>>    1 file changed, 3 deletions(-)
>>>>
>>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
>>>> b/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
>>>> index ae6ab93c868b..4896c876ffec 100644
>>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
>>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c
>>>> @@ -61,9 +61,6 @@ static int amdgpu_dma_buf_attach(struct dma_buf
>>>> *dmabuf,
>>>>    	if (pci_p2pdma_distance_many(adev->pdev, &attach->dev, 1, true)
>> <
>>>> 0)
>>>>    		attach->peer2peer = false;
>>>>
>>>> -	if (attach->dev->driver == adev->dev->driver)
>>>> -		return 0;
>>>> -
>>>>    	r = pm_runtime_get_sync(adev_to_drm(adev)->dev);
>>>>    	if (r < 0)
>>>>    		goto out;
>>>> --
>>>> 2.25.1



More information about the amd-gfx mailing list