You are looking at outdated code, that stuff is gone by now.
amd-staging-drm-next probably needs a rebase.
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
