PEL: Remove PowerFault from message entries, registries, and code
The PowerFault flag is deprecated and should be removed in both the
registry and code references. Its functionality is superseded by the
critical_system severity level.
Signed-off-by: Mike Capps <mikepcapps@gmail.com>
Change-Id: I1bc1898d9589e0678b5a37b87289c6588859f2ee
diff --git a/extensions/openpower-pels/registry.cpp b/extensions/openpower-pels/registry.cpp
index b853bca..c8c37f9 100644
--- a/extensions/openpower-pels/registry.cpp
+++ b/extensions/openpower-pels/registry.cpp
@@ -675,11 +675,6 @@
entry.src.symptomID = helper::getSRCSymptomIDFields(src, name);
}
- if (src.contains("PowerFault"))
- {
- entry.src.powerFault = src["PowerFault"];
- }
-
auto& doc = (*e)["Documentation"];
entry.doc.message = doc["Message"];
entry.doc.description = doc["Description"];
diff --git a/extensions/openpower-pels/registry.hpp b/extensions/openpower-pels/registry.hpp
index bb26c70..c3b3bc0 100644
--- a/extensions/openpower-pels/registry.hpp
+++ b/extensions/openpower-pels/registry.hpp
@@ -82,11 +82,6 @@
uint16_t reasonCode;
/**
- * @brief Specifies if the SRC represents a power fault. Optional.
- */
- std::optional<bool> powerFault;
-
- /**
* @brief An optional vector of SRC hexword numbers that should be used
* along with the SRC ASCII string to build the Symptom ID, which
* is a field in the Extended Header section.
diff --git a/extensions/openpower-pels/registry/README.md b/extensions/openpower-pels/registry/README.md
index ac0c266..fa6b923 100644
--- a/extensions/openpower-pels/registry/README.md
+++ b/extensions/openpower-pels/registry/README.md
@@ -216,14 +216,6 @@
}
```
-### SRC Power Fault flag
-The SRC has a bit in it to indicate if the error is a power fault. This is an
-optional field in the message registry and defaults to false.
-
-```
-"PowerFault: false
-```
-
### Documentation Fields
The documentation fields are used by PEL parsers to display a human readable
description of a PEL. They are also the source for the Redfish event log
diff --git a/extensions/openpower-pels/registry/message_registry.json b/extensions/openpower-pels/registry/message_registry.json
index eac4f68..d4ce8eb 100644
--- a/extensions/openpower-pels/registry/message_registry.json
+++ b/extensions/openpower-pels/registry/message_registry.json
@@ -1174,8 +1174,7 @@
{
"Type": "11",
"ReasonCode": "0x00AC",
- "Words6To9": {},
- "PowerFault": true
+ "Words6To9": {}
},
"Callouts": [
@@ -1205,8 +1204,7 @@
"SRC": {
"Type": "11",
"ReasonCode": "0x2500",
- "Words6To9": {},
- "PowerFault": true
+ "Words6To9": {}
},
"Callouts": [
@@ -1260,8 +1258,7 @@
"SRC": {
"Type": "11",
"ReasonCode": "0x2602",
- "Words6To9": {},
- "PowerFault": true
+ "Words6To9": {}
},
"Callouts": [
@@ -2431,8 +2428,7 @@
"SRC": {
"Type": "11",
"ReasonCode": "0xA014",
- "Words6To9": {},
- "PowerFault": true
+ "Words6To9": {}
},
"Callouts": [
@@ -2777,8 +2773,7 @@
"SRC": {
"Type": "11",
"ReasonCode": "0x76F3",
- "Words6To9": {},
- "PowerFault": true
+ "Words6To9": {}
},
"Callouts": [
diff --git a/extensions/openpower-pels/registry/schema/schema.json b/extensions/openpower-pels/registry/schema/schema.json
index 6fa6cf4..7c03f88 100644
--- a/extensions/openpower-pels/registry/schema/schema.json
+++ b/extensions/openpower-pels/registry/schema/schema.json
@@ -104,9 +104,7 @@
"SymptomIDFields": {"$ref": "#/definitions/symptomID" },
- "Words6To9": {"$ref": "#/definitions/srcWords6To9" },
-
- "PowerFault": {"$ref": "#/definitions/powerFault" }
+ "Words6To9": {"$ref": "#/definitions/srcWords6To9" }
},
"required": ["ReasonCode", "Words6To9"],
@@ -320,12 +318,6 @@
"dump_notification", "env_normal"]
},
- "powerFault":
- {
- "description": "The Power Fault SRC field (bit 6 in byte 1 of header). Optional and defaults to false",
- "type": "boolean"
- },
-
"actionFlags":
{
"description": "The action flags Private Header PEL field",
diff --git a/extensions/openpower-pels/src.cpp b/extensions/openpower-pels/src.cpp
index f4aaf20..19d140d 100644
--- a/extensions/openpower-pels/src.cpp
+++ b/extensions/openpower-pels/src.cpp
@@ -326,13 +326,6 @@
_flags = 0;
- auto item = additionalData.getValue("POWER_THERMAL_CRITICAL_FAULT");
- if ((regEntry.src.powerFault.value_or(false)) ||
- (item.value_or("") == "TRUE"))
- {
- _flags |= powerFaultEvent;
- }
-
_reserved1B = 0;
_wordCount = numSRCHexDataWords + 1;
@@ -724,8 +717,6 @@
pv::boolString.at(_flags & i5OSServiceEventBit), 1);
jsonInsert(ps, "Hypervisor Dump Initiated",
pv::boolString.at(_flags & hypDumpInit), 1);
- jsonInsert(ps, "Power Control Net Fault",
- pv::boolString.at(isPowerFaultEvent()), 1);
if (isBMCSRC())
{