commit 98be984491da67666534f9b958fa689ca22d645c
author Zbigniew Kurzynski <zbigniew.kurzynski@intel.com> Mon Sep 07 18:49:15 2020 +0200
committer Zbigniew Kurzynski <zbigniew.kurzynski@intel.com> Tue Sep 15 06:41:18 2020 +0000
tree 5fec0c041053cbbd89ddc1e77ce501c25e1a4a32
parent 8aeffd91ff3434f7812e9fdb6b0b03c6119921dd

Reading min, max values for ‘cap’ every 8 times.

There is no need to read min and max values of CPU
capabilities with every second. The CPU capabilities
doesn’t change to often if ever. The hwmon updated the
cap files with cycle of 10sec.
Reading min and max value every 8 times should be enough
in this case.

Tested:
   Working as expected, no regression detected.

Signed-off-by: Zbigniew Kurzynski <zbigniew.kurzynski@intel.com>
Change-Id: I86c6b5542d3e2f3475aef3c5b05b305f87c6ef39

include/CPUSensor.hpp

diff --git a/include/CPUSensor.hpp b/include/CPUSensor.hpp
index 07f7ffd..a3045c5 100644
--- a/include/CPUSensor.hpp
+++ b/include/CPUSensor.hpp
@@ -47,6 +47,7 @@
     bool show;
     size_t pollTime;
     bool loggedInterfaceDown = false;
+    uint8_t minMaxReadCounter;
     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 127d68a..d68e980 100644
--- a/src/CPUSensor.cpp
+++ b/src/CPUSensor.cpp
@@ -47,7 +47,8 @@
            minReading, PowerState::on),
     objServer(objectServer), inputDev(io), waitTimer(io), path(path),
     privTcontrol(std::numeric_limits<double>::quiet_NaN()),
-    dtsOffset(dtsOffset), show(show), pollTime(CPUSensor::sensorPollMs)
+    dtsOffset(dtsOffset), show(show), pollTime(CPUSensor::sensorPollMs),
+    minMaxReadCounter(0)
 {
     nameTcontrol = labelTcontrol;
     nameTcontrol += " CPU" + std::to_string(cpuId);
@@ -235,7 +236,10 @@
             {
                 value = nvalue;
             }
-            updateMinMaxValues();
+            if (minMaxReadCounter++ % 8 == 0)
+            {
+                updateMinMaxValues();
+            }
 
             double gTcontrol = gCpuSensors[nameTcontrol]
                                    ? gCpuSensors[nameTcontrol]->value