PEL: Add hidden and subsystem properties
This story adds a new PELEntry interface
org.open_power.Logging.PEL.Entry on existing PEL entry. Then we add 2
properties with their values initialized during the creation of the Interface.
Tested:
1. Verified that the new PEL creation with recoverable error sets the new
attribute in the PEL.entry Interface.
2. Restarted the phosphor-log-manager daemon and made sure the flag is set
with the new Interface created for d-bus
3. Tested with passing a RAWPEL to create from file system and verified
that the hidden attribute is set accordingly on the PEL.entry Interface.
4. Copied the new format error to a system with old code and restored
the error log to make sure it does not crash.
5. Validated both hidden and subsystem properties.
Signed-off-by: Vijay Lobo <vijaylobo@gmail.com>
Change-Id: Ida287ed84a4a3f9ddd054cde37d752219ffb1882
diff --git a/extensions/openpower-pels/repository.hpp b/extensions/openpower-pels/repository.hpp
index b8ef126..b2fc797 100644
--- a/extensions/openpower-pels/repository.hpp
+++ b/extensions/openpower-pels/repository.hpp
@@ -29,6 +29,7 @@
std::filesystem::path path;
size_t sizeOnDisk;
uint8_t creator;
+ uint8_t subsystem;
uint8_t severity;
std::bitset<16> actionFlags;
TransmissionState hostState;
@@ -37,11 +38,13 @@
PELAttributes() = delete;
PELAttributes(const std::filesystem::path& p, size_t size,
- uint8_t creator, uint8_t sev, uint16_t flags,
- TransmissionState hostState, TransmissionState hmcState) :
+ uint8_t creator, uint8_t subsystem, uint8_t sev,
+ uint16_t flags, TransmissionState hostState,
+ TransmissionState hmcState) :
path(p),
- sizeOnDisk(size), creator(creator), severity(sev),
- actionFlags(flags), hostState(hostState), hmcState(hmcState)
+ sizeOnDisk(size), creator(creator), subsystem(subsystem),
+ severity(sev), actionFlags(flags), hostState(hostState),
+ hmcState(hmcState)
{
}
};