[AMD Official Use Only] The change breaks initialization as well. shall we check in_gpu_reset to exclude reset case? Regards, Hawking -----Original Message----- From: Zhou1, Tao <Tao.Zhou1@xxxxxxx> Sent: Tuesday, October 26, 2021 17:27 To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx; Zhang, Hawking <Hawking.Zhang@xxxxxxx> Cc: Zhou1, Tao <Tao.Zhou1@xxxxxxx> Subject: [PATCH] drm/amdgpu: remove GPRs init for ALDEBARAN Remove GPRs init for ALDEBARAN temporarily, will add the init once the algorithm is stable. Signed-off-by: Tao Zhou <tao.zhou1@xxxxxxx> --- drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2.c index 24852a24cc58..c85bad4f1394 100644 --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2.c +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_4_2.c @@ -706,10 +706,9 @@ int gfx_v9_4_2_do_edc_gpr_workarounds(struct amdgpu_device *adev) if (!amdgpu_ras_is_supported(adev, AMDGPU_RAS_BLOCK__GFX)) return 0; - /* Workaround for some CU settings, skip GPRs init. - Will remove it once GPRs init algorithm works for all CU settings */ - if (adev->gfx.cu_info.bitmap[1][1] == 0x3f7f) - return 0; + /* Workaround for ALDEBARAN, skip GPRs init. + Will remove it once GPRs init algorithm works for all CU settings. */ + return 0; gfx_v9_4_2_do_sgprs_init(adev); -- 2.17.1
