diff options
author | Matt Spinler <spinler@us.ibm.com> | 2019-12-09 13:44:17 -0600 |
---|---|---|
committer | Matt Spinler <spinler@us.ibm.com> | 2019-12-17 17:28:40 +0000 |
commit | 6d51224bd46ae2fca3adfc32c94d14d31f3daed6 (patch) | |
tree | 4ad80b3c2aaf457debf1c4c08e93d6709cd052b9 /extensions | |
parent | 57a24e387dca1c8866bb11885f05b5af4f7cb9c0 (diff) | |
download | phosphor-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.cpp | 23 | ||||
-rw-r--r-- | extensions/openpower-pels/repository.hpp | 10 |
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&)>; /** |