From 291d697c4e55bcad6c5dfd6e26f191e00ffb5339 Mon Sep 17 00:00:00 2001 From: Maxime Grenu Date: Tue, 10 Mar 2026 20:48:36 +0100 Subject: [PATCH] diag: rename 'Device IDs' to 'PCI Device IDs' in dcgmi diag output The dcgmi diag metadata section labels the PCI device ID column as 'GPU Device IDs Detected', which is misleading on multi-GPU systems where every GPU of the same model shares an identical PCI device ID. Users reasonably expect N distinct values for N GPUs. Rename the label to 'GPU PCI Device IDs Detected' so it accurately reflects that the values are PCI hardware SKU identifiers rather than unique per-GPU identifiers. Also update the corresponding JSON string constant NVVS_GPU_DEV_IDS for consistency. Fixes: NVIDIA/DCGM#282 Signed-off-by: Maxime Grenu --- dcgmi/Diag.cpp | 2 +- nvvs/include/NvvsJsonStrings.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dcgmi/Diag.cpp b/dcgmi/Diag.cpp index 626acc6d..ac3d899a 100644 --- a/dcgmi/Diag.cpp +++ b/dcgmi/Diag.cpp @@ -768,7 +768,7 @@ void Diag::HelperDisplayVersionAndDevIds(dcgmDiagResponse_v12 const &response) c { cmdView.addDisplayParameter( DATA_NAME_TAG, - fmt::format("{} Device IDs Detected", + fmt::format("{} PCI Device IDs Detected", DcgmFieldsGetEntityGroupString(static_cast(gId)))); cmdView.addDisplayParameter(DATA_INFO_TAG, str); cmdView.display(); diff --git a/nvvs/include/NvvsJsonStrings.h b/nvvs/include/NvvsJsonStrings.h index 17bef231..4b71a304 100644 --- a/nvvs/include/NvvsJsonStrings.h +++ b/nvvs/include/NvvsJsonStrings.h @@ -38,7 +38,7 @@ #define NVVS_ERROR_SEVERITY "error_severity" #define NVVS_ITERATIONS "iterations" #define NVVS_RESULT "Overall Result" -#define NVVS_GPU_DEV_IDS "GPU Device IDs" +#define NVVS_GPU_DEV_IDS "GPU PCI Device IDs" #define NVVS_GPU_SERIALS "GPU Device Serials" #define NVVS_DRIVER_VERSION "Driver Version Detected" #define NVVS_AUX_DATA "aux_data"