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 ED14794133C for ; Mon, 27 May 2024 08:21:29 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=sjhbzgT40Hw0umRJbBu42VZ9zpLB4Ithn33esyjQlm4=; 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-Language; s=20240206; t=1716798089; v=1; b=bAQ/DMONUo/U0uzs3ha+83nPgHyCqOGKu8q/deooMZzMGtjYEsnnRxTRxR9f51qKXuGqP3c9 waj8V6b5IhMA8nc73tNZG3iBh7DkaNuIirwpyDFj4uSfGrSkmzkg1Jm52qArWgFjRadbIQwi/jk AJxWxLlB38bSmAsTMJEG5VKdN9yaOW2UBpartz3w1h02DATr7koP+4Rj/2HTTwa5wUxsyEwIGNW pnQ6d7pkJnuUVhuPC6+2enzHex1xJcayb5+bdvK+YLh+86zmf99hRB3dL5Vs+Mjq9QPNkYtIAMB o0tTstPTLs1KAE+K8D0cRVWUQpgq2dCXC8RrKHsUjxEuA== X-Received: by 127.0.0.2 with SMTP id lgajYY7687511xeQoNmWzfxk; Mon, 27 May 2024 01:21:28 -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.23040.1716798086303136806 for ; Mon, 27 May 2024 01:21:27 -0700 X-Received: from zrleap.intel-email.com (localhost [127.0.0.1]) by zrleap.intel-email.com (Postfix) with ESMTP id 52A83A32E1E8 for ; Mon, 27 May 2024 16:21:23 +0800 (CST) X-Received: from localhost (localhost [127.0.0.1]) by zrleap.intel-email.com (Postfix) with ESMTP id 3DE58A32E1E7 for ; Mon, 27 May 2024 16:21:23 +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 0505EA32E1C4 for ; Mon, 27 May 2024 16:21:16 +0800 (CST) X-Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP(SSL) for ; Mon, 27 May 2024 16:21:03 +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 via groups.io" To: , , "'Kinney, Michael D'" , "'Andrew Fish'" , "'Leif Lindholm'" Cc: "'Ard Biesheuvel'" , , , "'Mathews, John'" , "'Li, Yi1'" , "'Yao, Jiewen'" References: <17CD1075E73EEFD7.30273@groups.io> <002501daa2ee$c4445660$4ccd0320$@byosoft.com.cn> <033201daa661$b5fbd670$21f38350$@byosoft.com.cn> <07b501daab8a$20f3b510$62db1f30$@byosoft.com.cn> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gU29mdCBGZWF0dXJlIEZyZWV6ZSBzdGFydHMgbm93IGZvciBlZGsyLXN0YWJsZTIwMjQwNQ==?= Date: Mon, 27 May 2024 16:21:03 +0800 Message-ID: <001201dab00e$d0ccbbc0$72663340$@byosoft.com.cn> MIME-Version: 1.0 Thread-Index: AQKfbKTbXoEYRIOo6T31ZtZb+qHX+wLixre/AjTZjFUBoY5x1gKwsRPUA3duR2wBQDt5vq+wNr0g 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: Mon, 27 May 2024 01:21:27 -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: j8scqhPfUtwQ03yPTr9CxKClx7686176AA= Content-Type: multipart/alternative; boundary="----=_NextPart_000_0013_01DAB051.DEF727B0" 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="bAQ/DMON"; 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 ------=_NextPart_000_0013_01DAB051.DEF727B0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Wenxing: If this patch is required to be merged immediately, I will help add push = label. =20 =20 Thanks Liming =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io = =E4=BB=A3=E8=A1=A8 Wenxing Hou =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B45=E6=9C=8827=E6=97=A5 12= :24 =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming ; devel@ed= k2.groups.io; Kinney, Michael D ; 'Andrew Fish'= ; 'Leif Lindholm' =E6=8A=84=E9=80=81: 'Ard Biesheuvel' ; sam.kaynor@arm.com;= dougflick@microsoft.com; Mathews, John ; Li, Yi1 <= yi1.li@intel.com>; Yao, Jiewen =E4=B8=BB=E9=A2=98: Re: [edk2-devel] Soft Feature Freeze starts now for edk= 2-stable202405 =20 Hi Liming, =20 =20 Our PR: https://github.com/tianocore/edk2/pull/5645 need to be merged too. And the CryptoPkg maintainer Li Yi can not add the Push label. =20 Could you please merge this PR? =20 Thanks, Wenxing =20 =20 From: gaoliming = >=20 Sent: Tuesday, May 21, 2024 10:21 PM To: devel@edk2.groups.io ; Hou, Wenxing >; Kinney, Michael D >; 'Andrew Fish'= >; 'Leif Lindholm' > Cc: 'Ard Biesheuvel' >; sam.kayno= r@arm.com ; dougflick@microsoft.com ; Mathews, John >; Li, Yi1 >= ; Yao, Jiewen > Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] Soft Feature Freeze starts now fo= r edk2-stable202405 =20 Wenxing: We are in hard feature freeze phase. In this phase, only bug fixes will b= e accepted (https://github.com/tianocore/tianocore.github.io/wiki/HardFeatu= reFreeze).=20 =20 So, this patch set may be merged after the stable tag is created.=20 =20 Thanks Liming =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.grou= ps.io < devel@edk2.groups.io> =E4=BB=A3=E8=A1= =A8 Wenxing Hou =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B45=E6=9C=8817=E6=97=A5 23= :10 =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming < = gaoliming@byosoft.com.cn>; Kinney, Michael D < michael.d.kinney@intel.com>; devel@e= dk2.groups.io; 'Andrew Fish' < afish@apple.com>; '= Leif Lindholm' < quic_llindhol@quicinc.c= om> =E6=8A=84=E9=80=81: 'Ard Biesheuvel' < ardb@kernel= .org>; sam.kaynor@arm.com; dougflick@microsoft.com; Mathews, John < john.mathews@intel.com>; Li, Yi1 < y= i1.li@intel.com>; Yao, Jiewen < jiewen.yao@in= tel.com> =E4=B8=BB=E9=A2=98: Re: [edk2-devel] Soft Feature Freeze starts now for edk= 2-stable202405 =20 Hi Liming and Mike, =20 For the PATCH: Add more crypt APIs based on Mbedtls ( https://edk2.groups.io/g/devel/message/118772), I have fixed the code based on Mike=E2=80=99s feedback and added the platform integration test table in commit message. =20 Could you approve this PATCH to catch this table tag? =20 =20 Thanks, Wenxing From: gaoliming < gaoliming@byosoft.com.c= n>=20 Sent: Wednesday, May 15, 2024 8:49 AM To: Kinney, Michael D < michael.d.kinne= y@intel.com>; devel@edk2.groups.io; 'Andrew = Fish' < afish@apple.com>; 'Leif Lindholm' < quic_llindhol@quicinc.com> Cc: 'Ard Biesheuvel' < ardb@kernel.org>; sam.kaynor@arm.com; d= ougflick@microsoft.com; Mathews, John < joh= n.mathews@intel.com>; Hou, Wenxing < wenxing= .hou@intel.com>; Li, Yi1 < yi1.li@intel.com>; Yao= , Jiewen < jiewen.yao@intel.com> Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] Soft Feature Freeze starts now fo= r edk2-stable202405 =20 Mike: Thanks for your comments. I update the status below.=20 =20 Thanks Liming =E5=8F=91=E4=BB=B6=E4=BA=BA: Kinney, Michael D < michael.d.kinney@intel.com>=20 =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B45=E6=9C=8815=E6=97=A5 0:= 35 =E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming < = gaoliming@byosoft.com.cn>; devel@edk2.groups= .io; 'Andrew Fish' < afish@apple.com>; 'Leif Lindh= olm' < quic_llindhol@quicinc.com> =E6=8A=84=E9=80=81: 'Ard Biesheuvel' < ardb@kernel= .org>; sam.kaynor@arm.com; dougflick@microsoft.com; Mathews, John < john.mathews@intel.com>; Hou, Wenxing < wenxing.hou@intel.com>; Li, Yi1 < yi1.li= @intel.com>; Yao, Jiewen < jiewen.yao@intel.c= om>; Kinney, Michael D < michael.d.kinn= ey@intel.com> =E4=B8=BB=E9=A2=98: RE: [edk2-devel] Soft Feature Freeze starts now for edk= 2-stable202405 =20 Hi Liming, =20 My responses below in [Mike]. =20 Mike =20 =20 From: gaoliming < gaoliming@byosoft.com.c= n>=20 Sent: Friday, May 10, 2024 8:29 AM To: devel@edk2.groups.io; Kinney, Michael D = < michael.d.kinney@intel.com>; 'Andrew = Fish' < afish@apple.com>; 'Leif Lindholm' < quic_llindhol@quicinc.com> Cc: 'Ard Biesheuvel' < ardb@kernel.org>; sam.kaynor@arm.com; d= ougflick@microsoft.com; Mathews, John < joh= n.mathews@intel.com>; Hou, Wenxing < wenxing= .hou@intel.com> Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] Soft Feature Freeze starts now fo= r edk2-stable202405 =20 Stewards: Now, there are several patches to catch this table tag. Could you give th= e comments for them? =20 1. Adding support for verbose UEFI Table dumping to Dmem.c (https://ed= k2.groups.io/g/devel/message/118582) [Liming] This patch set has been reviewed before soft feature freeze. It pl= ans to catch this stable tag.=20 =20 [Mike] I see this PR: https:= //github.com/tianocore/edk2/pull/5653 that is not passing CI and appears it= will require additional code changes =20 [Mike] Reject for edk2-stable202405 =20 2. MdePkg/BaseLib: Fix AARCH64 compilation error (https://edk2.groups.io/g= /devel/message/118690) [Liming] This bug fix is reviewed in soft feature freeze phase. It plans to= catch this stable tag.=20 =20 [Mike] Approved for edk2-stable202405 [Liming] I add push label for https://github.com/tianocore/edk2/pull/5642 =20 3. MdeModulePkg: Potential UINT32 overflow in S3 ResumeCount ( https://edk2.groups.io/g/devel/message= /118745) [Liming] This security fix is reviewed in soft feature freeze phase. It pla= ns to catch this stable tag. =20 [Mike] Approved for edk2-stable202405 [Liming] I add push label for https://github.com/tianocore/edk2/pull/5659 =20 4. NetworkPkg: CVE-2023-45236 and CVE-2023-45237 ( https://edk2.groups.io/g/devel/message/118768) [Liming] This security fix is still under code review. It plans to catch th= is stable tag. =20 [Mike] Is the code review complete? Is there a link to the PR? [Liming] NetworkPkg reviewer will review this patch set this week.=20 =20 Thanks =20 5. Add more crypt APIs based on Mbedtls ( https://edk2.groups.io/g/devel/message/118772) [Liming] This patch set passes code review in soft feature freeze phase. It= plans to catch this stable tag. =20 [Mike] This patch series uses =E2=80=98..=E2=80=99 in INF to access source = files in another component. This is not legal. I am surprised this was no= t caught in code review. =20 DEFINE OPENSSL_PATH =3D ../OpensslLib/openssl DEFINE BASE_CRYPT_PATH =3D ../BaseCryptLib =20 [Mike] I see a reference to some =E2=80=9Cplatform integration=E2=80=9D tes= ting. Given that this patch series implements a number of crypto service APIs and is a large number of new lines of code, it would be= good to know if all of the newly added APIs=20 were tested in a platform integration. A table of the added APIs and the pl= atform integration test status would be good to=20 know if there was any functional testing of each API. If there are APIs th= at are not covered by any platform integration=20 testing, then I would be concerned with such a large change with limited te= sting. =20 Thanks Liming =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.grou= ps.io < devel@edk2.groups.io> =E4=BB=A3=E8=A1= =A8 gaoliming via groups.io =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B45=E6=9C=887=E6=97=A5 9:2= 5 =E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk2.grou= ps.io; announce@edk2.groups.io =E6=8A=84=E9=80=81: 'Michael D Kinney' < michael.d.kinney@intel.com>; 'Andrew Fish' < afi= sh@apple.com>; 'Leif Lindholm' < quic_ll= indhol@quicinc.com> =E4=B8=BB=E9=A2=98: [edk2-devel] Soft Feature Freeze starts now for edk2-st= able202405 =20 Hi, all =20 We enter into Soft Feature Freeze phase now. In this phase, the feature under review will not be allowed to be pushed. The feature passed review can still be merged. =20 The patch review can continue without break in edk2 community. If the patch is sent before Soft Feature Freeze, and plans to catch this stable ta= g, the patch contributor need reply to his patch and notify edk2 community. If the patch is sent after Soft Feature Freeze, and plans to catch this stable tag= , please add edk2-stable202405 key words in the patch title and BZ, so the community know this patch target and give the feedback. =20 To avoid the unnecessary changes to be merged in edk2 stable tag release, all edk2 maintainers' write access will be temporarily disabled until stabl= e tag is released on 05-24. That means edk2 maintainer can't set push label i= n pull request after soft feature freeze starts. =20 If the change wants to catch this stable tag 202405, please follow above rules, then send the merge request to ga= oliming@byosoft.com.cn or=20 michael.d.kinney@intel.com. =20 We will help merge the code change in soft feature freeze and hard featur= e freeze phase.=20 =20 Below is edk2-stable202405 tag planning Proposed Schedule Date (00:00:00 UTC-8) Description =20 2024-02-23 Beginning of development 2024-05-06 Soft Feature Freeze 2024-05-10 Hard Feature Freeze 2024-05-24 Release =20 Thanks Liming =20 -=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 (#119279): https://edk2.groups.io/g/devel/message/119279 Mute This Topic: https://groups.io/mt/106326950/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- ------=_NextPart_000_0013_01DAB051.DEF727B0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

Wenxing:<= /span>

=C2=A0 If this patch is required to be mer= ged immediately, I will help add push label. =C2=A0

 

Tha= nks

Liming

=E5=8F=91=E4= =BB=B6=E4=BA=BA: devel@edk2.groups.io <devel@edk2.groups.io> = =E4=BB=A3=E8=A1=A8 Wenxing Hou
=E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: = 2024=E5=B9=B45=E6=9C=8827=E6=97=A5 12:2= 4
=E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.gr= oups.io; Kinney, Michael D <michael.d.kinney@intel.com>; 'Andrew Fish= ' <afish@apple.com>; 'Leif Lindholm' <quic_llindhol@quicinc.com>= ;
=E6=8A=84=E9=80=81: 'Ard Biesheuvel' <ardb@kernel.org>; sam.kaynor@arm.com; dou= gflick@microsoft.com; Mathews, John <john.mathews@intel.com>; Li, Yi1= <yi1.li@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
=E4=B8=BB=E9=A2=98: = Re: [edk2-devel] Soft Feature Freeze starts now for edk2-stable202405<= /o:p>

 

Hi Liming,

 

 

=

Our PR: https://github.com/tianocore/edk2/pull/5645 need to be merg= ed too.

And the CryptoPkg main= tainer Li Yi  can not add the Push label.

 

Could yo= u please merge this PR?

&n= bsp;

Thanks,

Wenxing

 

 

From: gaoliming <gaoliming@byosoft.com.cn>
Sent: Tuesday, May = 21, 2024 10:21 PM
To: dev= el@edk2.groups.io; Hou, Wenxing <wenxing.hou@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; 'Andrew F= ish' <afish@apple.com>; 'Leif = Lindholm' <quic_llindhol@qu= icinc.com>
Cc: 'Ard Biesheuvel' <ardb@kernel.org>; sam.kaynor@arm.com; dougfli= ck@microsoft.com; Mathews, John <john.mathews@intel.com>; Li, Yi1 <yi1.li@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
Subject: <= span style=3D'font-size:11.0pt;font-family:"=E5=BE=AE=E8=BD=AF=E9=9B=85=E9= =BB=91",sans-serif'>=E5=9B=9E=E5=A4=8D: [edk2-devel] Soft Featur= e Freeze starts now for edk2-stable202405

=

 

Wen= xing:

  W= e are in hard feature freeze phase. In this phase, only bug fixes will be a= ccepted (https://github.com/tianocore/tianocore.github.io/wiki/Har= dFeatureFreeze).

 

So, this patch set may be merged after the s= table tag is created.

 

Thanks

Limin= g

=E5=8F=91=E4=BB= =B6=E4=BA=BA: devel@edk2.groups.io<= /a> <devel@edk2.groups.io> =E4=BB= =A3=E8=A1=A8 Wenxi= ng Hou
=E5=8F=91=E9=80=81=E6= =97=B6=E9=97=B4: 2024=E5=B9= =B45=E6=9C=8817=E6=97= =A5 23:10
=E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming <gaoliming@byosoft.com.cn>; Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.grou= ps.io; 'And= rew Fish' <= afish@apple.com>; 'Leif Lindholm' <quic_llindhol@quicinc.com>
=E6=8A=84=E9=80=81: 'Ard Biesheuvel' <ardb@kernel.org>; sam.kaynor@arm.com; dougf= lick@microsoft.com; Mathews, John <john.mathews@intel.com= >; Li, Y= i1 <yi1.li@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
=E4=B8=BB=E9= =A2=98: Re: [edk2-devel] Soft Feature Freeze starts now for edk2-st= able202405

 

Hi Liming and Mike,

 

For the PATCH: Add more crypt APIs based on Mbedtls (https://edk2.groups.i= o/g/devel/message/118772),

I have fixed the code based on Mike=E2=80=99s fee= dback

and added the platform = integration test table in commit message.

 

Could you= approve this PATCH to catch this table tag?

 

=  

Thanks,

Wenxing

From:<= /b> gaoliming <gaoliming@byosoft.com.cn>
Sent: Wednesday, May 15, 2024 8:49 AM
To: Kinney, Michael D <michael.d.kinney@= intel.com>; devel@edk2.groups.io; 'Andrew Fish= ' <afish@apple.com>; 'Leif Lindholm' <quic_llindhol@quicinc.com>
Cc: 'Ard Biesheuvel' <ardb@kernel.org>; sam.kaynor@arm.com<= /span>; dou= gflick@microsoft.com; Mathews, John <john.mathews@intel.com= >; Hou, Wenxing <wenxing.hou@intel.com>; Li, Y= i1 <yi1.li@intel.co= m>; Yao, Jiewen <jiewen.yao@intel.com>Subject: =E5=9B=9E=E5=A4=8D: [edk2-devel] Soft Feature Freeze starts now for edk2-stable202405

 

Mike:

  Thanks for your comments. I update the status below.=

 <= /o:p>

Thanks

Liming

=E5=8F=91=E4=BB=B6=E4=BA=BA: Kinney, Michael D <michael.d.kinney@= intel.com&g= t;
=E5=8F=91=E9=80=81=E6=97= =B6=E9=97=B4: 2024=E5=B9= =B45=E6=9C=8815=E6=97= =A5 0:35
=E6=94=B6=E4=BB=B6=E4=BA=BA: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.gro= ups.io; 'An= drew Fish' <afish@apple.com>; 'Leif Lindholm' <quic_llindhol@quicinc.com>
=E6=8A=84=E9=80=81: 'Ard Biesheuvel' <ardb@kernel.org>; sam.kaynor@arm.com; dougf= lick@microsoft.com; Mathews, John <john.mathews@intel.com= >; Hou, = Wenxing <wenxing.hou@intel.com>; Li, Yi1 <yi1.li@intel.com>; Yao, Jiewen <jiewen.yao@int= el.com>;= Kinney, Michael D <michael.d.kinney@intel= .com>=E4=B8=BB=E9=A2=98: RE:= [edk2-devel] Soft Feature Freeze starts now for edk2-stable202405

 

Hi Liming,

 

My responses below in [Mik= e].

Mike

 

 

From: gaoliming <gaoliming@byosoft.com.cn>
Sent: Frida= y, May 10, 2024 8:29 AM
To: devel@edk2.groups.io; Kinney, Mic= hael D <m= ichael.d.kinney@intel.com>; 'Andrew Fish' <afish@apple.com<= /span>>; 'Leif Lindholm' <quic_llindhol@quicinc.com><= br>Cc: 'Ard Biesheuvel' <ardb@kernel.org>; sam.kaynor@arm.com= ; dougflick@mic= rosoft.com; Mathews, John <john.mathews@intel.com>; Hou, Wenxing <wenxing.hou@intel.com>
Subject:<= /b> = =E5=9B=9E=E5=A4=8D: [edk2-devel] Soft Feature Freeze starts now = for edk2-stable202405

 

Stewards:

  Now, there are sever= al patches to catch this table tag. Could you give the comments for them?

 

1.&= nbsp;     Adding support for verbose = UEFI Table dumping to Dmem.c (https://edk2.g= roups.io/g/devel/message/118582)

[Liming] This patch set has been rev= iewed before soft feature freeze. It plans to catch this stable tag.

 <= /p>

[Mike] I se= e this PR: https://github.com/tianocore/edk2/pull/5653<= /span> that is not passing CI and appears it wil= l require additional code changes

 

[Mike] Reject for ed= k2-stable202405

 

= 2.  MdePkg/BaseLib: Fix AARCH64 compilation error = (https://edk2.groups.io/g/devel/message/1186= 90)

[Liming] This bug fix = is reviewed in soft feature freeze phase. It plans to catch this stable tag= .

 

=

[Mike] Approved for edk2-stable202405=

[Liming] I add push label for https:= //github.com/tianocore/edk2/pull/5642

=  

3.  MdeModulePkg: Potential UINT32 overflow in S3 ResumeCount (https://edk2.groups.i= o/g/devel/message/118745)

[Liming] This security fix is reviewed in soft feature freeze phase.= It plans to catch this stable tag.

 

[Mike] Approved for edk2-stable202405

[Liming] I= add push label for https://= github.com/tianocore/edk2/pull/5659

&= nbsp;

4.  NetworkPkg: CVE-2023-45236 and CVE-2023-45237 (https://edk2.groups.io/g/devel/mes= sage/118768)

[Limin= g] This security fix is still under code review. It plans to catch this sta= ble tag.

 

[Mike] Is the code review compl= ete?  Is there a link to the PR?

[Liming] NetworkPkg= reviewer will review this patch set this week.

=  

Thanks

 

5.  Add more crypt APIs based on Mbedtls (https://edk2.groups.io/g/devel/mess= age/118772)

[Liming= ] This patch set passes code review in soft feature freeze phase. It plans = to catch this stable tag.

 

[Mike] This patch series uses =E2=80=98..=E2=80=99 in INF = to access source files in another component.  This is not legal. = I am surprised this was not caught in code review.

 

DEFINE OPENSSL_PATH =            =3D ../Openssl= Lib/openssl

DEFINE BASE_CRYPT_PATH   &nbs= p;     =3D ../BaseCryptLib

 

[Mik= e] I see a reference to some =E2=80=9Cplatform integration=E2=80=9D testing= .  Given that this patch series implements a number of

crypto service APIs and is a large number of= new lines of code, it would be good to know if all of the newly added APIs=

were tested in a platform i= ntegration. A table of the added APIs and the platform integration test sta= tus would be good to

know i= f there was any functional testing of each API.  If there are APIs tha= t are not covered by any platform integration

testing, then I would be concerned with such a large chan= ge with limited testing.

 

Thanks=

Liming<= /span>

=E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io <devel@= edk2.groups.io> =E4=BB=A3=E8=A1=A8 gaoliming via groups.io=
=E5=8F=91=E9=80=81=E6=97=B6= =E9=97=B4: 2024=E5=B9=B45=E6=9C=887=E6=97=A5 9:25
=E6=94=B6=E4=BB=B6=E4=BA=BA: devel@edk= 2.groups.io= ; announce@edk2.groups.io
=E6=8A=84=E9=80=81: 'Michael D Kinney' <michael.d.kinney@intel.com>; 'Andrew Fish' <afish@apple.com>; 'Leif Lindholm' <quic_llindho= l@quicinc.com>
=E4=B8=BB=E9=A2=98: [edk2-devel] Soft Feature Freeze starts now for edk2-stable202405

 

Hi, = all

 

&nbs= p; We enter into Soft Feature Freeze phase now. In this phase,

the feature under review will not be allowed to be pushed= . The feature

passed review can still be mer= ged.

 

&nb= sp; The patch review can continue without break in edk2 community. If the

patch is sent before Soft Feature Freeze, and = plans to catch this stable tag, the

patch con= tributor need reply to his patch and notify edk2 community. If the

patch is sent after Soft Feature Freeze, and plans to= catch this stable tag,

please add edk2-stabl= e202405 key words in the patch title and BZ, so the

community know this patch target and give the feedback.

 

  To avoid the = unnecessary changes to be merged in edk2 stable tag release,

all edk2 maintainers' write access will be temporarily disa= bled until stable

tag is released on 05-24. T= hat means edk2 maintainer can't set push label in

pull request after soft feature freeze starts.

 

  If the change wants to = catch this stable tag 202405, please follow above

rules, then send the merge request to gaoliming@byosoft.com.cn or <= /p>

michael.d.kinney@intel.com.

 =

  We will help merge the code change in soft= feature freeze and hard feature

freeze phase= .

 

Below= is edk2-stable202405 tag planning Proposed Schedule

<= p class=3DMsoNormal align=3Dleft style=3D'text-align:left;line-height:14.25= pt;background:white'>Date (00:00:00 UTC-8) Description

 

2024-02-23 Beginning of development<= /o:p>

2024-05-06 Soft Feature Freeze

2024-05-10 Hard Feature Freeze

202= 4-05-24 Release

 

<= p class=3DMsoNormal align=3Dleft style=3D'text-align:left;line-height:14.25= pt;background:white'>Thanks

Liming

 

<= /span>

_._,_._,_
Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#119279) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
------=_NextPart_000_0013_01DAB051.DEF727B0--