From 6b1e6187f684854331737e82123150e75294efda Mon Sep 17 00:00:00 2001 From: gabrpham Date: Fri, 21 Nov 2025 10:01:10 -0600 Subject: [PATCH] [SWDEV-560681] Allowed GPU enumeration to continue with non-contiguous render nodes (#1609) * Fix uninitialized variable in GPU enumeration loop (#1643) * Initialize node_to_gpu_id to prevent undefined behavior --------- Signed-off-by: gabrpham Co-authored-by: Allan Xavier --- projects/rocm-smi-lib/src/rocm_smi_kfd.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/projects/rocm-smi-lib/src/rocm_smi_kfd.cc b/projects/rocm-smi-lib/src/rocm_smi_kfd.cc index beb5270025..cf1fb78223 100644 --- a/projects/rocm-smi-lib/src/rocm_smi_kfd.cc +++ b/projects/rocm-smi-lib/src/rocm_smi_kfd.cc @@ -685,7 +685,7 @@ KFDNode::Initialize(void) { std::map>::iterator it; uint32_t node_to; - uint64_t node_to_gpu_id; + uint64_t node_to_gpu_id = 0; std::shared_ptr link; bool numa_node_found = false; for (it = io_link_map_tmp.begin(); it != io_link_map_tmp.end(); it++) { @@ -693,7 +693,7 @@ KFDNode::Initialize(void) { node_to = it->first; link = it->second; ret = ReadKFDGpuId(node_to, &node_to_gpu_id); - if (ret) {return ret;} + if (ret) {continue;} if (node_to_gpu_id == 0) { // CPU node if (numa_node_found) { if (numa_node_weight_ > link->weight()) {