commit 3452f0eacd013ae458f5d88af8714d715864037e
author James Feist <james.feist@linux.intel.com> Wed Oct 31 10:08:47 2018 -0700
committer James Feist <james.feist@linux.intel.com> Fri Nov 16 11:34:25 2018 -0800
tree dd9f94f6667220f54c169cda4011afd3bf5399df
parent f21c35a67ca464b4a0a99cb288721f0f100ba363

Increase poll time on failure

Polling on failure causes increased CPU usage,
slow it down. Also always use update value instead
of setting value directly.

Tested-by: Turned off system and noticed lower cpu
usage.

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

sensors/src/CPUSensor.cpp

diff --git a/sensors/src/CPUSensor.cpp b/sensors/src/CPUSensor.cpp
index 98201b1..236dd81 100644
--- a/sensors/src/CPUSensor.cpp
+++ b/sensors/src/CPUSensor.cpp
@@ -89,6 +89,7 @@
     {
         return; // we're being destroyed
     }
+    size_t pollTime = CPUSensor::sensorPollMs;
     std::istream responseStream(&readBuf);
     if (!err)
     {
@@ -112,6 +113,7 @@
     }
     else
     {
+        pollTime = sensorFailedPollTimeMs;
         errCount++;
     }
 
@@ -129,8 +131,7 @@
         else
         {
             errCount = 0; // check power again in 10 cycles
-            sensorInterface->set_property(
-                "Value", std::numeric_limits<double>::quiet_NaN());
+            updateValue(std::numeric_limits<double>::quiet_NaN());
         }
     }
 
@@ -142,8 +143,7 @@
         return; // we're no longer valid
     }
     inputDev.assign(fd);
-    waitTimer.expires_from_now(
-        boost::posix_time::milliseconds(CPUSensor::sensorPollMs));
+    waitTimer.expires_from_now(boost::posix_time::milliseconds(pollTime));
     waitTimer.async_wait([&](const boost::system::error_code &ec) {
         if (ec == boost::asio::error::operation_aborted)
         {