[AMD Official Use Only - General] Thanks for sorting out this, that is making sense since driver already skip kfd device suspend and this will skip kfd_iommu_suspend() as well.
Reviewed-by: Prike Liang <prike.li...@amd.com> Regards, --Prike > -----Original Message----- > From: Limonciello, Mario <mario.limoncie...@amd.com> > Sent: Thursday, April 13, 2023 9:25 AM > To: Liu, Aaron <aaron....@amd.com>; amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Kuehling, Felix > <felix.kuehl...@amd.com>; Liang, Prike <prike.li...@amd.com> > Subject: Re: drm/amdgpu: skip kfd-iommu suspend/resume for S0ix > > On 4/5/2023 06:29, Aaron Liu wrote: > > GFX is in gfxoff mode during s0ix so we shouldn't need to actually > > execute kfd_iommu_suspend/kfd_iommu_resume operation. > > > > Signed-off-by: Aaron Liu <aaron....@amd.com> > > Acked-by: Alex Deucher <alexander.deuc...@amd.com> > Reviewed-by: Mario Limonciello <mario.limoncie...@amd.com> > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 8 +++++--- > > 1 file changed, 5 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > index 3b6b85d9e0be..5094be94fa06 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > > @@ -3304,9 +3304,11 @@ static int amdgpu_device_ip_resume(struct > amdgpu_device *adev) > > { > > int r; > > > > - r = amdgpu_amdkfd_resume_iommu(adev); > > - if (r) > > - return r; > > + if (!adev->in_s0ix) { > > + r = amdgpu_amdkfd_resume_iommu(adev); > > + if (r) > > + return r; > > + } > > > > r = amdgpu_device_ip_resume_phase1(adev); > > if (r)