commit 838529b5400129e8a3a8007ca83102c7733ec01c
author James Feist <james.feist@linux.intel.com> Wed Sep 02 16:57:05 2020 -0700
committer James Feist <james.feist@linux.intel.com> Thu Sep 03 00:00:48 2020 +0000
tree 05b059b3089c461c0b6216fc13be3470cbd67809
parent 7d7579f85e8dcf46411f6604b31a437496514811

CPUSensor: Mark interface down when timeout

If we timeout it is because we cannot access the sensor.
Timeouts take a second, but with many sensors, they add
up. Mark the sensor failed the first time this
happens to boost the fans as soon as possible.

Tested: Fans boost after interface down

Change-Id: I753a3db65a24d45423998cb18bdcbc1f4147dbf4
Signed-off-by: James Feist <james.feist@linux.intel.com>

include/CPUSensor.hpp

diff --git a/include/CPUSensor.hpp b/include/CPUSensor.hpp
index 24e4de6..07f7ffd 100644
--- a/include/CPUSensor.hpp
+++ b/include/CPUSensor.hpp
@@ -46,6 +46,7 @@
     double dtsOffset;
     bool show;
     size_t pollTime;
+    bool loggedInterfaceDown = false;
     void setupRead(void);
     void handleResponse(const boost::system::error_code& err);
     void checkThresholds(void) override;

src/CPUSensor.cpp

diff --git a/src/CPUSensor.cpp b/src/CPUSensor.cpp
index 291d72f..127d68a 100644
--- a/src/CPUSensor.cpp
+++ b/src/CPUSensor.cpp
@@ -195,10 +195,26 @@
 
 void CPUSensor::handleResponse(const boost::system::error_code& err)
 {
+
     if (err == boost::system::errc::bad_file_descriptor)
     {
         return; // we're being destroyed
     }
+    else if (err == boost::system::errc::operation_canceled)
+    {
+        if (readingStateGood())
+        {
+            if (!loggedInterfaceDown)
+            {
+                std::cerr << name << " interface down!\n";
+                loggedInterfaceDown = true;
+            }
+            pollTime = 10000 + rand() % 10000;
+            markFunctional(false);
+        }
+        return;
+    }
+    loggedInterfaceDown = false;
     pollTime = CPUSensor::sensorPollMs;
     std::istream responseStream(&readBuf);
     if (!err)