Re: [edk2-devel] [PATCH] MdeModulePkg: Correct memory type in PrePiDxeCis.h

["Ard Biesheuvel" <ardb@kernel.org>]

On Mon, 30 Jan 2023 at 17:27, Kinney, Michael D
<michael.d.kinney@intel.com> wrote:
>
> Hi Ard,
>
> Can you explain the process being followed here for content like this.
>
> The EDK II Code First Process as detailed below requires all content to only
> be added to edk2-staging repo and used from there before spec changes become
> public.  Content is only added to the edk2 repo until after the spec with the
> change is public.
>
> https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Code-First-Process
>
> This process was reviewed and approved by the community.  If there is a need
> for an alternate process, then that needs to be proposed, reviewed, and accepted.
>

Hello Mike,

I was not involved at all in the creation of these definitions. This
patch merely fixes the wrong index value that was introduced by

commit 502c01c5028038e4e6b4512e9c66be0ec4d11492
Author: Min M Xu <min.m.xu@intel.com>
Date:   Tue Nov 1 13:13:43 2022 +0800

    MdePkg: Add UEFI Unaccepted memory definition

    RFC: https://bugzilla.tianocore.org/show_bug.cgi?id=3937

    Plase refer to:
      UEFI Spec v2.9 Table 7-5 Memory Type Usage before ExitBootServices()

    Cc: Michael D Kinney <michael.d.kinney@intel.com>
    Cc: Liming Gao <gaoliming@byosoft.com.cn>
    Cc: Zhiguang Liu <zhiguang.liu@intel.com>
    Cc: Erdem Aktas <erdemaktas@google.com>
    Cc: Gerd Hoffmann <kraxel@redhat.com>
    Cc: James Bottomley <jejb@linux.ibm.com>
    Cc: Jiewen Yao <jiewen.yao@intel.com>
    Cc: Tom Lendacky <thomas.lendacky@amd.com>
    Reviewed-by: Jiewen Yao <jiewen.yao@intel.com>
    Acked-by: Gerd Hoffmann <kraxel@redhat.com>
    Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>
    Signed-off-by: Min Xu <min.m.xu@intel.com>