PEL: Create object for every section
When unflattening a PEL, create objects for every PEL section in the
log. It will use a factory method to choose which object type to create
based on the section ID in the section header. All of these object will
go into a vector of Section objects, which is the base class for every
PEL section class.
The factory will default to creating a Generic object when it doesn't
have any other type to create.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: Ief0e4df5c586a46cea66ca47b4479e3444815309
diff --git a/extensions/openpower-pels/openpower-pels.mk b/extensions/openpower-pels/openpower-pels.mk
index f1603b6..cca1228 100644
--- a/extensions/openpower-pels/openpower-pels.mk
+++ b/extensions/openpower-pels/openpower-pels.mk
@@ -13,5 +13,6 @@
extensions/openpower-pels/private_header.cpp \
extensions/openpower-pels/registry.cpp \
extensions/openpower-pels/repository.cpp \
+ extensions/openpower-pels/section_factory.cpp \
extensions/openpower-pels/severity.cpp \
extensions/openpower-pels/user_header.cpp
diff --git a/extensions/openpower-pels/pel.cpp b/extensions/openpower-pels/pel.cpp
index 6a56e15..c7df5e8 100644
--- a/extensions/openpower-pels/pel.cpp
+++ b/extensions/openpower-pels/pel.cpp
@@ -2,6 +2,7 @@
#include "bcd_time.hpp"
#include "log_id.hpp"
+#include "section_factory.hpp"
#include "stream.hpp"
namespace openpower
@@ -42,6 +43,13 @@
}
_uh = std::make_unique<UserHeader>(pelData);
+
+ // Use the section factory to create the rest of the objects
+ for (size_t i = 2; i < _ph->sectionCount(); i++)
+ {
+ auto section = section_factory::create(pelData);
+ _optionalSections.push_back(std::move(section));
+ }
}
bool PEL::valid() const
@@ -53,6 +61,15 @@
valid = _uh->valid();
}
+ if (valid)
+ {
+ if (!std::all_of(_optionalSections.begin(), _optionalSections.end(),
+ [](const auto& section) { return section->valid(); }))
+ {
+ valid = false;
+ }
+ }
+
return valid;
}
diff --git a/extensions/openpower-pels/pel.hpp b/extensions/openpower-pels/pel.hpp
index ff45acb..29a7978 100644
--- a/extensions/openpower-pels/pel.hpp
+++ b/extensions/openpower-pels/pel.hpp
@@ -163,6 +163,17 @@
}
/**
+ * @brief Returns the optional sections, which is everything but
+ * the Private and User Headers.
+ *
+ * @return const std::vector<std::unique_ptr<Section>>&
+ */
+ const std::vector<std::unique_ptr<Section>>& optionalSections() const
+ {
+ return _optionalSections;
+ }
+
+ /**
* @brief Returns the PEL data.
*
* @return std::vector<uint8_t> - the raw PEL data
@@ -225,6 +236,11 @@
* @brief If the PEL came from a flattened data stream.
*/
bool _fromStream = false;
+
+ /**
+ * @brief Holds all sections by the PH and UH.
+ */
+ std::vector<std::unique_ptr<Section>> _optionalSections;
};
} // namespace pels
diff --git a/extensions/openpower-pels/section_factory.cpp b/extensions/openpower-pels/section_factory.cpp
new file mode 100644
index 0000000..69274c0
--- /dev/null
+++ b/extensions/openpower-pels/section_factory.cpp
@@ -0,0 +1,52 @@
+#include "section_factory.hpp"
+
+#include "failing_mtms.hpp"
+#include "generic.hpp"
+#include "pel_types.hpp"
+#include "private_header.hpp"
+#include "user_header.hpp"
+
+namespace openpower
+{
+namespace pels
+{
+namespace section_factory
+{
+std::unique_ptr<Section> create(Stream& pelData)
+{
+ std::unique_ptr<Section> section;
+
+ // Peek the section ID to create the appriopriate object.
+ // If not enough data remains to do so, an invalid
+ // Generic object will be created in the default case.
+ uint16_t sectionID = 0;
+
+ if (pelData.remaining() >= 2)
+ {
+ pelData >> sectionID;
+ pelData.offset(pelData.offset() - 2);
+ }
+
+ switch (sectionID)
+ {
+ case static_cast<uint16_t>(SectionID::privateHeader):
+ section = std::make_unique<PrivateHeader>(pelData);
+ break;
+ case static_cast<uint16_t>(SectionID::userHeader):
+ section = std::make_unique<UserHeader>(pelData);
+ break;
+ case static_cast<uint16_t>(SectionID::failingMTMS):
+ section = std::make_unique<FailingMTMS>(pelData);
+ break;
+ default:
+ // A generic object, but at least an object.
+ section = std::make_unique<Generic>(pelData);
+ break;
+ }
+
+ return section;
+}
+
+} // namespace section_factory
+} // namespace pels
+} // namespace openpower
diff --git a/extensions/openpower-pels/section_factory.hpp b/extensions/openpower-pels/section_factory.hpp
new file mode 100644
index 0000000..8807eba
--- /dev/null
+++ b/extensions/openpower-pels/section_factory.hpp
@@ -0,0 +1,30 @@
+#pragma once
+
+#include "section.hpp"
+#include "stream.hpp"
+
+namespace openpower
+{
+namespace pels
+{
+namespace section_factory
+{
+
+/**
+ * @brief Create a PEL section based on its data
+ *
+ * This creates the appropriate PEL section object based on the section ID in
+ * the first 2 bytes of the stream, but returns the base class Section pointer.
+ *
+ * If there isn't a class specifically for that section, it defaults to
+ * creating an instance of the 'Generic' class.
+ *
+ * @param[in] pelData - The PEL data stream
+ *
+ * @return std::unique_ptr<Section> - class of the appropriate type
+ */
+std::unique_ptr<Section> create(Stream& pelData);
+
+} // namespace section_factory
+} // namespace pels
+} // namespace openpower