|
[Public]
No objections from me.
Acked-by: Alex Deucher <alexander.deucher@xxxxxxx>
From: Christian König <ckoenig.leichtzumerken@xxxxxxxxx>
Sent: Thursday, December 9, 2021 10:34 AM To: Quan, Evan <Evan.Quan@xxxxxxx>; Deucher, Alexander <Alexander.Deucher@xxxxxxx> Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx> Subject: Re: [PATCH] drm/amdgpu: don't skip runtime pm get on A+A config Am 07.12.21 um 08:40 schrieb Quan, Evan:
> [AMD Official Use Only] >> -----Original Message----- >> From: Christian König <ckoenig.leichtzumerken@xxxxxxxxx> >> Sent: Tuesday, December 7, 2021 3:03 PM >> To: Quan, Evan <Evan.Quan@xxxxxxx>; Deucher, Alexander >> <Alexander.Deucher@xxxxxxx> >> Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx >> 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@xxxxxxx> 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@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of >>>> Christian König >>>> Sent: Monday, December 6, 2021 4:46 PM >>>> To: Deucher, Alexander <Alexander.Deucher@xxxxxxx> >>>> Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx >>>> 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@xxxxxxx> >>>> --- >>>> 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 |
