回复: [edk2-devel] Soft Feature Freeze start date delays to 2020-08-24 for edk2-stable202008

2020-08-26 Thread gaoliming
Jiewen:

For this patch set V3, I see you give reviewed-by on Aug 15th 
https://edk2.groups.io/g/devel/message/64299 
For V4, https://edk2.groups.io/g/devel/message/64354, it makes the change per 
your comment. 
So, I agree you as SecurityPkg maintainter reviewed-by is OK for V4 patch set. 

And, I also see Chiu, Chasel as IntelFsp2WrapperPkg maintainer gives 
reviewed-by on Aug 18th for the changes in IntelFsp2WrapperPkg

So, I think the latest patch set V4 finished code review before new SFF start 
date (2020-08-24). 
Based on SFF definition, I agree this patch set can be merged for this stable 
tag 202008. 

Thanks
Liming
> -邮件原件-
> 发件人: bounce+27952+64638+4905953+8761...@groups.io
>  代表 Yao, Jiewen
> 发送时间: 2020年8月26日 18:17
> 收件人: Laszlo Ersek ; devel@edk2.groups.io; gaoliming
> ; 'Leif Lindholm' ;
> af...@apple.com; Kinney, Michael D ; Guptha,
> Soumya K 
> 抄送: annou...@edk2.groups.io; 'Chang, Abner (HPS SW/FW Technologist)'
> ; Zhang, Qi1 ;
> marcello.ba...@9elements.com
> 主题: Re: [edk2-devel] Soft Feature Freeze start date delays to 2020-08-24 for
> edk2-stable202008
> 
> HI Laszlo
> I checked the history.
> 
> Jiewen replied " [PATCH v3 0/8] Need add a FSP binary measurement" with
> review-by on V3 patch series in August 15, with comment to rename
> FvEventLogRecordLib to TcgEventLogRecordLib.
> Qi sent v4 series in August 17, with only naming change from
> FvEventLogRecordLib to TcgEventLogRecordLib.
> Jian replied "[PATCH v3 0/8] Need add a FSP binary measurement" with
> reviewed-by on V3 patch series in August 18.
> 
> So I treat this patch series is qualified to check in (since V4 adopted my
> comment). But please let me know if there is any misunderstanding.
> 
> 
> When I am about to merge, I am told that we are in SFF and I cannot check in.
> According to the plan, I will check in after August 28, which is end of 
> August. It
> is still OK for me.
> 2020-08-14  Soft Feature Freeze
> 2020-08-21  Hard Feature Freeze
> 2020-08-28  Release
> 
> But now, if we need delay one week, then the final release data will be
> September. If I cannot check in now, I will have to check in at September.
> That is why I said, it impacts me, because of this one week delay.
> 
> Thank you
> Yao Jiewen
> 
> 
> > -Original Message-
> > From: Laszlo Ersek 
> > Sent: Wednesday, August 26, 2020 5:54 PM
> > To: Yao, Jiewen ; devel@edk2.groups.io; gaoliming
> > ; 'Leif Lindholm' ;
> > af...@apple.com; Kinney, Michael D ;
> Guptha,
> > Soumya K 
> > Cc: annou...@edk2.groups.io; 'Chang, Abner (HPS SW/FW Technologist)'
> > ; Zhang, Qi1 ;
> > marcello.ba...@9elements.com
> > Subject: Re: [edk2-devel] Soft Feature Freeze start date delays to
> 2020-08-24 for
> > edk2-stable202008
> >
> > Hi Jiewen,
> >
> > On 08/26/20 03:19, Yao, Jiewen wrote:
> > > To clarify below:
> > > I just notice this one week delay. It impacts us.
> > >
> > >> https://edk2.groups.io/g/devel/message/64354
> > >> [PATCH v4 0/8] Need add a FSP binary measurement
> > >
> > > The SecurityPkg patches have not been approved yet, and Bret and
> > > Jiewen are still testing / discussing (as far as I understand):
> > > . Material for the next
> > > tag.
> > >
> > > [Jiewen] I think the security pkg is already approved by me and Jian
> > > (SecurityPkg maintainer) Bret also provides feedback that it looks
> > > good.
> >
> > The series in question has three SecurityPkg patches:
> >
> >   [PATCH v4 1/8] SecurityPkg/TcgEventLogRecordLib: add new lib for
> firmware
> > measurement
> >   [PATCH v4 5/8] SecurityPkg/dsc: add FvEventLogRecordLib
> >   [PATCH v4 7/8] SecurityPkg/Tcg2: handle PRE HASH and LOG ONLY
> >
> > As I'm writing this, *none* of the listed patches have any kind of
> > Reviewed-by or Acked-by, either included in the patches themselves, or
> > posted in response to them.
> >
> > > I request to check in to stable202008, if possible.
> >
> > We can do that only if (a) we extend the SFF deadline again, and (b)
> > each of the SecurityPkg patches receives at least an Acked-by from one
> > of the SecurityPkg maintainers, until the new deadline.
> >
> > I'm certainly not against the idea. I don't mind if the release slips
> > some more; it's OK to say that we're not ready to release yet. The point
> > is, as long as we're doing more work for completing the release, we
> > should prolong the stabilization period as well (opportunity for people
> > to test).
> >
> > Thanks,
> > Laszlo
> 
> 
> 




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#64640): https://edk2.groups.io/g/devel/message/64640
Mute This Topic: https://groups.io/mt/76427263/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



回复: [edk2-devel] Soft Feature Freeze start date delays to 2020-08-24 for edk2-stable202008

2020-08-24 Thread gaoliming
Laszlo:
  Thanks for your great support. 

  I merge SmBios type 42 patch @ 78ab44cb96808bd3ff0edf312599f94a6a92172a.

Thanks
Liming
> -邮件原件-
> 发件人: bounce+27952+64572+4905953+8761...@groups.io
>  代表 Laszlo Ersek
> 发送时间: 2020年8月24日 21:38
> 收件人: gaoliming ; 'Leif Lindholm'
> ; af...@apple.com; 'Kinney, Michael D'
> ; 'Guptha, Soumya K'
> 
> 抄送: devel@edk2.groups.io; annou...@edk2.groups.io; 'Chang, Abner (HPS
> SW/FW Technologist)' ; qi1.zh...@intel.com;
> marcello.ba...@9elements.com
> 主题: Re: [edk2-devel] Soft Feature Freeze start date delays to 2020-08-24
for
> edk2-stable202008
> 
> Hi Liming,
> 
> On 08/24/20 03:49, gaoliming wrote:
> > Hi, all
> >
> > Based on the discussion https://edk2.groups.io/g/devel/message/64493,
> > we make the conclusion to defer one week for edk2-stable202008 to
> > include the important patch for RiscV. Soft Feature Freeze (SFF) will
> > start from today (2020-08-24). Below is new edk2-stable202008 tag
> > planning proposed schedule
> >
>
 ing>.
> > If you have any comments, please raise it.
> >
> >
> > Date (00:00:00 UTC-8) Description
> > 2020-06-03Beginning of development
> > 2020-08-07Feature Planning Freeze
> > 2020-08-24Soft Feature Freeze
> > 2020-08-28Hard Feature Freeze
> > 2020-09-04Release
> >
> > Because SFF date is changed, if the patches passed code review before
> > SFF date (2020-08-24), the patches can still be merged for this stable
> > tag. Here is the patch list those passed code review before new SFF
> > date. For below features, I will let the feature owner decides whether
> > to merge it for this stable tag.
> 
> As of this writing:
> 
> - Mon Aug 24 12:59:59 UTC 2020
> 
> we've entered the (new) Soft Feature Freeze. (The SFF date is 2020-08-24
> 00:00:00 UTC-8; in UTC, that's Mon Aug 24 08:00:00 UTC 2020 -- about five
> hours ago.)
> 
> So:
> 
> > Feature List:
> >
> > https://edk2.groups.io/g/devel/message/63767
> > [PATCH] EmbeddedPkg/libfdt: Add strncmp macro to use AsciiStrnCmp
> 
> I'm going to merge this soon (with Leif's review).
> 
> > https://edk2.groups.io/g/devel/message/64363
> > [PATCH v5 0/3] UefiPayloadPkg: Runtime MMCONF
> 
> The UefiPayloadPkg patches in the series (#1, #3) have not received
reviews.
> We have to delay this until after the stable tag.
> 
> > https://edk2.groups.io/g/devel/message/64354
> > [PATCH v4 0/8] Need add a FSP binary measurement
> 
> The SecurityPkg patches have not been approved yet, and Bret and Jiewen
> are still testing / discussing (as far as I understand):
> . Material for the next tag.
> 
> 
> On the other hand, I'm going to push two patches that had been reviewed
just
> before we entered the *delayed* SFF:
> 
>   https://edk2.groups.io/g/devel/message/64345
>   [PATCH 1/1] OvmfPkg/Bhyve: rename files to remove 'Pkg' infix
> 
>   https://edk2.groups.io/g/devel/message/62561
>   [PATCH] OvmfPkg/SmmControl2Dxe: negotiate
> ICH9_LPC_SMI_F_CPU_HOTPLUG
> 
> > Bug List:
> >
> > https://edk2.groups.io/g/devel/message/50406
> > [PATCH 1/1] MdePkg/Include: Add missing definitions of SMBIOS type 42h
> > in SmBios.h
> 
> Liming, can you please merge this patch? For some reason I can't see it in
my
> local archives! (Only responses to it.)
> 
> > https://edk2.groups.io/g/devel/message/64507
> > [PATCH v2 1/1] UefiCpuPkg/MpInitLib: Always initialize the DoDecrement
> > variable
> 
> I'm going to merge this soon.
> 
> > https://edk2.groups.io/g/devel/message/64539
> > [PATCH] Maintainers.txt: Update Liming mail address
> 
> I'll merge this one as well.
> 
> > https://edk2.groups.io/g/devel/message/64529
> > [PATCH] .pytool/EccCheck: Disable Ecc error code 10014 for open CI
> 
> Already merged as commit d4e0b9607c9a.
> 
> >
> > https://edk2.groups.io/g/devel/message/61938
> > [PATCH v2 1/1] MdePkg : UefiFileHandleLib: fix buffer overrun in
> > FileHandleReadLine()
> 
> I'll merge it.
> 
> Thanks!
> Laszlo
> 
> 
> 




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#64588): https://edk2.groups.io/g/devel/message/64588
Mute This Topic: https://groups.io/mt/76398678/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-