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

2021-12-09 Thread Deucher, Alexander
[Public] No objections from me. Acked-by: Alex Deucher From: Christian König Sent: Thursday, December 9, 2021 10:34 AM To: Quan, Evan ; Deucher, Alexander Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: don't skip runtime pm get o

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

2021-12-09 Thread Christian König
Am 07.12.21 um 08:40 schrieb Quan, Evan: [AMD Official Use Only] -Original Message- From: Christian König Sent: Tuesday, December 7, 2021 3:03 PM To: Quan, Evan ; Deucher, Alexander Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: don't skip runtime pm get o

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

2021-12-06 Thread Quan, Evan
[AMD Official Use Only] > -Original Message- > From: Christian König > Sent: Tuesday, December 7, 2021 3:03 PM > To: Quan, Evan ; Deucher, Alexander > > Cc: amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH] drm/amdgpu: don't skip runtime pm get on A+A conf

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

2021-12-06 Thread Christian König
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: [AM

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

2021-12-06 Thread 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-