From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web11.40982.1624238566463515747 for ; Sun, 20 Jun 2021 18:22:48 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Mon, 21 Jun 2021 09:22:43 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , Cc: "'Michael D Kinney'" , "'Zhiguang Liu'" , "'Michael Kubacki'" References: <20210618094829.2651-1-kuqin12@gmail.com> <20210618094829.2651-2-kuqin12@gmail.com> In-Reply-To: <20210618094829.2651-2-kuqin12@gmail.com> Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIHYxIDEvMl0gTWRlUGtnOiBNbUNvbmZpZ3VyYXRpb246IE1vdmVkIEVGSV9NTV9SRVNFUlZFRF9NTVJBTV9SRUdJT04gdG8gUGlNbUNpcy5o?= Date: Mon, 21 Jun 2021 09:22:44 +0800 Message-ID: <007501d7663b$f03110d0$d0933270$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQHQww6ya9FZaIbG5mUiJRQFYgl4eQMSA6c5qxJhQuA= Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Kun: There is one header file edk2\MdePkg\Include\PiMultiPhase.h for this purpose. Can you place the common definition to it? Thanks Liming > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: devel@edk2.groups.io =B4=FA= =B1=ED Kun Qin > =B7=A2=CB=CD=CA=B1=BC=E4: 2021=C4=EA6=D4=C218=C8=D5 17:48 > =CA=D5=BC=FE=C8=CB: devel@edk2.groups.io > =B3=AD=CB=CD: Michael D Kinney ; Liming Gao > ; Zhiguang Liu ; > Michael Kubacki > =D6=F7=CC=E2: [edk2-devel] [PATCH v1 1/2] MdePkg: MmConfiguration: Moved > EFI_MM_RESERVED_MMRAM_REGION to PiMmCis.h >=20 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3440 >=20 > The definition of EFI_MM_RESERVED_MMRAM_REGION, according to PI Spec > 1.5 > is also referenced in EFI_PEI_MM_CONFIGURATION_PPI. Defining this > structure as is will enforce any potential usage of MM Configuration PPI > interface to include . >=20 > This change moves EFI_MM_RESERVED_MMRAM_REGION definition into > PiMmCis.h, > which is already included in Protocol/MmConfiguration.h. It also paves > way for introducing Ppi/MmConfiguration.h with proper dependency. >=20 > Cc: Michael D Kinney > Cc: Liming Gao > Cc: Zhiguang Liu > Cc: Michael Kubacki >=20 > Signed-off-by: Kun Qin > --- > MdePkg/Include/Pi/PiMmCis.h | 16 ++++++++++++++++ > MdePkg/Include/Protocol/MmConfiguration.h | 16 ---------------- > 2 files changed, 16 insertions(+), 16 deletions(-) >=20 > diff --git a/MdePkg/Include/Pi/PiMmCis.h b/MdePkg/Include/Pi/PiMmCis.h > index fdf0591a03d6..422a3ea6c2bb 100644 > --- a/MdePkg/Include/Pi/PiMmCis.h > +++ b/MdePkg/Include/Pi/PiMmCis.h > @@ -242,6 +242,22 @@ VOID > IN CONST EFI_MM_ENTRY_CONTEXT *MmEntryContext > ); >=20 > +/// > +/// Structure describing a MMRAM region which cannot be used for the > MMRAM heap. > +/// > +typedef struct _EFI_MM_RESERVED_MMRAM_REGION { > + /// > + /// Starting address of the reserved MMRAM area, as it appears while > MMRAM is open. > + /// Ignored if MmramReservedSize is 0. > + /// > + EFI_PHYSICAL_ADDRESS MmramReservedStart; > + /// > + /// Number of bytes occupied by the reserved MMRAM area. A size of > zero indicates the > + /// last MMRAM area. > + /// > + UINT64 MmramReservedSize; > +} EFI_MM_RESERVED_MMRAM_REGION; > + > /// > /// Management Mode System Table (MMST) > /// > diff --git a/MdePkg/Include/Protocol/MmConfiguration.h > b/MdePkg/Include/Protocol/MmConfiguration.h > index eeb94f64bdf7..d2fb6a13d4af 100644 > --- a/MdePkg/Include/Protocol/MmConfiguration.h > +++ b/MdePkg/Include/Protocol/MmConfiguration.h > @@ -21,22 +21,6 @@ > 0x26eeb3de, 0xb689, 0x492e, {0x80, 0xf0, 0xbe, 0x8b, 0xd7, 0xda, 0x4b, > 0xa7 } \ > } >=20 > -/// > -/// Structure describing a MMRAM region which cannot be used for the > MMRAM heap. > -/// > -typedef struct _EFI_MM_RESERVED_MMRAM_REGION { > - /// > - /// Starting address of the reserved MMRAM area, as it appears while > MMRAM is open. > - /// Ignored if MmramReservedSize is 0. > - /// > - EFI_PHYSICAL_ADDRESS MmramReservedStart; > - /// > - /// Number of bytes occupied by the reserved MMRAM area. A size of > zero indicates the > - /// last MMRAM area. > - /// > - UINT64 MmramReservedSize; > -} EFI_MM_RESERVED_MMRAM_REGION; > - > typedef struct _EFI_MM_CONFIGURATION_PROTOCOL > EFI_MM_CONFIGURATION_PROTOCOL; >=20 > /** > -- > 2.31.1.windows.1 >=20 >=20 >=20 >=20 >=20