From 34b3c9775fb0d42f59133f406a94d9eb392280a2 Mon Sep 17 00:00:00 2001 From: "Bill(Shuzhou) Liu" Date: Thu, 1 Dec 2022 15:02:02 -0600 Subject: [PATCH] SWDEV-370010 - SMILIB returns 'Aborted (core dumped)' When the sysfs file does not exist, it will return NOT_SUPPORTED instead of assert faiure. Change-Id: I3a1c9d7cf627901e141e2617496dfafd2e6b3f25 [ROCm/amdsmi commit: 0a3ca5c555bd4de6e0de8d4165b77390622a65f2] --- projects/amdsmi/rocm_smi/src/rocm_smi.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/projects/amdsmi/rocm_smi/src/rocm_smi.cc b/projects/amdsmi/rocm_smi/src/rocm_smi.cc index f6d1638b0e..7d49aef5ad 100755 --- a/projects/amdsmi/rocm_smi/src/rocm_smi.cc +++ b/projects/amdsmi/rocm_smi/src/rocm_smi.cc @@ -3151,14 +3151,17 @@ rsmi_counter_available_counters_get(uint32_t dv_ind, TRY CHK_SUPPORT_VAR(available, grp) DEVICE_MUTEX - uint64_t val; + uint64_t val = 0; switch (grp) { case RSMI_EVNT_GRP_XGMI: case RSMI_EVNT_GRP_XGMI_DATA_OUT: ret = get_dev_value_int(amd::smi::kDevDFCountersAvailable, dv_ind, &val); - assert(val < UINT32_MAX); + if (ret != RSMI_STATUS_SUCCESS) + return ret; + if (val == UINT32_MAX) + return RSMI_STATUS_NOT_SUPPORTED; *available = static_cast(val); break;