commit c2a311b0055b5af0d93bf5db0e9834271d93d458
author Jinliang Wang <jinliangw@google.com> Wed Apr 26 18:36:56 2023 +0000
committer Jinliang Wang <jinliangw@google.com> Wed Apr 26 18:41:01 2023 +0000
tree 3ba45ea181686b5de20bcf5d2d0688bde155c289
parent cdc8dca44b5a60af4bb86598043b54f955b81bcd

Handle InterfacesRemoved signal under /xyz/openbmc_project/sensor path

Listen for InterfacesRemoved signal and restart control loops. Without
this, the pid loop may run with staled sensor value if the corresponding
object or interface is removed.

Tested:
  ...
  PID Zone 0 max SetPoint 16800 requested by ...
  PID Zone 0 max SetPoint 16800 requested by ...
<- systemctl stop  xyz.openbmc_project.nvmesensor here ->
  New configuration detected, reloading
  .1 Failed during restartControlLoops, try again: wait for count back to 1
  Zone Zone 1 is at ZoneIndex 1
  Zone Zone 0 is at ZoneIndex 0
  Zone Zone 2 is at ZoneIndex 2
  ...

Change-Id: I900edda48b71dc7b6ae79a4b51f4088aea69183a
Signed-off-by: Jinliang Wang <jinliangw@google.com>

dbus/dbusconfiguration.cpp

diff --git a/dbus/dbusconfiguration.cpp b/dbus/dbusconfiguration.cpp
index 96d2b4b..f055c11 100644
--- a/dbus/dbusconfiguration.cpp
+++ b/dbus/dbusconfiguration.cpp
@@ -249,6 +249,10 @@
         "type='signal',member='InterfacesAdded',arg0path='/xyz/openbmc_project/"
         "sensors/'",
         eventHandler, &timer);
+    matches.emplace_back(bus,
+                         "type='signal',member='InterfacesRemoved',arg0path='/"
+                         "xyz/openbmc_project/sensors/'",
+                         eventHandler, &timer);
 }
 
 /**