commit d7966892aae75d48bd6adabdaec4aede03818f3d
author Prithvi Pai <ppai@nvidia.com> Sat Aug 02 18:40:13 2025 +0530
committer Prithvi Pai <ppai@nvidia.com> Tue Aug 05 00:51:55 2025 +0000
tree d338136ccf430db1261a4d8a8992e79530932f67
parent bffaa1100755e28549d034251c6c2b3feb9c27cc

Fix GetSystemGUID

IPMI spec mentions the following
```
This command returns a GUID (Globally Unique ID), also referred to
as a UUID (Universally Unique IDentifier),for the management controller.
```

The current implementation returns the UUID implemented by FRU, MCTP
etc.
On gb200nvl-bmc, the UUID returned was from a MCTP  endpoint device.
This goes against the spec as the UUID is not of management controller

Tested:
UUID now returned is from phosphor-settings-manager

Change-Id: Ia914f126f3d8cab72b81cc13b237fa8d00f9d79c
Signed-off-by: Prithvi Pai <ppai@nvidia.com>

apphandler.cpp

diff --git a/apphandler.cpp b/apphandler.cpp
index 1e32061..c059f56 100644
--- a/apphandler.cpp
+++ b/apphandler.cpp
@@ -849,25 +849,16 @@
 auto ipmiAppGetSystemGuid(ipmi::Context::ptr& ctx)
     -> ipmi::RspType<std::array<uint8_t, 16>>
 {
+    static constexpr auto uuidService = "xyz.openbmc_project.Settings";
+    static constexpr auto uuidObject = "/xyz/openbmc_project/Common/UUID";
     static constexpr auto uuidInterface = "xyz.openbmc_project.Common.UUID";
     static constexpr auto uuidProperty = "UUID";
 
-    // Get the Inventory object implementing BMC interface
-    ipmi::DbusObjectInfo objectInfo{};
-    boost::system::error_code ec = ipmi::getDbusObject(
-        ctx, uuidInterface, ipmi::sensor::inventoryRoot, objectInfo);
-    if (ec.value())
-    {
-        lg2::error("Failed to locate System UUID object, "
-                   "interface: {INTERFACE}, error: {ERROR}",
-                   "INTERFACE", uuidInterface, "ERROR", ec.message());
-    }
-
-    // Read UUID property value from bmcObject
+    // Read UUID property value from settings object
     // UUID is in RFC4122 format Ex: 61a39523-78f2-11e5-9862-e6402cfc3223
     std::string rfc4122Uuid{};
-    ec = ipmi::getDbusProperty(ctx, objectInfo.second, objectInfo.first,
-                               uuidInterface, uuidProperty, rfc4122Uuid);
+    boost::system::error_code ec = ipmi::getDbusProperty(
+        ctx, uuidService, uuidObject, uuidInterface, uuidProperty, rfc4122Uuid);
     if (ec.value())
     {
         lg2::error("Failed to read System UUID property, "