commit ce75b511690b68822882ed04a232bb07ec7696ff
author Matt Spinler <spinler@us.ibm.com> Wed Jul 26 15:10:48 2017 -0500
committer Patrick Williams <patrick@stwcx.xyz> Sun Jul 30 11:31:39 2017 +0000
tree 26de70641e0b53b1d9b79457d70ac6d9ffbcc4d5
parent c5f5351d985cf9d5d7d19de4b0d9a7ec083a4f55

fan-monitor: Capture more data on failure

When a fan gets set to nonfunctional, it is useful
to know how fast the rotor was actually going and
what the expected speed was.

Change-Id: I760d6fa7d193038f9740d241bf4d4d0039020f64
Signed-off-by: Matt Spinler <spinler@us.ibm.com>

monitor/fan.cpp

diff --git a/monitor/fan.cpp b/monitor/fan.cpp
index 195317c..f3937c2 100644
--- a/monitor/fan.cpp
+++ b/monitor/fan.cpp
@@ -187,7 +187,10 @@
     if (_functional && tooManySensorsNonfunctional())
     {
         log<level::ERR>("Setting a fan to nonfunctional",
-                        entry("FAN=%s", _name.c_str()));
+                entry("FAN=%s", _name.c_str()),
+                entry("TACH_SENSOR=%s", sensor.name().c_str()),
+                entry("ACTUAL_SPEED=%lld", sensor.getInput()),
+                entry("TARGET_SPEED=%lld", getTargetSpeed(sensor)));
 
         updateInventory(false);
     }

monitor/tach_sensor.hpp

diff --git a/monitor/tach_sensor.hpp b/monitor/tach_sensor.hpp
index 84e02fe..d564f73 100644
--- a/monitor/tach_sensor.hpp
+++ b/monitor/tach_sensor.hpp
@@ -111,6 +111,14 @@
          */
         std::chrono::microseconds getTimeout();
 
+        /**
+         * Returns the sensor name
+         */
+        inline const std::string& name() const
+        {
+            return _name;
+        };
+
     private:
 
         /**