#
e1646276
|
| 25-Jul-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
PowerStatusMonitor: create DBus match in ctor
On request [1], create this change to create the DBus match for our power status monitor in its constructor.
We get rid of the invalid state of the mat
PowerStatusMonitor: create DBus match in ctor
On request [1], create this change to create the DBus match for our power status monitor in its constructor.
We get rid of the invalid state of the match not existing and can drop a check for it.
We can also drop a call in 'main' function previously used to initialize the match. The public API of this class is simplified.
Tested: Inspection only.
References: [1] https://gerrit.openbmc.org/c/openbmc/entity-manager/+/81483/comment/858d95be_e2f9f6b8/
Change-Id: Ib05941a90ba2520b25e2c2b1e9a1e325caa722ae Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
show more ...
|
#
0a4bb2a9
|
| 25-Jul-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
extract PowerStatusMonitor::handlePowerMatch
Extract this inline lambda into a named function.
Add a debug log statement for easier testing.
Tested: power match is triggered on power state change.
extract PowerStatusMonitor::handlePowerMatch
Extract this inline lambda into a named function.
Add a debug log statement for easier testing.
Tested: power match is triggered on power state change.
``` ipmitool power on ```
``` Jul 30 14:14:33 s8030-bmc-30303035c0c1 entity-manager[696]: power match triggered Jul 30 14:14:34 s8030-bmc-30303035c0c1 entity-manager[696]: power match triggered Jul 30 14:14:34 s8030-bmc-30303035c0c1 entity-manager[696]: power match triggered Jul 30 14:14:34 s8030-bmc-30303035c0c1 entity-manager[696]: power match triggered ```
It looks like it is triggered multiple times but that has nothing to do with the changes done in this patch.
Change-Id: If7828a1195565485aeb890223342d6a135da2c46 Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
show more ...
|
#
8c99fcf5
|
| 24-Jul-2025 |
Alexander Hansen <alexander.hansen@9elements.com> |
entity-manager: create power_status_monitor.cpp
These functions belong to PowerStatusMonitor class but were still in utils.cpp.
Code was only moved and otherwise unchanged.
Tested: Inspection only
entity-manager: create power_status_monitor.cpp
These functions belong to PowerStatusMonitor class but were still in utils.cpp.
Code was only moved and otherwise unchanged.
Tested: Inspection only.
Change-Id: I179eb53718df376116e056b1285f7d1eb4a0ec22 Signed-off-by: Alexander Hansen <alexander.hansen@9elements.com>
show more ...
|