PLDM: Implementing Phosphor-Logging/LG2 logging
This commit adds changes in PLDM for implementing
structured LG2 logging, thereby moving away from
std::cout/cerr practice of logging which are
output streams and not logging mechanism.
PLDM now can make use of lg2 features like accurate
CODE LINE Number and CODE_FUNCTION Name and better
detailing in json object values which can be used in
log tracking.
More detailed logging change:
https://gist.github.com/riyadixitagra/c251685c1ba84248181891f7bc282395
Tested:
Ran a power off, on, cycle, and reset-reload.
Change-Id: I0485035f15f278c3fd172f0581b053c1c37f3a5b
Signed-off-by: Riya Dixit <riyadixitagra@gmail.com>
diff --git a/libpldmresponder/bios_config.hpp b/libpldmresponder/bios_config.hpp
index 4d0638a..4e69089 100644
--- a/libpldmresponder/bios_config.hpp
+++ b/libpldmresponder/bios_config.hpp
@@ -8,6 +8,7 @@
#include <libpldm/bios_table.h>
#include <nlohmann/json.hpp>
+#include <phosphor-logging/lg2.hpp>
#include <functional>
#include <iostream>
@@ -17,6 +18,8 @@
#include <string>
#include <vector>
+PHOSPHOR_LOG2_USING;
+
namespace pldm
{
namespace responder
@@ -224,8 +227,8 @@
}
catch (const std::exception& e)
{
- std::cerr << "Constructs Attribute Error, " << e.what()
- << std::endl;
+ error("Constructs Attribute Error, {ERR_EXCEP}", "ERR_EXCEP",
+ e.what());
}
}