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.web08.3397.1612317698315245558 for ; Tue, 02 Feb 2021 18:01:39 -0800 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 ; Wed, 03 Feb 2021 10:01:24 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: "'Sheng, W'" , "'Kinney, Michael D'" , Cc: "'Liu, Zhiguang'" , "'Yao, Jiewen'" , "'Andrew Fish'" , "'Laszlo Ersek'" , "'Feng, Roger'" , "'Leif Lindholm'" References: <20210129023448.30348-1-w.sheng@intel.com> <000a01d6f90d$3718edc0$a54ac940$@byosoft.com.cn> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbUEFUQ0hdIE1kZVBrZy9JbmNsdWRlOiBBZGQgQ0VUIGluc3RydWN0aW9ucyB0byBOYXNtLmluYw==?= Date: Wed, 3 Feb 2021 10:01:32 +0800 Message-ID: <00af01d6f9d0$7ea45120$7becf360$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQN3wkfJ+S6PaEcX1atqqqHz4MZGtQHw/3iJAqDLipIBHmN/QgHGVAO8AgZd7GmmuC0bIA== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Wei: The normal process is to send RFC in edk2 mail list and collect the = feedback. This RFC introduces the proposed change, the impact and the = suggestion. Then, wait for one or two weeks to collect the feedback. If = there are more feedback, more time will be required. Now, we are in the late phase of edk2-stable202102 tag development. If = the change brings the big impact, I suggest to add it in the early phase = of one stable tag. For this change, I suggest to collect its impact and = send RFC to start the discussion first.=20 Proposed Schedule Date (00:00:00 UTC-8) Description 2020-11-27 Beginning of development 2021-02-15 Feature Planning Freeze 2021-02-22 Soft Feature Freeze 2021-03-01 Hard Feature Freeze 2021-03-05 Release Thanks Liming > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > =E5=8F=91=E4=BB=B6=E4=BA=BA: Sheng, W > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: = 2021=E5=B9=B42=E6=9C=883=E6=97=A5 8:07 > =E6=94=B6=E4=BB=B6=E4=BA=BA: Kinney, Michael D = ; gaoliming > ; devel@edk2.groups.io > =E6=8A=84=E9=80=81: Liu, Zhiguang ; Yao, = Jiewen > ; 'Andrew Fish' ; 'Laszlo = Ersek' > ; Feng, Roger > =E4=B8=BB=E9=A2=98: RE: [PATCH] MdePkg/Include: Add CET instructions = to Nasm.inc >=20 > Hi Mike, > If we plan to update the NSAM version, it will affect everyone why is = building > the BIOS image, do you know the process ? > How to request all the people update their NSAM tool ? > How long will be the whole process ? > Thanks > BR > Sheng Wei >=20 > > -----Original Message----- > > From: Kinney, Michael D > > Sent: 2021=E5=B9=B42=E6=9C=882=E6=97=A5 23:22 > > To: Sheng, W ; gaoliming > > ; devel@edk2.groups.io; Kinney, Michael D > > > > Cc: Liu, Zhiguang ; Yao, Jiewen > > ; 'Andrew Fish' ; 'Laszlo = Ersek' > > ; Feng, Roger > > Subject: RE: [PATCH] MdePkg/Include: Add CET instructions to = Nasm.inc > > > > Hi Sheng, > > > > I recommend we update required NASM version. > > > > From your investigation, the new min version would be 2.15.01. > > > > Mike > > > > > -----Original Message----- > > > From: Sheng, W > > > Sent: Monday, February 1, 2021 7:39 PM > > > To: gaoliming ; Kinney, Michael D > > > ; devel@edk2.groups.io > > > Cc: Liu, Zhiguang ; Yao, Jiewen > > > ; 'Andrew Fish' ; 'Laszlo > > > Ersek' ; Feng, Roger > > > Subject: RE: [PATCH] MdePkg/Include: Add CET instructions to = Nasm.inc > > > > > > Hi Mike, Liming, > > > The latest NASM version is version 2.15.05. > > > The CET instructions is supported since NASM version 2.15.01, it = is > released > > at 2020-06-27. > > > Do you think we need to request everyone to update the NASM tool = now? > > > Or could I still use macros with DB for CET instructions in my = patch ? > > > BR > > > Sheng Wei > > > > > > > -----Original Message----- > > > > From: gaoliming > > > > Sent: 2021=E5=B9=B42=E6=9C=882=E6=97=A5 10:44 > > > > To: Kinney, Michael D ; Sheng, W > > > > ; devel@edk2.groups.io > > > > Cc: Liu, Zhiguang ; Yao, Jiewen > > > > ; 'Andrew Fish' ; 'Laszlo > > Ersek' > > > > > > > > Subject: =E5=9B=9E=E5=A4=8D: [PATCH] MdePkg/Include: Add CET = instructions to > > > > Nasm.inc > > > > > > > > Mike and Sheng: > > > > Linux or Mac OS includes the pre-built nasm. Their nasm = version may > be > > old. > > > > Can you evaluate the impact to update nasm version? > > > > > > > > Thanks > > > > Liming > > > > > -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6----- > > > > > =E5=8F=91=E4=BB=B6=E4=BA=BA: Kinney, Michael D = > > > > > =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: = 2021=E5=B9=B41=E6=9C=8830=E6=97=A5 1:23 > > > > > =E6=94=B6=E4=BB=B6=E4=BA=BA: Sheng, W ; = devel@edk2.groups.io; > > Kinney, > > > > > Michael D > > > > > =E6=8A=84=E9=80=81: Liming Gao ; = Liu, Zhiguang > > > > > ; Yao, Jiewen > > > > > =E4=B8=BB=E9=A2=98: RE: [PATCH] MdePkg/Include: Add CET = instructions to Nasm.inc > > > > > > > > > > Sheng, > > > > > > > > > > The following version of the NASM documentation shows support = for > > > > > these instructions. > > > > > > > > > > https://nasm.us/doc/nasmdocb.html > > > > > > > > > > Do we need to increase the min NASM version requirements for = EDK > > > > > II to avoid adding macros with DB? > > > > > > > > > > Thanks, > > > > > > > > > > Mike > > > > > > > > > > > -----Original Message----- > > > > > > From: Sheng, W > > > > > > Sent: Thursday, January 28, 2021 6:35 PM > > > > > > To: devel@edk2.groups.io > > > > > > Cc: Kinney, Michael D ; Liming = Gao > > > > > ; Liu, Zhiguang > > > > > > ; Yao, Jiewen > > > > > > Subject: [PATCH] MdePkg/Include: Add CET instructions to > > > > > > Nasm.inc > > > > > > > > > > > > This is to add instruction SAVEPREVSSP, CLRSSBSY and > > > > > > RSTORSSP_RAX in > > > > > Nasm, > > > > > > because these instructions are not supported yet. > > > > > > > > > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3192 > > > > > > > > > > > > Signed-off-by: Sheng Wei > > > > > > Cc: Michael D Kinney > > > > > > Cc: Liming Gao > > > > > > Cc: Zhiguang Liu > > > > > > Cc: Jiewen Yao > > > > > > --- > > > > > > MdePkg/Include/Ia32/Nasm.inc | 14 +++++++++++++- > > > > > > MdePkg/Include/X64/Nasm.inc | 14 +++++++++++++- > > > > > > 2 files changed, 26 insertions(+), 2 deletions(-) > > > > > > > > > > > > diff --git a/MdePkg/Include/Ia32/Nasm.inc > > > > > b/MdePkg/Include/Ia32/Nasm.inc > > > > > > index 31ce861f1e..9c1b7796ea 100644 > > > > > > --- a/MdePkg/Include/Ia32/Nasm.inc > > > > > > +++ b/MdePkg/Include/Ia32/Nasm.inc > > > > > > @@ -1,6 +1,6 @@ > > > > > > > > > > > > = ;--------------------------------------------------------------- > > > > > > ---- > > > > > > ----------- > > > > > > ; > > > > > > -; Copyright (c) 2019, Intel Corporation. All rights > > > > > > reserved.
> > > > > > +; Copyright (c) 2019 - 2021, Intel Corporation. All rights > > > > > > +reserved.
> > > > > > ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; = Abstract: > > > > > > @@ -20,3 +20,15 @@ > > > > > > %macro INCSSP_EAX 0 > > > > > > DB 0xF3, 0x0F, 0xAE, 0xE8 > > > > > > %endmacro > > > > > > + > > > > > > +%macro SAVEPREVSSP 0 > > > > > > + DB 0xF3, 0x0F, 0x01, 0xEA > > > > > > +%endmacro > > > > > > + > > > > > > +%macro CLRSSBSY_EAX 0 > > > > > > + DB 0x67, 0xF3, 0x0F, 0xAE, 0x30 %endmacro > > > > > > + > > > > > > +%macro RSTORSSP_EAX 0 > > > > > > + DB 0x67, 0xF3, 0x0F, 0x01, 0x28 %endmacro > > > > > > diff --git a/MdePkg/Include/X64/Nasm.inc > > > > > b/MdePkg/Include/X64/Nasm.inc > > > > > > index 42412735ea..c5189982bb 100644 > > > > > > --- a/MdePkg/Include/X64/Nasm.inc > > > > > > +++ b/MdePkg/Include/X64/Nasm.inc > > > > > > @@ -1,6 +1,6 @@ > > > > > > > > > > > > = ;--------------------------------------------------------------- > > > > > > ---- > > > > > > ----------- > > > > > > ; > > > > > > -; Copyright (c) 2019, Intel Corporation. All rights > > > > > > reserved.
> > > > > > +; Copyright (c) 2019 - 2021, Intel Corporation. All rights > > > > > > +reserved.
> > > > > > ; SPDX-License-Identifier: BSD-2-Clause-Patent ; ; = Abstract: > > > > > > @@ -20,3 +20,15 @@ > > > > > > %macro INCSSP_RAX 0 > > > > > > DB 0xF3, 0x48, 0x0F, 0xAE, 0xE8 %endmacro > > > > > > + > > > > > > +%macro SAVEPREVSSP 0 > > > > > > + DB 0xF3, 0x0F, 0x01, 0xEA > > > > > > +%endmacro > > > > > > + > > > > > > +%macro CLRSSBSY_RAX 0 > > > > > > + DB 0xF3, 0x0F, 0xAE, 0x30 > > > > > > +%endmacro > > > > > > + > > > > > > +%macro RSTORSSP_RAX 0 > > > > > > + DB 0xF3, 0x0F, 0x01, 0x28 > > > > > > +%endmacro > > > > > > -- > > > > > > 2.16.2.windows.1 > > > > > > > >