[edk2-devel] [PATCH v10 2/4] MdePkg: Introduce the SevMemoryAcceptance protocol
Yao, Jiewen
jiewen.yao at intel.com
Thu Jan 26 01:24:05 UTC 2023
I don’t think MdePkg is the right place. MdePkg only includes the approved industry standard.
As Gerd and I discussed before, this protocol should be in OvmfPkg.
Please move to https://github.com/tianocore/edk2/tree/master/OvmfPkg/Include/Protocol
Thank you
Yao, Jiewen
> -----Original Message-----
> From: Dionna Glaze <dionnaglaze at google.com>
> Sent: Thursday, January 26, 2023 8:57 AM
> To: devel at edk2.groups.io
> Cc: Dionna Glaze <dionnaglaze at google.com>; Gerd Hoffmann
> <kraxel at redhat.com>; James Bottomley <jejb at linux.ibm.com>; Yao, Jiewen
> <jiewen.yao at intel.com>; Tom Lendacky <thomas.lendacky at amd.com>; Ard
> Biesheuvel <ardb at kernel.org>; Xu, Min M <min.m.xu at intel.com>; Andrew Fish
> <afish at apple.com>; Kinney, Michael D <michael.d.kinney at intel.com>
> Subject: [PATCH v10 2/4] MdePkg: Introduce the SevMemoryAcceptance
> protocol
>
> The default behavior for unaccepted memory in SEV-SNP is to accept all
> memory when ExitBootServices is called. An OS loader can use this
> protocol to disable this behavior to assume responsibility for memory
> acceptance and to affirm that the OS can handle the unaccepted memory
> type.
>
> This is a candidate for standardization.
>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: James Bottomley <jejb at linux.ibm.com>
> Cc: Jiewen Yao <jiewen.yao at intel.com>
> Cc: Tom Lendacky <thomas.lendacky at amd.com>
> Cc: Ard Biesheuvel <ardb at kernel.org>
> Cc: "Min M. Xu" <min.m.xu at intel.com>
> Cc: Andrew Fish <afish at apple.com>
> Cc: "Michael D. Kinney" <michael.d.kinney at intel.com>
>
> Signed-off-by: Dionna Glaze <dionnaglaze at google.com>
> ---
> MdePkg/Include/Protocol/Bz3987SevMemoryAcceptance.h | 44
> ++++++++++++++++++++
> MdePkg/MdePkg.dec | 3 ++
> 2 files changed, 47 insertions(+)
>
> diff --git a/MdePkg/Include/Protocol/Bz3987SevMemoryAcceptance.h
> b/MdePkg/Include/Protocol/Bz3987SevMemoryAcceptance.h
> new file mode 100644
> index 0000000000..c3691e1c93
> --- /dev/null
> +++ b/MdePkg/Include/Protocol/Bz3987SevMemoryAcceptance.h
> @@ -0,0 +1,44 @@
> +/** @file
> + The file provides the protocol that disables the behavior that all memory
> + gets accepted at ExitBootServices(). This protocol is only meant to be called
> + by the OS loader, and not EDK2 itself. The SEV naming is due to the
> coincidence
> + that only SEV-SNP needs this protocol, since SEV-SNP kernel support released
> + before kernel support for unaccepted memory. The technology enablement
> thus
> + does not strictly imply support for the unaccepted memory type.
> +
> + Copyright (c) 2023, Google LLC. All rights reserved.<BR>
> + SPDX-License-Identifier: BSD-2-Clause-Patent
> +**/
> +
> +#ifndef SEV_MEMORY_ACCEPTANCE_H_
> +#define SEV_MEMORY_ACCEPTANCE_H_
> +
> +#define BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL_GUID \
> + {0xc5a010fe, \
> + 0x38a7, \
> + 0x4531, \
> + {0x8a, 0x4a, 0x05, 0x00, 0xd2, 0xfd, 0x16, 0x49}}
> +
> +typedef struct _BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL \
> + BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL;
> +
> +/**
> + @param This A pointer to a BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL.
> +**/
> +typedef
> + EFI_STATUS
> +(EFIAPI *BZ3987_SEV_ALLOW_UNACCEPTED_MEMORY)(
> + IN BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL *This
> + );
> +
> +///
> +/// The BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL allows the OS loader
> to
> +/// indicate to EDK2 that ExitBootServices should not accept all memory.
> +///
> +struct _BZ3987_SEV_MEMORY_ACCEPTANCE_PROTOCOL {
> + BZ3987_SEV_ALLOW_UNACCEPTED_MEMORY AllowUnacceptedMemory;
> +};
> +
> +extern EFI_GUID gBz3987SevMemoryAcceptanceProtocolGuid;
> +
> +#endif
> diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec
> index 3d08f20d15..b82d6e46a4 100644
> --- a/MdePkg/MdePkg.dec
> +++ b/MdePkg/MdePkg.dec
> @@ -1031,6 +1031,9 @@
> gEfiPeiDelayedDispatchPpiGuid = { 0x869c711d, 0x649c, 0x44fe, { 0x8b, 0x9e,
> 0x2c, 0xbb, 0x29, 0x11, 0xc3, 0xe6 }}
>
> [Protocols]
> + ## Include/Protocol/Bz3987SevMemoryAcceptance.h
> + gBz3987SevMemoryAcceptanceProtocolGuid = { 0xc5a010fe, 0x38a7, 0x4531,
> {0x8a, 0x4a, 0x05, 0x00, 0xd2, 0xfd, 0x16, 0x49 }}
> +
> ## Include/Protocol/MemoryAccept.h
> gEdkiiMemoryAcceptProtocolGuid = { 0x38c74800, 0x5590, 0x4db4, { 0xa0,
> 0xf3, 0x67, 0x5d, 0x9b, 0x8e, 0x80, 0x26 }}
>
> --
> 2.39.1.456.gfc5497dd1b-goog
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#99054): https://edk2.groups.io/g/devel/message/99054
Mute This Topic: https://groups.io/mt/96534753/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list