From 729a26605bce06b664aa84aa6c95fc17c80a3764 Mon Sep 17 00:00:00 2001 From: "Oliveira, Daniel" Date: Fri, 1 Mar 2024 17:30:07 -0600 Subject: [PATCH] fix: [SWDEV-432974] [rocm/rocm_smi_lib] Checks returned error by get_gpu_pci_bandwith() before assert Code changes related to the following: * Unit tests Change-Id: Ia0fe64f168711147c5e66c7917cf633be40dee9f Signed-off-by: Oliveira, Daniel [ROCm/rocm_smi_lib commit: 35b561fd69b8255ec486f49867e7a240c0eea6cb] --- .../tests/rocm_smi_test/functional/frequencies_read.cc | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/projects/rocm-smi-lib/tests/rocm_smi_test/functional/frequencies_read.cc b/projects/rocm-smi-lib/tests/rocm_smi_test/functional/frequencies_read.cc index 5b956e9951..412b69d7c4 100755 --- a/projects/rocm-smi-lib/tests/rocm_smi_test/functional/frequencies_read.cc +++ b/projects/rocm-smi-lib/tests/rocm_smi_test/functional/frequencies_read.cc @@ -173,7 +173,14 @@ void TestFrequenciesRead::Run(void) { print_frequencies(&b.transfer_rate, b.lanes); // Verify api support checking functionality is working err = rsmi_dev_pci_bandwidth_get(i, nullptr); - ASSERT_EQ(err, RSMI_STATUS_INVALID_ARGS); + if (err != rsmi_status_t::RSMI_STATUS_NOT_SUPPORTED) { + ASSERT_EQ(err, RSMI_STATUS_INVALID_ARGS); + } + else { + auto status_string(""); + rsmi_status_string(err, &status_string); + std::cout << "\t\t** rsmi_dev_pci_bandwidth_get(): " << status_string << "\n"; + } } } }