commit 5f5352e5f6b0ebcdd5bf0a7306f13c503246c9b9
author Matt Spinler <spinler@us.ibm.com> Thu Mar 05 16:23:27 2020 -0600
committer Matt Spinler <spinler@us.ibm.com> Tue Mar 10 21:40:57 2020 +0000
tree 8bf094016aaf020b91ffde91295e835d17df95bc
parent 7cb985ff41b58e983a186cfc6a3eb1dd1c4b510e

PEL: Add HostNotifier related debug traces

Add some debug traces that make it easier to debug issues in the code
paths that send PELs to the host.  They will only show up when the
journald conf files are modified to turn on debug journal entries.

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

extensions/openpower-pels/manager.cpp

diff --git a/extensions/openpower-pels/manager.cpp b/extensions/openpower-pels/manager.cpp
index 4eb6772..2e6a07e 100644
--- a/extensions/openpower-pels/manager.cpp
+++ b/extensions/openpower-pels/manager.cpp
@@ -112,6 +112,9 @@
 
         try
         {
+            log<level::DEBUG>("Adding external PEL to repo",
+                              entry("PEL_ID=0x%X", pel->id()));
+
             _repo.add(pel);
         }
         catch (std::exception& e)
@@ -149,6 +152,9 @@
 {
     std::vector<uint8_t> data;
 
+    log<level::DEBUG>("Adding PEL from ESEL",
+                      entry("OBMC_LOG_ID=%d", obmcLogID));
+
     try
     {
         data = std::move(eselToRawData(esel));
@@ -256,6 +262,8 @@
     Repository::LogID id{Repository::LogID::Pel(pelID)};
     std::optional<int> fd;
 
+    log<level::DEBUG>("getPEL", entry("PEL_ID=0x%X", pelID));
+
     try
     {
         fd = _repo.getPELFD(id);
@@ -294,6 +302,8 @@
     Repository::LogID id{Repository::LogID::Obmc(obmcLogID)};
     std::optional<std::vector<uint8_t>> data;
 
+    log<level::DEBUG>("getPELFromOBMCID", entry("OBMC_LOG_ID=%d", obmcLogID));
+
     try
     {
         data = _repo.getPELData(id);
@@ -315,6 +325,8 @@
 {
     Repository::LogID id{Repository::LogID::Pel(pelID)};
 
+    log<level::DEBUG>("HostAck", entry("PEL_ID=0x%X", pelID));
+
     if (!_repo.hasPEL(id))
     {
         throw common_error::InvalidArgument();
@@ -330,6 +342,9 @@
 {
     Repository::LogID id{Repository::LogID::Pel(pelID)};
 
+    log<level::DEBUG>("HostReject", entry("PEL_ID=0x%X", pelID),
+                      entry("REASON=%d", static_cast<int>(reason)));
+
     if (!_repo.hasPEL(id))
     {
         throw common_error::InvalidArgument();