public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Laszlo Ersek" <lersek@redhat.com>
To: Leif Lindholm <leif@nuviainc.com>,
	devel@edk2.groups.io, liming.gao@intel.com
Cc: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"afish@apple.com" <afish@apple.com>,
	"Guptha, Soumya K" <soumya.k.guptha@intel.com>,
	"Marvin Häuser" <marvin.haeuser@outlook.com>,
	"Gao, Zhichao" <zhichao.gao@intel.com>,
	"'ard.biesheuvel@linaro.org'" <ard.biesheuvel@linaro.org>,
	"Wu, Hao A" <hao.a.wu@intel.com>,
	vit9696 <vit9696@protonmail.com>,
	"gaurav.jain@nxp.com" <gaurav.jain@nxp.com>,
	"Ni, Ray" <ray.ni@intel.com>,
	"Feng, Bob C" <bob.c.feng@intel.com>,
	"maciej.rabeda@linux.intel.com" <maciej.rabeda@linux.intel.com>,
	"leo.duran@amd.com" <leo.duran@amd.com>
Subject: Re: [edk2-devel] Patch List for 202002 stable tag
Date: Thu, 27 Feb 2020 18:25:05 +0100	[thread overview]
Message-ID: <51eae9d3-b891-2685-186b-023a82bdebcc@redhat.com> (raw)
In-Reply-To: <20200227162318.GC23627@bivouac.eciton.net>

On 02/27/20 17:23, Leif Lindholm wrote:
> Hi Liming,
> 
> On Thu, Feb 27, 2020 at 16:06:22 +0000, Liming Gao wrote:
>> Stewards:
>>   I update the patch lists and status. Based on current information,
>>   only one patch (item 5) needs catch this stable tag. Its fix is
>>   clear, and risk is low. So, I think we can still keep current
>>   planning to create stable tag edk2-stable202002 on 2020 Feb 28th
>>   (UTC – 8). If you think the stable tag needs to be delay for few
>>   days, please reply the mail before Feb 28th (00:00:00 UTC-8).
>>
>>   1.  https://edk2.groups.io/g/devel/message/54665 [edk2-devel] [PATCH v2 1/1] EmbeddedPkg: Fixed Asserts in SCT Runtime Services test.
>> [Liming] This patch is still under review. So, it will not catch this stable tag.
>>
>>   1.  https://edk2.groups.io/g/devel/message/54693 [edk2-stable202002][edk2-devel] [PATCH v2 1/1] MdeModulePkg/Pci: Fixed Asserts in SCT PCIIO Protocol Test.
> 
> Unrelated to the release process, only the formatting:
> It looks like you are doing ordered lists using markdown syntax
> (1.). This renders in plain text email simply as all items being 1.
> 
>> [Liming] The patch has passed review. Package maintainer thinks this is an enhancement. It will be added after stable tag.
>>
>>   1.  https://edk2.groups.io/g/devel/message/54122 [PATCH 1/1] ShellPkg: Add support for input with separately reported modifiers
>> [Liming] The discussion shows this change needs UEFI spec clarification. So, it may not be resolved in short team. It will not catch this stable tag. The discussion is in BZ 2510.
>>
>>   1.  https://edk2.groups.io/g/devel/message/54797 [PATCH 0/2] UefiCpuPkg/Library: Fix bug in MpInitLib
>> [Liming] The solution is under discussion (BZ 2556). The submitter requests this issue to be fixed happen reasonably soon. So, it may not catch this stable tag.
>>
>>   1.  https://edk2.groups.io/g/devel/message/54992 [Patch 1/1][edk2-stable202002]BaseTools: Fixed a regression issue in Makefile for incremental build
>> [Liming] This patch has passed review. This regression causes the basic incremental build not work with VS nmake tool. The fix is clear. So, it need to catch this stable tag.
> 
> I agree it needs to catch the stable tag. If it affects only VS builds
> then I am not going to insist on extending the hard freeze, but I
> (technically on holiday today/tomorrow) don't have time to dig much
> deeper into it.
> 
> However, I think the process is pretty clear that this *should* extend
> the hard freeze.
> 
> I will note that from the trail (commitdate of 818283de3f6d until
> BZ2563 was raised) it appears that detecting this bug itself, which
> went in two days before the soft freeze, took 15 days.

I agree with Liming's analysis on the patches (i.e., what goes in and
what gets postponed), and I agree with Leif that we should extend the
hard freeze by at least a couple of days.

This is not unusual. Originally I thought that edk2 freeze and release
dates were set in stone, but then Mike explained to me that that had
never been the intent. And other open source projects do several
pre-releases (rc0, rc1, .... pre-releases with "release critical" (rc)
bug fixes), before a final release. For example, QEMU regularly plans
rc0..rc2 or even rc3, and then *optionally* adds an rc4 if even rc3
receives significant bugfixes. The idea is that the final release / tag
should be preceded by a silent / calm period, where we've waited a few
days and become reasonably convinced that "OK, there's nothing else we
should obviously fix right now".

I wouldn't immediately suggest a full week extension, but maybe until
March 4th (middle of next week)?

Thanks
Laszlo


  reply	other threads:[~2020-02-27 17:25 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-02-18 14:08 Patch List for 202002 stable tag Liming Gao
2020-02-18 20:04 ` Laszlo Ersek
2020-02-18 20:42   ` Michael D Kinney
2020-02-19  8:53     ` Laszlo Ersek
2020-02-19 15:39     ` Liming Gao
2020-02-19 18:09       ` Vitaly Cheptsov
2020-02-20  1:17         ` Liming Gao
2020-02-20  1:35           ` Gao, Zhichao
2020-02-20  3:13           ` Ni, Ray
2020-02-20  6:58             ` Liming Gao
2020-02-20  7:07               ` Vitaly Cheptsov
     [not found]             ` <15F50A1858BD174A.18319@groups.io>
2020-02-21  8:22               ` [edk2-devel] " Liming Gao
     [not found]               ` <15F55D425BF8837D.15709@groups.io>
2020-02-27 16:06                 ` Liming Gao
2020-02-27 16:23                   ` Leif Lindholm
2020-02-27 17:25                     ` Laszlo Ersek [this message]
2020-02-28  4:13                       ` Liming Gao
2020-02-28 12:48                         ` Leif Lindholm
2020-03-03  8:29                           ` Liming Gao
2020-03-03 11:37                             ` 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=51eae9d3-b891-2685-186b-023a82bdebcc@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