summaryrefslogtreecommitdiffstats
path: root/extensions
diff options
context:
space:
mode:
authorMatt Spinler <spinler@us.ibm.com>2019-12-09 13:44:17 -0600
committerMatt Spinler <spinler@us.ibm.com>2019-12-17 17:28:40 +0000
commit6d51224bd46ae2fca3adfc32c94d14d31f3daed6 (patch)
tree4ad80b3c2aaf457debf1c4c08e93d6709cd052b9 /extensions
parent57a24e387dca1c8866bb11885f05b5af4f7cb9c0 (diff)
downloadphosphor-logging-6d51224bd46ae2fca3adfc32c94d14d31f3daed6.tar.gz
phosphor-logging-6d51224bd46ae2fca3adfc32c94d14d31f3daed6.zip
PEL: Add repo function to get a PEL FD
Add Repository::getPELFD() to return a file descriptor to a PEL data file based on its ID. This will be used by a future D-Bus method to return the descriptor to the PLDM daemon. Signed-off-by: Matt Spinler <spinler@us.ibm.com> Change-Id: I1110a514e30a9387d9672e42862139b988717c53
Diffstat (limited to 'extensions')
-rw-r--r--extensions/openpower-pels/repository.cpp23
-rw-r--r--extensions/openpower-pels/repository.hpp10
2 files changed, 33 insertions, 0 deletions
diff --git a/extensions/openpower-pels/repository.cpp b/extensions/openpower-pels/repository.cpp
index f90a628..3bf3cb0 100644
--- a/extensions/openpower-pels/repository.cpp
+++ b/extensions/openpower-pels/repository.cpp
@@ -197,6 +197,29 @@ std::optional<std::vector<uint8_t>> Repository::getPELData(const LogID& id)
return std::nullopt;
}
+std::optional<sdbusplus::message::unix_fd> Repository::getPELFD(const LogID& id)
+{
+ auto pel = findPEL(id);
+ if (pel != _pelAttributes.end())
+ {
+ FILE* fp = fopen(pel->second.path.c_str(), "rb");
+
+ if (fp == nullptr)
+ {
+ auto e = errno;
+ log<level::ERR>("Unable to open PEL File", entry("ERRNO=%d", e),
+ entry("PATH=%s", pel->second.path.c_str()));
+ throw file_error::Open();
+ }
+
+ // Must leave the file open here. It will be closed by sdbusplus
+ // when it sends it back over D-Bus.
+
+ return fileno(fp);
+ }
+ return std::nullopt;
+}
+
void Repository::for_each(ForEachFunc func) const
{
for (const auto& [id, attributes] : _pelAttributes)
diff --git a/extensions/openpower-pels/repository.hpp b/extensions/openpower-pels/repository.hpp
index b6ac545..e55808f 100644
--- a/extensions/openpower-pels/repository.hpp
+++ b/extensions/openpower-pels/repository.hpp
@@ -169,6 +169,16 @@ class Repository
*/
std::optional<std::vector<uint8_t>> getPELData(const LogID& id);
+ /**
+ * @brief Get a file descriptor to the PEL data
+ *
+ * @param[in] id - The ID to get the FD for
+ *
+ * @return std::optional<sdbusplus::message::unix_fd> -
+ * The FD, or an empty optional object.
+ */
+ std::optional<sdbusplus::message::unix_fd> getPELFD(const LogID& id);
+
using ForEachFunc = std::function<bool(const PEL&)>;
/**
OpenPOWER on IntegriCloud