psu-ng: Limit calling of missing PSU helper
Only call updateMissingPSUs() if we have an interface added for the
supported configuration data, or new power supply device information.
Tested:
Simulated Rainier 2S2U
- Verify fewer "D-Bus property ... access failure exception" traces.
- Verify proper operations during power on.
- Run simulated input fault, and repair, verify errors.
Rainier 2S2U
- Apply patched binary.
- Verify D-Bus traces
- Poweron, verify lack of additional D-Bus traces.
Change-Id: I37833138e862ed910c072aa76491b0a413c18782
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
diff --git a/phosphor-power-supply/psu_manager.cpp b/phosphor-power-supply/psu_manager.cpp
index 35743f2..50917ae 100644
--- a/phosphor-power-supply/psu_manager.cpp
+++ b/phosphor-power-supply/psu_manager.cpp
@@ -377,6 +377,7 @@
if (itIntf != interfaces.cend())
{
populateSysProperties(itIntf->second);
+ updateMissingPSUs();
}
itIntf = interfaces.find(IBMCFFPSInterface);
@@ -386,10 +387,9 @@
fmt::format("InterfacesAdded for: {}", IBMCFFPSInterface)
.c_str());
getPSUProperties(itIntf->second);
+ updateMissingPSUs();
}
- updateMissingPSUs();
-
// Call to validate the psu configuration if the power is on and both
// the IBMCFFPSConnector and SupportedConfiguration interfaces have been
// processed