oem_ibm: Enhance PEL D-Bus call failure tracing
Add the PEL ID, the D-Bus error and the newline character to the code
that is handling D-Bus method call fails when dealing with PELs in order
to provide more information to help debug.
Show the PEL ID in hex as that is how other places display it.
Without the newline characters they were showing up as:
pldmd[523]: HostAck D-Bus call failedHostAck D-Bus call failed
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I3b00d974cf96c814ec257a7707c485f731033482
diff --git a/oem/ibm/libpldmresponder/file_io_type_pel.cpp b/oem/ibm/libpldmresponder/file_io_type_pel.cpp
index 00b8e99..6a40ebb 100644
--- a/oem/ibm/libpldmresponder/file_io_type_pel.cpp
+++ b/oem/ibm/libpldmresponder/file_io_type_pel.cpp
@@ -115,8 +115,8 @@
}
catch (const std::exception& e)
{
- std::cerr << "GetPEL D-Bus call failed, PEL id = " << fileHandle
- << ", error = " << e.what() << "\n";
+ std::cerr << "GetPEL D-Bus call failed, PEL id = 0x" << std::hex
+ << fileHandle << ", error = " << e.what() << "\n";
return PLDM_ERROR;
}
@@ -183,7 +183,8 @@
}
catch (const std::exception& e)
{
- std::cerr << "GetPEL D-Bus call failed";
+ std::cerr << "GetPEL D-Bus call failed on PEL ID 0x" << std::hex
+ << fileHandle << ", error = " << e.what() << "\n";
return PLDM_ERROR;
}
return PLDM_SUCCESS;
@@ -229,7 +230,8 @@
}
catch (const std::exception& e)
{
- std::cerr << "HostAck D-Bus call failed";
+ std::cerr << "HostAck D-Bus call failed on PEL ID 0x" << std::hex
+ << fileHandle << ", error = " << e.what() << "\n";
return PLDM_ERROR;
}