public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "gaoliming" <gaoliming@byosoft.com.cn>
To: <devel@edk2.groups.io>, <gaoliming@byosoft.com.cn>,
	<lersek@redhat.com>, "'Leif Lindholm'" <leif@nuviainc.com>,
	<afish@apple.com>,
	"'Kinney, Michael D'" <michael.d.kinney@intel.com>,
	"'Guptha, Soumya K'" <soumya.k.guptha@intel.com>,
	"'Zhiguang Liu'" <zhiguang.liu@intel.com>,
	"'Zhang, Qi1'" <qi1.zhang@intel.com>
Subject: 回复: [edk2-devel] Patch List for 202008 stable tag
Date: Tue, 1 Sep 2020 17:35:45 +0800	[thread overview]
Message-ID: <001b01d68043$44cda3e0$ce68eba0$@byosoft.com.cn> (raw)
In-Reply-To: <162D6EE567916534.7353@groups.io>

Hi, all
  There are still some patches to be merged for 202008 stable tag. They are
all bug fixes. The risk is low. I agree to catch them in this stable tag. If
you have other comments, please reply this mail. 

https://edk2.groups.io/g/devel/message/64873 [PATCH v3]
MdeModulePkg/Library: add PEIM and SEC module type to TpmMeasurementLibNull
https://edk2.groups.io/g/devel/message/64865 [PATCH] SecurityPkg: Initailize
variable Status before it is consumed.
https://edk2.groups.io/g/devel/message/64812 [PATCH]
IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec: add FspMeasurementLib.h

Thanks
Liming
> -----邮件原件-----
> 发件人: bounce+27952+64554+4905953+8761045@groups.io
> <bounce+27952+64554+4905953+8761045@groups.io> 代表 gaoliming
> 发送时间: 2020年8月22日 8:16
> 收件人: devel@edk2.groups.io; lersek@redhat.com; 'Leif Lindholm'
> <leif@nuviainc.com>; afish@apple.com; 'Kinney, Michael D'
> <michael.d.kinney@intel.com>; 'Guptha, Soumya K'
> <soumya.k.guptha@intel.com>; 'Chang, Abner (HPS SW/FW Technologist)'
> <abner.chang@hpe.com>; 'Vladimir Olovyannikov'
> <vladimir.olovyannikov@broadcom.com>; 'Tom Lendacky'
> <thomas.lendacky@amd.com>; gaoliming@byosoft.com.cn
> 主题: 答复: [edk2-devel] Patch List for 202008 stable tag
> 
> Laszlo:
> 
> > -----邮件原件-----
> > 发件人: bounce+27952+64532+4905953+8761045@groups.io
> > [mailto:bounce+27952+64532+4905953+8761045@groups.io] 代表 Laszlo
> > Ersek
> > 发送时间: 2020年8月21日 19:07
> > 收件人: Gao, Liming <liming.gao@intel.com>; Leif Lindholm
> > <leif@nuviainc.com>; afish@apple.com; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Guptha, Soumya K
> > <soumya.k.guptha@intel.com>; Chang, Abner (HPS SW/FW Technologist)
> > <abner.chang@hpe.com>; Vladimir Olovyannikov
> > <vladimir.olovyannikov@broadcom.com>; Tom Lendacky
> > <thomas.lendacky@amd.com>
> > 抄送: devel@edk2.groups.io
> > 主题: Re: [edk2-devel] Patch List for 202008 stable tag
> >
> > On 08/20/20 13:30, Gao, Liming wrote:
> > > Hi Stewards and all:
> > >   I collect current patch lists in devel mail list. Those patch
> contributors
> > request to add them for 202008 stable tag. Because we have enter into
Soft
> > Feature Freeze, I want to collect your feedback for them. If any patches
> are
> > missing, please reply this mail to add them.
> > >
> > > Feature List:
> > > https://edk2.groups.io/g/devel/message/63767 [PATCH]
> > > EmbeddedPkg/libfdt: Add strncmp macro to use AsciiStrnCmp [Liming]
> This
> > patch pass code review after Soft Feature Freeze (SFF) starts. According
> to SFF
> > definition, it should not be merged for this stable tag. But, the patch
> submitter
> > says this patch is important to RISC-V community. To catch it for this
> stable tag,
> > Laszlo proposed the solution to defer SFF start day from 2020-08-14 to
> > 2020-08-21, then hard feature freeze and release date will also defer
one
> > week. Any patches those pass review before new SFF start day can be
> merged.
> > @ Stewards, please give your comments to defer this stable tag release
by
> > one week.
> >
> > Thank you, very nice summary.
> >
> > As stated earlier, I'm OK with 1 week (or even two weeks, if needed)
> > extension. I invite Andrew, Leif and Mike to comment.
> >
> [Liming] I don't see any other urgent request. I think it is enough to
defer
> 1 week for this stable tag.
> So far, there is objection for this defer. I will send the announcement
> mail.
> 
> > >
> > > https://edk2.groups.io/g/devel/message/63348 [PATCH v5 1/1]
> > > ShellPkg/DynamicCommand: add HttpDynamicCommand [Liming] This
> > patch has collected the review comment. New version will be sent. I have
> no
> > information how important it is. @Vladimir, does this patch must catch
> this
> > stable tag? If yes, can you give the reason?
> >
> > This is a feature addition. While it's a super useful feature, it needs
to
> mature
> > (mainly from the coding style perspective, as I gather).
> > Posting of new versions and review are on-going.
> >
> > Most likely material for the next stable tag. In case we extend the
> deadlines,
> > and Maciej and the ShellPkg maintainers approve one of the upcoming
> > versions until the new SFF, then yes, we can merge that too.
> >
> [Liming] So, this is not urgent. Let's defer it to next stable tag.
> 
> Thanks
> Liming
> 
> > > Bug List:
> > > https://edk2.groups.io/g/devel/message/64383 [PATCH 1/1]
> > > UefiCpuPkg/MpInitLib: Always initialize the DoDecrement variable
> [Liming]
> > This patch has collected the review comment. New version will be sent.
> >
> > This is a bugfix. Certainly a build fix on CLANGPDB. I tried to do some
> > (lightweight) analysis to see whether the bug is a functional one as
well
> (i.e.
> > whether the bad code path that CLANG warns about can actually occur in
> > practice), but in the end I didn't want to spend much time on it, as the
> build
> > breakage needs to be fixed anyway.
> >
> > So, this patch (more precisely, v2 of this patch) would even qualify
> during the
> > hard feature freeze (as it's clearly a bugfix).
> >
> > >
> > > https://edk2.groups.io/g/devel/message/50406 [PATCH 1/1]
> > > MdePkg/Include: Add missing definitions of SMBIOS type 42h in SmBios.h
> > [Liming] This patch passed review early. But, it is not merged. I will
> merge it.
> >
> > Right, it was approved on 2019-Nov-18. If it still applies cleanly, it
> should be
> > merged, before we enter the Hard Feature Freeze.
> >
> > Well... now that I'm writing this, we have already entered the HFF (as
> planned
> > originally) and the patch doesn't seem to be upstream yet. So unless we
> > extend, I don't think this patch qualifies.
> >
> > (On the other hand -- it seems like we really should extend.)
> >
> > Thanks!
> > Laszlo
> >
> >
> >
> 
> 
> 
> 
> 
> 




  parent reply	other threads:[~2020-09-01  9:35 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-20 11:30 Patch List for 202008 stable tag Liming Gao
2020-08-20 12:43 ` Lendacky, Thomas
2020-08-20 15:41   ` Vladimir Olovyannikov
2020-08-21 11:10     ` Laszlo Ersek
2020-08-21 16:26       ` Vladimir Olovyannikov
2020-08-22  0:09         ` 答复: [edk2-devel] " gaoliming
2020-08-21  3:21 ` Abner Chang
2020-08-21  4:11   ` [edk2-devel] " Liming Gao
2020-08-21  4:14     ` Abner Chang
2020-08-21 11:06 ` Laszlo Ersek
2020-08-22  0:15   ` 答复: [edk2-devel] " gaoliming
     [not found]   ` <162D6EE567916534.7353@groups.io>
2020-09-01  9:35     ` gaoliming [this message]
2020-09-01 12:33       ` 回复: " Laszlo Ersek
2020-09-01 14:57         ` 回复: " gaoliming
2020-09-01 15:17           ` Laszlo Ersek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='001b01d68043$44cda3e0$ce68eba0$@byosoft.com.cn' \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox