From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id F1499740035 for ; Mon, 6 May 2024 02:44:07 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=GIq3xqapuTzHdHmZn0z8wBxZncQjrxYystj+atzHatc=; c=relaxed/simple; d=groups.io; h=From:To:Cc:References:In-Reply-To:Subject:Date:Message-ID:MIME-Version:Thread-Index:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Type:Content-Transfer-Encoding:Content-Language; s=20240206; t=1714963446; v=1; b=qviZ7iF75QzWU7vEFNQAcU/oScQgIZWJTKYDmBjKgt6uooTEqphvWtZWHy3P34oxZ0qT03hi o9dsHHcxzoDyiYYyIKJwn3kJyWV5GByLisPYpTrOXDGXYPvPzVHCSZdoQ+IYvcEP44dyPmNFwoh gF2X8Bzok7amP/YpAJXcz2a2CPwEIzXA8f4aL3IsYYzIXF9K68cJAtut64gRfWDvYiooxbKX2tq 9NX6JAifyS5xqLA60+Kj5rX5TbAsz/SL3CN2vFOM7J/1FWmArsX26KVO1Vgw/I92xK8IlBT49Fx GHDmkRPCfZDaqQQbJ0VzPwejEdt7MK3HhKX5bZjd68TgA== X-Received: by 127.0.0.2 with SMTP id EFcyYY7687511xUJynXcU3e2; Sun, 05 May 2024 19:44:06 -0700 X-Received: from zrleap.intel-email.com (zrleap.intel-email.com [114.80.218.36]) by mx.groups.io with SMTP id smtpd.web10.12264.1714963445019074230 for ; Sun, 05 May 2024 19:44:05 -0700 X-Received: from zrleap.intel-email.com (localhost [127.0.0.1]) by zrleap.intel-email.com (Postfix) with ESMTP id D0349A32E1B6 for ; Mon, 6 May 2024 10:44:02 +0800 (CST) X-Received: from localhost (localhost [127.0.0.1]) by zrleap.intel-email.com (Postfix) with ESMTP id 87656A32E19C for ; Mon, 6 May 2024 10:44:02 +0800 (CST) X-Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by zrleap.intel-email.com (Postfix) with SMTP id 4FDB0A32E155 for ; Mon, 6 May 2024 10:43:57 +0800 (CST) X-Received: from DESKTOPS6D0PVI ([39.149.62.91]) (envelope-sender ) by 192.168.6.13 with ESMTP(SSL) for ; Mon, 06 May 2024 10:43:45 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 39.149.62.91 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming via groups.io" To: "'Felix Polyudov'" , "'Sachin Ganesh'" , Cc: , , , , , , , , "'Dhanaraj V'" References: <01c301da958d$6e57dbf0$4b0793d0$@byosoft.com.cn> <022101da97bb$4a4511d0$decf3570$@byosoft.com.cn> In-Reply-To: Subject: =?UTF-8?B?W2VkazItZGV2ZWxdIOWbnuWkjTogW0VYVEVSTkFMXSDlm57lpI06IFtQQVRDSCAwLzddIEdlbmVyYWwgVXBkYXRlcyBiYXNlZCBvbiBVRUZJIDIuMTAgYW5kIFBJIDEuOCBTcGVjaWZpY2F0aW9u?= Date: Mon, 6 May 2024 10:43:45 +0800 Message-ID: <035801da9f5f$37cafb90$a760f2b0$@byosoft.com.cn> MIME-Version: 1.0 Thread-Index: AQJghfmseNM81mWw71qsppx8erC7ewGihDguAdmfEaACGtIj7gJgkR80AfL1E6OwLls6oA== Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Sun, 05 May 2024 19:44:05 -0700 Resent-From: gaoliming@byosoft.com.cn Reply-To: devel@edk2.groups.io,gaoliming@byosoft.com.cn List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: yJI0jkQoXhMkHUugLgtzYgSKx7686176AA= Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=qviZ7iF7; dmarc=pass (policy=none) header.from=groups.io; spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io Felix: I understand your point. I would suggest to directly add PI_SPECIFICATION_= VERSION definition, and update PEI/DXE/SMM version to align with it.=20 =20 Original PEI/DXE/SMM minor and major version definitions are still kept no= w.=20 Thanks Liming > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: Felix Polyudov > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B45=E6=9C=881=E6=97=A5 6= :35 > =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming ; Sachin= Ganesh > ; devel@edk2.groups.io > =E6=8A=84=E9=80=81: zhiguang.liu@intel.com; michael.d.kinney@intel.com; > ardb+tianocore@kernel.org; kraxel@redhat.com; jiewen.yao@intel.com; > erdemaktas@google.com; min.m.xu@intel.com; thomas.lendacky@amd.com; > Dhanaraj V > =E4=B8=BB=E9=A2=98: RE: [EXTERNAL] =E5=9B=9E=E5=A4=8D: [PATCH 0/7] Genera= l Updates based on UEFI > 2.10 and PI 1.8 Specification >=20 > Liming, >=20 > In PI 1.8 macros for major and minor versions have been replaced with a > single revision macro combining both version elements. > Refer to: > PEI: > https://uefi.org/specs/PI/1.8/V1_PEI_Services_Table.html#related-definiti= on > s > DXE: https://uefi.org/specs/PI/1.8/V2_UEFI_System_Table.html#dxe-services > MM: > https://uefi.org/specs/PI/1.8/V4_Management_Mode_System_Table.html#e > fi-mm-system-table >=20 > If patch is updated to literally follow the spec, the following macros wi= ll be > removed: > PEI_SPECIFICATION_MAJOR_REVISION > PEI_SPECIFICATION_MINOR_REVISION > DXE_SPECIFICATION_MAJOR_REVISION > DXE_SPECIFICATION_MINOR_REVISION > MM_SPECIFICATION_MAJOR_REVISION > MM_SPECIFICATION_MINOR_REVISION >=20 > These macros exist for a long time, so their removal can break some code = in > the downstream projects. > That's why we thought it would be safer to honor the spirit of the spec > without following it literally. >=20 > Let us know if you want us to update the patch to match the spec. >=20 > -----Original Message----- > From: gaoliming > Sent: Friday, April 26, 2024 5:23 AM > To: Sachin Ganesh ; Felix Polyudov > ; devel@edk2.groups.io > Cc: zhiguang.liu@intel.com; michael.d.kinney@intel.com; > ardb+tianocore@kernel.org; kraxel@redhat.com; jiewen.yao@intel.com; > erdemaktas@google.com; min.m.xu@intel.com; thomas.lendacky@amd.com; > Dhanaraj V > Subject: =E5=9B=9E=E5=A4=8D: [EXTERNAL] =E5=9B=9E=E5=A4=8D: [PATCH 0/7] G= eneral Updates based on > UEFI 2.10 and PI 1.8 Specification >=20 > Felix and Sachin: > I don't find PI_SPECIFICATION_MINOR_REVISION definition from the latest > public PI 1.8A. So, this change may not be added now. >=20 > Thanks > Liming > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > =E5=8F=91=E4=BB=B6=E4=BA=BA: Sachin Ganesh > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B44=E6=9C=8825=E6=97= =A5 17:57 > > =E6=94=B6=E4=BB=B6=E4=BA=BA: Felix Polyudov ; gaoliming > > ; devel@edk2.groups.io > > =E6=8A=84=E9=80=81: zhiguang.liu@intel.com; michael.d.kinney@intel.com; > > ardb+tianocore@kernel.org; kraxel@redhat.com; jiewen.yao@intel.com; > > erdemaktas@google.com; min.m.xu@intel.com; > thomas.lendacky@amd.com; > > Dhanaraj V > > =E4=B8=BB=E9=A2=98: RE: [EXTERNAL] =E5=9B=9E=E5=A4=8D: [PATCH 0/7] Gene= ral Updates based on UEFI > > 2.10 and PI 1.8 Specification > > > > Hi Felix/ Liming, > > > > Thank you for your comments. Patch 6 has been updated to only focus on > > consolidating the revision macros. > > > > Patch Link: > > > https://edk2.groups.io/g/devel/message/118246?p=3D%2C%2C%2C20%2C0%2 > 52 > > > C0%2C0%3A%3Arecentpostdate%2Fsticky%2C%2Csachin%2C20%2C2%2C0% > > 2C105721897 > > PR Link: https://github.com/tianocore/edk2/pull/5569 > > > > Thank you, > > Sachin. > > > > -----Original Message----- > > From: Felix Polyudov > > Sent: Tuesday, April 23, 2024 10:58 PM > > To: gaoliming ; Sachin Ganesh > > ; devel@edk2.groups.io > > Cc: zhiguang.liu@intel.com; michael.d.kinney@intel.com; > > ardb+tianocore@kernel.org; kraxel@redhat.com; jiewen.yao@intel.com; > > erdemaktas@google.com; min.m.xu@intel.com; > thomas.lendacky@amd.com; > > Dhanaraj V > > Subject: RE: [EXTERNAL] =E5=9B=9E=E5=A4=8D: [PATCH 0/7] General Updates= based on > UEFI > > 2.10 and PI 1.8 Specification > > > > I think patch 6 can be updated to introduce unified PI specification > versioning > > macros without incrementing the minor revision, by changing > > PI_SPECIFICATION_MINOR_REVISION back to 70. > > This will ensure compliance with the versioning schema introduced in PI > 1.7B > > without changing the PI support level. > > (the macros were introduced by PIWG mantis 2101) > > > > -----Original Message----- > > From: gaoliming > > Sent: Tuesday, April 23, 2024 10:49 AM > > To: Sachin Ganesh ; devel@edk2.groups.io > > Cc: zhiguang.liu@intel.com; michael.d.kinney@intel.com; > > ardb+tianocore@kernel.org; kraxel@redhat.com; jiewen.yao@intel.com; > > erdemaktas@google.com; min.m.xu@intel.com; > thomas.lendacky@amd.com; > > Felix Polyudov ; Dhanaraj V > > Subject: [EXTERNAL] =E5=9B=9E=E5=A4=8D: [PATCH 0/7] General Updates bas= ed on UEFI > 2.10 > > and PI 1.8 Specification > > > > > > **CAUTION: The e-mail below is from an external source. Please exercise > > caution before opening attachments, clicking links, or following guidan= ce.** > > > > Except for Patch 6/7, others are good to me. Reviewed-by: Liming Gao > > > > > > I suggest to merge others first. The patch 6/7 to update PI version fro= m 1.7 > to > > 1.8 can be discussed first. > > > > Thanks > > Liming > > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > > =E5=8F=91=E4=BB=B6=E4=BA=BA: Sachin Ganesh > > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B44=E6=9C=8820=E6=97= =A5 5:46 > > > =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io > > > =E6=8A=84=E9=80=81: gaoliming@byosoft.com.cn; zhiguang.liu@intel.com; > > > michael.d.kinney@intel.com; ardb+tianocore@kernel.org; > > > kraxel@redhat.com; jiewen.yao@intel.com; erdemaktas@google.com; > > > min.m.xu@intel.com; thomas.lendacky@amd.com; Felix Polyudov > > > ; Dhanaraj V ; Sachin Ganesh > > > > > > =E4=B8=BB=E9=A2=98: [PATCH 0/7] General Updates based on UEFI 2.10 an= d PI 1.8 > > > Specification > > > > > > This series of patches are for general updates to MdePkg and > > > MdeModulePkg based on UEFI 2.10 and PI 1.8 Specifications > > > > > > Sachin Ganesh (7): > > > MdePkg: Add definition for NVMe Over Fabric Device Path > > > MdePkg: Add new Resource Attributes defined in PI 1.8 Spec > > > MdePkg: Define Unaccepted Memory Type > > > MdeModulePkg: Use newly defined Unaccepted Memory Type > > > MdePkg: Update Delayed Dispatch PPI as per PI 1.8 Spec > > > MdePkg: Update to PI 1.8 Revision > > > OvmfPkg: Use newly defined Unaccepted Memory Type > > > > > > MdeModulePkg/Core/Dxe/Gcd/Gcd.c | 10 +++--- > > > MdeModulePkg/Core/Dxe/Mem/Page.c | 38 > > > ++++++++++---------- > > > MdeModulePkg/Include/Pi/PrePiDxeCis.h | 25 ------------- > > > MdeModulePkg/Include/Pi/PrePiHob.h | 20 ----------- > > > MdePkg/Include/Pi/PiDxeCis.h | 19 +++++----- > > > MdePkg/Include/Pi/PiHob.h | 14 +++----- > > > MdePkg/Include/Pi/PiMmCis.h | 6 ++-- > > > MdePkg/Include/Pi/PiMultiPhase.h | 6 ++++ > > > MdePkg/Include/Pi/PiPeiCis.h | 6 ++-- > > > MdePkg/Include/Pi/PiSmmCis.h | 2 +- > > > MdePkg/Include/Ppi/DelayedDispatch.h | 24 ++++++++++++- > > > MdePkg/Include/Protocol/DevicePath.h | 22 ++++++++++++ > > > OvmfPkg/AmdSevDxe/AmdSevDxe.c | 4 +-- > > > OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c | 8 ++--- > > > OvmfPkg/Library/PeilessStartupLib/Hob.c | 4 +-- > > > OvmfPkg/Library/PlatformInitLib/IntelTdx.c | 8 ++--- > > > OvmfPkg/PlatformPei/AmdSev.c | 4 +-- > > > 17 files changed, 108 insertions(+), 112 deletions(-) delete mode > > > 100644 MdeModulePkg/Include/Pi/PrePiDxeCis.h > > > delete mode 100644 MdeModulePkg/Include/Pi/PrePiHob.h > > > > > > -- > > > 2.24.1.windows.2 > > > -The information contained in this message may be confidential and > > > proprietary to American Megatrends (AMI). This communication is > > > intended > > to > > > be read only by the individual or entity to whom it is addressed or b= y > > their > > > designee. If the reader of this message is not the intended recipient= , > > > you > > are > > > on notice that any distribution of this message, in any form, is > > > strictly prohibited. Please promptly notify the sender by reply e-mai= l > > > or by > > telephone > > > at 770-246-8600, and then delete or destroy all copies of the > > transmission=3D > > > > > > -The information contained in this message may be confidential and > > proprietary to American Megatrends (AMI). This communication is intende= d > to > > be read only by the individual or entity to whom it is addressed or by = their > > designee. If the reader of this message is not the intended recipient, = you are > > on notice that any distribution of this message, in any form, is strict= ly > > prohibited. Please promptly notify the sender by reply e-mail or by > telephone > > at 770-246-8600, and then delete or destroy all copies of the transmiss= ion. >=20 >=20 > -The information contained in this message may be confidential and > proprietary to American Megatrends (AMI). This communication is intended = to > be read only by the individual or entity to whom it is addressed or by th= eir > designee. If the reader of this message is not the intended recipient, yo= u are > on notice that any distribution of this message, in any form, is strictly > prohibited. Please promptly notify the sender by reply e-mail or by telep= hone > at 770-246-8600, and then delete or destroy all copies of the transmissio= n. -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#118580): https://edk2.groups.io/g/devel/message/118580 Mute This Topic: https://groups.io/mt/105933432/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-