From c00a9a709daff4523b87e65568a102b389494b42 Mon Sep 17 00:00:00 2001 From: "Pryor, Adam" Date: Mon, 10 Feb 2025 08:40:03 -0600 Subject: [PATCH] SWDEV-512736 Fix RDC Policy callback printout (#114) Change-Id: I6e018dcb0a6b272812c959649d913e3ba33def40 [ROCm/rdc commit: 93a8ab8915a6c9b2d092e4a711aff87043e2f498] --- projects/rdc/rdc_libs/rdc/src/RdcPolicyImpl.cc | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/projects/rdc/rdc_libs/rdc/src/RdcPolicyImpl.cc b/projects/rdc/rdc_libs/rdc/src/RdcPolicyImpl.cc index 29422e07d4..1d3bb76881 100644 --- a/projects/rdc/rdc_libs/rdc/src/RdcPolicyImpl.cc +++ b/projects/rdc/rdc_libs/rdc/src/RdcPolicyImpl.cc @@ -82,8 +82,7 @@ rdc_status_t RdcPolicyImpl::rdc_policy_set(rdc_gpu_group_t group_id, rdc_policy_ gpu_index = group_info.entity_ids[i]; status = metric_fetcher_->fetch_smi_field(gpu_index, RDC_FI_GPU_PAGE_RETRIED, &value); - if (status == RDC_ST_MSI_ERROR) - return RDC_ST_NOT_SUPPORTED; + if (status == RDC_ST_MSI_ERROR) return RDC_ST_NOT_SUPPORTED; } } @@ -219,16 +218,16 @@ void RdcPolicyImpl::rdc_policy_check_condition() { status = metric_fetcher_->fetch_smi_field(gpu_index, map[policy.condition.type], &value); if (status == RDC_ST_OK) { if (value.value.l_int > policy.condition.value) { - if (RDC_POLICY_ACTION_GPU_RESET == policy.action) { - rdc_policy_gpu_reset(gpu_index); - } - // callback if needed if (callback) { rdc_policy_callback_response_t response = {1, policy.condition, group_id, value.value.l_int}; callback(&response); } + + if (RDC_POLICY_ACTION_GPU_RESET == policy.action) { + rdc_policy_gpu_reset(gpu_index); + } } } }