Skip to content

Commit a395f7f

Browse files
candicelicyalexdeucher
authored andcommitted
drm/amdgpu: Retrieve CE count from ce_count_lo_chip in EccInfo table
Retrieve correctable error count from ce_count_lo_chip instead of mca_umc_status. Signed-off-by: Candice Li <candice.li@amd.com> Reviewed-by: Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent d59fcfb commit a395f7f

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

drivers/gpu/drm/amd/amdgpu/umc_v8_10.c

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -336,7 +336,7 @@ static void umc_v8_10_ecc_info_query_correctable_error_count(struct amdgpu_devic
336336
uint32_t node_inst, uint32_t umc_inst, uint32_t ch_inst,
337337
unsigned long *error_count)
338338
{
339-
uint64_t mc_umc_status;
339+
uint16_t ecc_ce_cnt;
340340
uint32_t eccinfo_table_idx;
341341
struct amdgpu_ras *ras = amdgpu_ras_get_context(adev);
342342

@@ -345,12 +345,10 @@ static void umc_v8_10_ecc_info_query_correctable_error_count(struct amdgpu_devic
345345
umc_inst * adev->umc.channel_inst_num +
346346
ch_inst;
347347

348-
/* check the MCUMC_STATUS */
349-
mc_umc_status = ras->umc_ecc.ecc[eccinfo_table_idx].mca_umc_status;
350-
if (REG_GET_FIELD(mc_umc_status, MCA_UMC_UMC0_MCUMC_STATUST0, Val) == 1 &&
351-
REG_GET_FIELD(mc_umc_status, MCA_UMC_UMC0_MCUMC_STATUST0, CECC) == 1) {
352-
*error_count += 1;
353-
}
348+
/* Retrieve CE count */
349+
ecc_ce_cnt = ras->umc_ecc.ecc[eccinfo_table_idx].ce_count_lo_chip;
350+
if (ecc_ce_cnt)
351+
*error_count += ecc_ce_cnt;
354352
}
355353

356354
static void umc_v8_10_ecc_info_query_uncorrectable_error_count(struct amdgpu_device *adev,

0 commit comments

Comments
 (0)