drm/amdgpu: disable gfxoff when using register read interface
authorAlex Deucher <alexander.deucher@amd.com>
Thu, 14 Nov 2019 16:39:05 +0000 (11:39 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 19 Nov 2019 15:12:54 +0000 (10:12 -0500)
When gfxoff is enabled, accessing gfx registers via MMIO
can lead to a hang.

Bug: https://bugzilla.kernel.org/show_bug.cgi?id=205497
Acked-by: Xiaojie Yuan <xiaojie.yuan@amd.com>
Reviewed-by: Evan Quan <evan.quan@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c

index 39e37a53cb905954ce58087c5b7faf435d478e63..31d039a3f8c0e3443d310562327f5670852f42f7 100644 (file)
@@ -664,15 +664,19 @@ static int amdgpu_info_ioctl(struct drm_device *dev, void *data, struct drm_file
                        return -ENOMEM;
                alloc_size = info->read_mmr_reg.count * sizeof(*regs);
 
-               for (i = 0; i < info->read_mmr_reg.count; i++)
+               amdgpu_gfx_off_ctrl(adev, false);
+               for (i = 0; i < info->read_mmr_reg.count; i++) {
                        if (amdgpu_asic_read_register(adev, se_num, sh_num,
                                                      info->read_mmr_reg.dword_offset + i,
                                                      &regs[i])) {
                                DRM_DEBUG_KMS("unallowed offset %#x\n",
                                              info->read_mmr_reg.dword_offset + i);
                                kfree(regs);
+                               amdgpu_gfx_off_ctrl(adev, true);
                                return -EFAULT;
                        }
+               }
+               amdgpu_gfx_off_ctrl(adev, true);
                n = copy_to_user(out, regs, min(size, alloc_size));
                kfree(regs);
                return n ? -EFAULT : 0;