[Public] No objections from me. Acked-by: Alex Deucher <alexander.deuc...@amd.com> ________________________________ From: Christian König <ckoenig.leichtzumer...@gmail.com> Sent: Thursday, December 9, 2021 10:34 AM To: Quan, Evan <evan.q...@amd.com>; Deucher, Alexander <alexander.deuc...@amd.com> Cc: amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org> 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.leichtzumer...@gmail.com> >> Sent: Tuesday, December 7, 2021 3:03 PM >> To: Quan, Evan <evan.q...@amd.com>; Deucher, Alexander >> <alexander.deuc...@amd.com> >> Cc: amd-gfx@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.q...@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-boun...@lists.freedesktop.org> On Behalf Of >>>> Christian König >>>> Sent: Monday, December 6, 2021 4:46 PM >>>> To: Deucher, Alexander <alexander.deuc...@amd.com> >>>> Cc: amd-gfx@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.koe...@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