commit fa88c5e6a78dc4273254370650c784a8edbd2bf7
author Alexander Hansen <alexander.hansen@9elements.com> Fri Sep 12 12:33:16 2025 +0200
committer Alexander <alexander.hansen@9elements.com> Tue Oct 14 20:28:38 2025 +0000
tree 7cf39a83e07a3803e4cf65b50eb00fa175fcdbc4
parent 46906e5d3e9df92cf5ac495d21dc041f51f52fe9

entity-manager: use PDI constants for Host State

PDI already defines interface name and path namespace, use that instead
of duplicating it here.

Using the interface variable for service name since service name is not
defined in PDI and it's the same.

Defining default service name in [1].

Tested: on Tyan S8030

```
Sep 12 11:32:18 s8030-bmc-30303035c0c1 entity-manager[14717]: querying initial power state
```

```
ipmitool power on
```

```
Sep 12 11:37:27 s8030-bmc-30303035c0c1 entity-manager[14717]: power match triggered
Sep 12 11:37:28 s8030-bmc-30303035c0c1 entity-manager[14717]: power match triggered
Sep 12 11:37:28 s8030-bmc-30303035c0c1 entity-manager[14717]: power match triggered
Sep 12 11:37:28 s8030-bmc-30303035c0c1 entity-manager[14717]: power match triggered
```

```
ipmitool power off
```

```
Sep 12 11:37:52 s8030-bmc-30303035c0c1 entity-manager[14717]: power match triggered
```

References:
[1] https://gerrit.openbmc.org/c/openbmc/phosphor-dbus-interfaces/+/83644

Change-Id: Ibbb2828ab6b7e9216ccd4c37e5697c5ad4b02177
Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>

src/entity_manager/power_status_monitor.cpp

diff --git a/src/entity_manager/power_status_monitor.cpp b/src/entity_manager/power_status_monitor.cpp
index 04edbd5..079c320 100644
--- a/src/entity_manager/power_status_monitor.cpp
+++ b/src/entity_manager/power_status_monitor.cpp
@@ -4,15 +4,18 @@
 
 #include <phosphor-logging/lg2.hpp>
 #include <sdbusplus/bus/match.hpp>
+#include <xyz/openbmc_project/State/Host/client.hpp>
 
 #include <flat_map>
 
 namespace power
 {
 
-const static constexpr char* busname = "xyz.openbmc_project.State.Host";
-const static constexpr char* interface = "xyz.openbmc_project.State.Host";
-const static constexpr char* path = "/xyz/openbmc_project/state/host0";
+using HostState = sdbusplus::common::xyz::openbmc_project::state::Host;
+
+const static std::string path =
+    std::format("{}/{}{}", HostState::namespace_path::value,
+                HostState::namespace_path::host, 0);
 const static constexpr char* property = "CurrentHostState";
 
 PowerStatusMonitor::PowerStatusMonitor(sdbusplus::asio::connection& conn) :
@@ -21,7 +24,7 @@
                "type='signal',interface='" +
                    std::string(em_utils::properties::interface) + "',path='" +
                    std::string(power::path) + "',arg0='" +
-                   std::string(power::interface) + "'",
+                   std::string(HostState::interface) + "'",
                std::bind_front(&PowerStatusMonitor::handlePowerMatch, this))
 
 {
@@ -51,6 +54,8 @@
 void PowerStatusMonitor::getInitialPowerStatus(
     sdbusplus::asio::connection& conn)
 {
+    lg2::debug("querying initial power state");
+
     conn.async_method_call(
         [this](boost::system::error_code ec,
                const std::variant<std::string>& state) {
@@ -60,8 +65,8 @@
             }
             powerStatusOn = std::get<std::string>(state).ends_with("Running");
         },
-        power::busname, power::path, em_utils::properties::interface,
-        em_utils::properties::get, power::interface, power::property);
+        HostState::interface, power::path, em_utils::properties::interface,
+        em_utils::properties::get, HostState::interface, power::property);
 }
 
 } // namespace power