[SWDEV-549373] Added vbios and pldm information to version header and adjusted platform info display

Signed-off-by: gabrpham_amdeng <Gabriel.Pham@amd.com>


[ROCm/amdsmi commit: 71c8b92076]

projects/amdsmi/amdsmi_cli/amdsmi_commands.py

@@ -6992,6 +6992,8 @@ class AMDSMICommands():
         processors = amdsmi_interface.amdsmi_get_processor_handles()
         version_info = {"amd-smi": "N/A",
                         "amdgpu version": "N/A",
+                        "fw pldm version": "N/A",
+                        "vbios version": "N/A",
                         "rocm version": (False, "N/A")}
         version_info['rocm version'] = amdsmi_interface.amdsmi_get_rocm_version()
         try:
@@ -6999,6 +7001,23 @@ class AMDSMICommands():
         except amdsmi_exception.AmdSmiLibraryException as e:
             version_info["amdgpu version"] = "N/A"
             logging.debug("Failed to get driver info for gpu: %s", e.get_error_info())
+        try:
+            fw_info = amdsmi_interface.amdsmi_get_fw_info(processors[0])
+            for fw in fw_info['fw_list']:
+                if "pldm" in fw.keys():
+                    version_info['fw pldm version'] = fw['pldm']
+                    # we only need to find one of them
+                    break
+        except amdsmi_exception.AmdSmiLibraryException as e:
+            version_info['fw pldm version'] = "N/A"
+            logging.debug("Failed to get fw pldm info for gpu: %s", e.get_error_info())
+        try:
+            version_info['vbios version'] = amdsmi_interface.amdsmi_get_gpu_vbios_info(processors[0])["version"]
+            if version_info['vbios version'] == "":
+                version_info['vbios version'] = "N/A"
+        except amdsmi_exception.AmdSmiLibraryException as e:
+            version_info['vbios version'] = "N/A"
+            logging.debug("Failed to get vbios info for gpu: %s", e.get_error_info())
 
         version_info["amd-smi"] = f'{__version__}'
 

projects/amdsmi/amdsmi_cli/amdsmi_logger.py

@@ -1023,11 +1023,24 @@ class AMDSMILogger():
                     amdgpu_version = "N/A".ljust(8)
             else:
                 amdgpu_version = str(driver_version['driver_version'])[:8].ljust(8)
+        fw_pldm_version = str(output['version_info']['fw pldm version'])
+        vbios_version = str(output['version_info']['vbios version'])
 
         # print GPU info
         print(default_line_1)
         print("| AMD-SMI {0:20s} amdgpu version: {1:8s} ROCm version: {2:8s} |".format(amd_smi_version.ljust(20), amdgpu_version, rocm_version))
-        print("| Platform: {0:20.20s} {1:46s}|".format(str(self.helpers.os_info()), ""))
+
+        # adjust format depending on whether vbios or fw pldm version is present
+        if vbios_version != "N/A" and fw_pldm_version != "N/A":
+            print("| VBIOS version: {0:22s}  {1:12s}  FW PLDM: {2:15s}|".format(vbios_version, "", fw_pldm_version))
+        elif vbios_version != "N/A" and fw_pldm_version == "N/A":
+            print("| VBIOS version: {0:22s}  {1:37s} |".format(vbios_version, ""))
+        elif fw_pldm_version != "N/A" and vbios_version == "N/A":
+            print("| FW PLDM: {0:15s}  {1:50s} |".format(fw_pldm_version, ""))
+        else:
+            pass  # Both VBIOS and FW PLDM versions are "N/A" so skip this line
+
+        print("| Platform: {0:25.25s} {1:41s}|".format(str(self.helpers.os_info()), ""))
         print(default_line_2)
         print("| BDF                        GPU-Name | Mem-Uti   Temp   UEC       Power-Usage |")
         print("| GPU  HIP-ID  OAM-ID  Partition-Mode | GFX-Uti    Fan               Mem-Usage |")