[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
