Hi Mike,
Do you have further feedback regarding this updated change? If not,
could you please shed some light on how to proceed on this code first PI
spec change request?
Any help is appreciated.
Regards,
Kun
On 01/07/2022 17:32, Kun Qin via groups.io wrote:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3794
This change includes specification update markdown file that describes
the proposed PI Specification v1.7 Errata A in detail and potential
impact to the existing codebase.
Cc: Andrew Fish <af...@apple.com>
Cc: Leif Lindholm <l...@nuviainc.com>
Cc: Michael D Kinney <michael.d.kin...@intel.com>
Cc: Liming Gao <gaolim...@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang....@intel.com>
Signed-off-by: Kun Qin <kuqi...@gmail.com>
---
Notes:
v2:
- Removed "RELEASE_ASSERT" definition
- Removed reference to EDK2 based memory type info [Mike]
CodeFirst/BZ3794-SpecChange.md | 55 ++++++++++++++++++++
1 file changed, 55 insertions(+)
diff --git a/CodeFirst/BZ3794-SpecChange.md b/CodeFirst/BZ3794-SpecChange.md
new file mode 100644
index 000000000000..61a0be798d7f
--- /dev/null
+++ b/CodeFirst/BZ3794-SpecChange.md
@@ -0,0 +1,55 @@
+# Title: Introduction of `EFI_SW_EC_FRAGMENTED_MEMORY_MAP` Status Code
+
+## Status: Draft
+
+## Document: UEFI Platform Initialization Specification Version 1.7 Errata A
+
+## License
+
+SPDX-License-Identifier: CC-BY-4.0
+
+## Submitter: [TianoCore Community](https://www.tianocore.org)
+
+## Summary of the change
+
+Add `EFI_SW_EC_FRAGMENTED_MEMORY_MAP` into Status Codes definition.
+
+## Benefits of the change
+
+Current Status Codes covered various [software class error code
definitions](https://github.com/tianocore/edk2/blob/master/MdePkg/Include/Pi/PiStatusCode.h).
+
+However, fragmented memory map from boot to boot would fail to meet certain OS
ACPI requirements (i.e. S4 resume boot requires consistent memory maps) and
trigger system reboots. Yet the corresponding case was not covered by the
already defined status codes.
+
+The unexpected system reboots above could indicate decay of system health and
reporting of such generic events would provide helpful information to OEMs to
investigate/prevent system failures in general.
+
+The request of this change intends to expand definitions of `EFI_SW_EC_**`
under Status Codes to cover more unexpected system reboot events, which could
improve Status Code futility and readability.
+
+## Impact of the change
+
+Occupy a new macro definitions of Error Codes under Software class Status
Codes.
+
+## Detailed description of the change [normative updates]
+
+### Specification Changes
+
+1. In PI Specification v1.7 Errata A: Vol. 3, Table 3-61: Error Code
Operations: Host Software Class, add one new rows below
`EFI_SW_EC_FV_CORRUPTED` definition:
+
+ | Operation | Description | Extended Data |
+ | --- | --- | --- |
+ | EFI_SW_EC_FRAGMENTED_MEMORY_MAP | System will reboot due to fragmented
memory maps | None |
+
+1. In PI Specification v1.7 Errata A: Vol. 3, Table 3-61: Error Code
Operations: Host Software Class, replace the row of `0x0014-0x00FF` to:
+
+ | Operation | Description | Extended Data |
+ | --- | --- | --- |
+ | 0x0015-0x00FF | Reserved for future use by this specification for Host
Software class error codes. | None |
+
+1. In PI Specification v1.7 Errata A: Vol. 3, Section 6.7.4.3 Error Code
Definitions: Prototype, add one new definitions below `EFI_SW_EC_FV_CORRUPTED`
definition:
+
+ ```c
+ #define EFI_SW_EC_FRAGMENTED_MEMORY_MAP 0x00000014
+ ```
+
+### Code Changes
+
+1. Add macro definitions in `MdePkg/Include/Pi/PiStatusCode.h` to match new
specification.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85785): https://edk2.groups.io/g/devel/message/85785
Mute This Topic: https://groups.io/mt/88516885/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
