public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Kun Qin" <kuqin12@gmail.com>
To: devel@edk2.groups.io
Cc: Andrew Fish <afish@apple.com>, Leif Lindholm <leif@nuviainc.com>,
	Michael D Kinney <michael.d.kinney@intel.com>,
	Liming Gao <gaoliming@byosoft.com.cn>,
	Zhiguang Liu <zhiguang.liu@intel.com>
Subject: [PATCH v2 0/2] EDK2 Code First: PI Specification: Update EFI_MM_COMMUNICATE_HEADER
Date: Fri,  7 Jan 2022 17:32:40 -0800	[thread overview]
Message-ID: <20220108013242.2030-1-kuqin12@gmail.com> (raw)

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3794

This patch series is a rebase of previous submission:
https://edk2.groups.io/g/devel/message/85302

In current Status Codes definitions of PI spec v1.7 errata, there are a
few instances where the software could trigger system reboots while the
corresponding case were not covered by the already defined status codes.

One scenario that OEMs would be interested is that 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 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.

Compared to v1 series, v2 patch changes include:
a. Removed "RELEASE_ASSERT" definition;
b. Removed reference to memory type information;

Patch v2 branch: https://github.com/kuqin12/edk2/tree/BZ3794-expand_status_codes_v2

Cc: Andrew Fish <afish@apple.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>

Kun Qin (2):
  EDK2 Code First: PI Specification: New error codes of Host Software
    class
  MdePkg: MmCommunication: Add new Host Software class Error Code to
    MdePkg

 CodeFirst/BZ3794-SpecChange.md   | 55 ++++++++++++++++++++
 MdePkg/Include/Pi/PiStatusCode.h |  1 +
 2 files changed, 56 insertions(+)
 create mode 100644 CodeFirst/BZ3794-SpecChange.md

-- 
2.34.1.windows.1


             reply	other threads:[~2022-01-08  1:32 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-08  1:32 Kun Qin [this message]
2022-01-08  1:32 ` [PATCH v2 1/2] EDK2 Code First: PI Specification: New error codes of Host Software class Kun Qin
2022-01-08  1:32 ` [PATCH v2 2/2] MdePkg: MmCommunication: Add new Host Software class Error Code to MdePkg Kun Qin
     [not found] ` <16C82792D4F086F3.710@groups.io>
2022-01-18 19:00   ` [edk2-devel] [PATCH v2 1/2] EDK2 Code First: PI Specification: New error codes of Host Software class Kun Qin
2022-01-18 22:43     ` Michael D Kinney
2022-01-25 18:42       ` Kun Qin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220108013242.2030-1-kuqin12@gmail.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox