public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Laszlo Ersek" <lersek@redhat.com>
To: "Gao, Liming" <liming.gao@intel.com>,
	Leif Lindholm <leif@nuviainc.com>,
	"afish@apple.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>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>
Subject: Re: Patch List for 202008 stable tag
Date: Fri, 21 Aug 2020 13:06:47 +0200	[thread overview]
Message-ID: <55f8dfee-ef2e-b04a-f1cb-4fef1931a7c3@redhat.com> (raw)
In-Reply-To: <CY4PR11MB1526ECFA1B8EAFF7FD601D5F805A0@CY4PR11MB1526.namprd11.prod.outlook.com>

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.

> 
> 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.

> 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-08-21 11:06 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 [this message]
2020-08-22  0:15   ` 答复: " gaoliming
     [not found]   ` <162D6EE567916534.7353@groups.io>
2020-09-01  9:35     ` 回复: " gaoliming
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=55f8dfee-ef2e-b04a-f1cb-4fef1931a7c3@redhat.com \
    --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