Enhancements to default console output

Added the following pieces of extra information to the default
console output:

-- Attribute Overrides --
  3.49165|**Found 24 attribute overrides in Tank FAPI(1)
  3.49169|- PEC:n0:p0:c0
  3.49171|  ATTR 0EB604DF [1] = 00
  3.49174|  ATTR 085F1C7E [1] = 00
  3.49177|  ATTR 048A8902 [1] = 01
  3.49180|  ATTR 04D8DF8D [2] = 0000
  3.49182|- PEC:n0:p0:c1
  3.49185|  ATTR 0EB604DF [1] = 00
  3.49188|  ATTR 085F1C7E [1] = 00
  3.49191|  ATTR 048A8902 [1] = 03

-- New part detection --
  4.97449|Detected new part : 00030002 (Physical:/Sys0/Node0/DIMM2)

-- Applying GARD records --
  8.78767|HWAS|Applying GARD record for HUID=0x000E0000 (Physical:/Sys0/Node0/Pr
oc0/XBUS0) due to 0x90000135
  8.78912|HWAS|Deconfig HUID 0x000E0000, Physical:/Sys0/Node0/Proc0/XBUS0

Change-Id: Ic2ae04515453dbb49e359abff952c87c83e23b72
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88388
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
Reviewed-by: Roland Veloz <rveloz@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>

1 files changed, 21 insertions, 2 deletions

diff --git a/src/usr/hwas/common/deconfigGard.C b/src/usr/hwas/common/deconfigGard.C
index 220bfa101..89df5b160 100644
--- a/src/usr/hwas/common/deconfigGard.C
+++ b/src/usr/hwas/common/deconfigGard.C
@@ -173,6 +173,17 @@ errlHndl_t DeconfigGard::applyGardRecord(Target *i_pTarget,
         // all ok - do the work
         HWAS_MUTEX_LOCK(iv_mutex);
 
+#if (!defined(CONFIG_CONSOLE_OUTPUT_TRACE) && defined(CONFIG_CONSOLE))
+        const char* l_tmpstring =
+          i_pTarget->getAttr<TARGETING::ATTR_PHYS_PATH>().toString();
+        CONSOLE::displayf("HWAS", "Applying GARD record for HUID=0x%08X (%s) due to 0x%.8X",
+                    get_huid(i_pTarget),
+                    l_tmpstring,
+                    l_errlogEid);
+        free((void*)(l_tmpstring));
+        l_tmpstring = nullptr;
+#endif
+
         // Deconfigure the Target
         // don't need to check ATTR_DECONFIG_GARDABLE -- if we get
         //  here, it's because of a gard record on this target
@@ -642,9 +653,13 @@ errlHndl_t DeconfigGard::deconfigureTargetsFromGardRecordsForIpl(
             }
 
 #if (!defined(CONFIG_CONSOLE_OUTPUT_TRACE) && defined(CONFIG_CONSOLE))
+            const char* l_tmpstring =
+              l_pTarget->getAttr<TARGETING::ATTR_PHYS_PATH>().toString();
             CONSOLE::displayf("HWAS", "Deconfig HUID 0x%08X, %s",
                     get_huid(l_pTarget),
-                    l_pTarget->getAttr<TARGETING::ATTR_PHYS_PATH>().toString());
+                    l_tmpstring);
+            free((void*)(l_tmpstring));
+            l_tmpstring = nullptr;
 #endif
         } // for
 
@@ -911,9 +926,13 @@ errlHndl_t DeconfigGard::deconfigureTargetsFromGardRecordsForIpl(
                 }
 
 #if (!defined(CONFIG_CONSOLE_OUTPUT_TRACE) && defined(CONFIG_CONSOLE))
+                const char* l_tmpstring =
+                  l_pTarget->getAttr<TARGETING::ATTR_PHYS_PATH>().toString();
                 CONSOLE::displayf("HWAS", "Deconfig HUID 0x%08X, %s",
                     get_huid(l_pTarget),
-                    l_pTarget->getAttr<TARGETING::ATTR_PHYS_PATH>().toString());
+                    l_tmpstring);
+                free((void*)(l_tmpstring));
+                l_tmpstring = nullptr;
 #endif
 
                 l_specDeconfigVector.erase(l_specDeconfigVector.begin());