commit f65c406096e3e68fde6b0171283e5a0f51c9753d
author Brandon Wyman <bjwyman@gmail.com> Wed Aug 19 13:15:53 2020 -0500
committer Brandon Wyman <bjwyman@gmail.com> Tue Oct 13 17:49:20 2020 -0500
tree 057c987b91469d8047a1a25b9e67d58e84c5c1d7
parent 52e54e875dcd7a76368bb4a70b5e24ff7fcdbcb8

psu-ng: Limit tracing/logging read failures

Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: I433f164b82e9c5e6529d197354f4ab1169417540

phosphor-power-supply/power_supply.cpp

diff --git a/phosphor-power-supply/power_supply.cpp b/phosphor-power-supply/power_supply.cpp
index ab1caf9..77d533a 100644
--- a/phosphor-power-supply/power_supply.cpp
+++ b/phosphor-power-supply/power_supply.cpp
@@ -35,11 +35,13 @@
 {
     using namespace phosphor::pmbus;
 
-    if (present)
+    if ((present) && (readFail < LOG_LIMIT))
     {
         try
         {
             statusWord = pmbusIntf->read(STATUS_WORD, Type::Debug);
+            // Read worked, reset the fail count.
+            readFail = 0;
 
             if (statusWord)
             {
@@ -94,6 +96,7 @@
         }
         catch (ReadFailure& e)
         {
+            readFail++;
             phosphor::logging::commit<ReadFailure>();
         }
     }
@@ -129,6 +132,7 @@
     inputFault = false;
     mfrFault = false;
     vinUVFault = false;
+    readFail = 0;
     faultLogged = false;
 
     // The PMBus device driver does not allow for writing CLEAR_FAULTS