commit c1778dac509d6d7ae09214a2cca06c7af8b89e01
author Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com> Fri Feb 04 01:46:49 2022 -0600
committer Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com> Wed Oct 12 05:40:46 2022 -0500
tree a5b6a8efc98b05831122d2eb07ee9daea2bc303d
parent b5a7547f78cc34b7f29d64fe57d55f4009134cb6

OpenPOWER: Add additional trace to host dump operations

Adding additional traces to dump offload and
delete operations

Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
Change-Id: I5f5ae40798483b4d4ef52fd518d4d8ffab6e15c4

dump-extensions/openpower-dumps/resource_dump_entry.cpp

diff --git a/dump-extensions/openpower-dumps/resource_dump_entry.cpp b/dump-extensions/openpower-dumps/resource_dump_entry.cpp
index 0f45bc2..ac745b5 100644
--- a/dump-extensions/openpower-dumps/resource_dump_entry.cpp
+++ b/dump-extensions/openpower-dumps/resource_dump_entry.cpp
@@ -26,6 +26,11 @@
     using NotAllowed =
         sdbusplus::xyz::openbmc_project::Common::Error::NotAllowed;
     using Reason = xyz::openbmc_project::Common::NotAllowed::REASON;
+    log<level::INFO>(
+        fmt::format(
+            "Resource dump offload request id({}) uri({}) source dumpid()", id,
+            uri, sourceDumpId())
+            .c_str());
 
     if (!phosphor::dump::isHostRunning())
     {

dump-extensions/openpower-dumps/system_dump_entry.cpp

diff --git a/dump-extensions/openpower-dumps/system_dump_entry.cpp b/dump-extensions/openpower-dumps/system_dump_entry.cpp
index 3d2e734..fe313d4 100644
--- a/dump-extensions/openpower-dumps/system_dump_entry.cpp
+++ b/dump-extensions/openpower-dumps/system_dump_entry.cpp
@@ -4,6 +4,8 @@
 #include "host_transport_exts.hpp"
 #include "op_dump_consts.hpp"
 
+#include <fmt/core.h>
+
 #include <phosphor-logging/elog-errors.hpp>
 #include <xyz/openbmc_project/Common/error.hpp>
 
@@ -21,6 +23,11 @@
 
 void Entry::initiateOffload(std::string uri)
 {
+    log<level::INFO>(
+        fmt::format(
+            "System dump offload request id({}) uri({}) source dumpid()", id,
+            uri, sourceDumpId())
+            .c_str());
     phosphor::dump::Entry::initiateOffload(uri);
     phosphor::dump::host::requestOffload(sourceDumpId());
 }
@@ -28,6 +35,10 @@
 void Entry::delete_()
 {
     auto srcDumpID = sourceDumpId();
+    auto dumpId = id;
+    log<level::INFO>(fmt::format("System dump delete id({}) srcdumpid({})",
+                                 dumpId, srcDumpID)
+                         .c_str());
 
     // Remove Dump entry D-bus object
     phosphor::dump::Entry::delete_();