public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Abner Chang" <abner.chang@hpe.com>
To: Laszlo Ersek <lersek@redhat.com>, Leif Lindholm <leif@nuviainc.com>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	liming.gao <liming.gao@intel.com>,
	"announce@edk2.groups.io" <announce@edk2.groups.io>,
	"afish@apple.com" <afish@apple.com>,
	"Kinney, Michael D" <michael.d.kinney@intel.com>
Subject: Re: [edk2-announce] Re: Soft Feature Freeze starts now for edk2-stable202008
Date: Wed, 19 Aug 2020 14:58:41 +0000	[thread overview]
Message-ID: <CS1PR8401MB1144794B96BF9DCB4C14B231FF5D0@CS1PR8401MB1144.NAMPRD84.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <dcef2e3a-0bb7-10a8-0702-11d4ddbb9dff@redhat.com>



> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Wednesday, August 19, 2020 10:30 PM
> To: Chang, Abner (HPS SW/FW Technologist) <abner.chang@hpe.com>; Leif
> Lindholm <leif@nuviainc.com>
> Cc: devel@edk2.groups.io; liming.gao <liming.gao@intel.com>;
> announce@edk2.groups.io; afish@apple.com; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Subject: Re: [edk2-announce] Re: Soft Feature Freeze starts now for edk2-
> stable202008
> 
> On 08/19/20 15:34, Chang, Abner (HPS SW/FW Technologist) wrote:
> >
> >
> >> -----Original Message-----
> >> From: Laszlo Ersek [mailto:lersek@redhat.com]
> >> Sent: Wednesday, August 19, 2020 9:19 PM
> >> To: Leif Lindholm <leif@nuviainc.com>; Chang, Abner (HPS SW/FW
> >> Technologist) <abner.chang@hpe.com>
> >> Cc: devel@edk2.groups.io; liming.gao <liming.gao@intel.com>;
> >> announce@edk2.groups.io; afish@apple.com; Kinney, Michael D
> >> <michael.d.kinney@intel.com>
> >> Subject: Re: [edk2-announce] Re: Soft Feature Freeze starts now for
> >> edk2-
> >> stable202008
> >>
> >> On 08/19/20 13:48, Leif Lindholm wrote:
> >>> (Slightly trimmed recipient list due to different patch being
> >>> discussed.)
> >>>
> >>> So, I can't make this call, because I'm the one who messed up.
> >>>
> >>> This patch does exactly what I had requested Abner to do some time
> >>> back (off-list, unfortunately), and I was *convinced* I gave it an
> >>> R-b as soon as it hit my inbox - until Abner nudged me about it yesterday.
> >>>
> >>> The patch in question is
> >>> https://edk2.groups.io/g/devel/topic/76021725
> >>
> >> My understanding is:
> >>
> >> (1) there is an external project that consumes the FDT library in
> >> EmbeddedPkg, meaning the lib class header
> >> "EmbeddedPkg/Include/libfdt.h"
> >> and the lib instance "EmbeddedPkg/Library/FdtLib/FdtLib.inf",
> > [Chang, Abner] Yes
> >>
> >> (2) the lib class header pulls in "fdt.h" and "libfdt_env.h",
> > [Chang, Abner] yes
> >>
> >> (3) the external project is not edk2-platforms,
> > [Chang, Abner] yes
> >>
> >> (4) the external project wants -- for some strange reason -- edk2's
> >> "libfdt_env.h" to provide an strncmp() function (or function-like
> >> macro), with that particular stncmp() implementation not being needed
> >> in either edk2- platforms or edk2 itself,
> > [Chang, Abner] yes, at least so far
> >>
> >> (5) the patch for adding said strncmp() was posted on Aug 6th (at
> >> least when viewed from my time zone), i.e., before the SFF,
> > [Chang, Abner] Yes
> >>
> >> (6) it was reviewed 12 days later (within the SFF)
> > [Chang, Abner] yes.
> >>
> >> If my understanding is correct, then I don't see how this patch could
> >> be considered a bugfix -- even as a feature addition, it seems hardly
> >> justified to me --, and there would have been ~8 days before the SFF to
> review it.
> >>
> >> I think we should postpone the patch until after the stable tag.
> > This patch is important because the edk2-stable202008 would be the stable
> tag (if this patch is accepted) for booting RISC-V platform to Linux kernel with
> EFI Runtime service on either real platform and QEMU. We can publish this
> information in RISC-V community which is considered as a valuable milestone
> for RISC-V edk2 port.
> 
> Let's move out the dates for the stable tag then, by one week:
> 
> - let the SFF start on 2020-08-21
> - let the HFF start on 2020-08-28
> - let's release edk2-stable202008 on 2020-09-04
> 
> Release slips are permitted and there have been examples.
Appreciate for this.

> 
> What doesn't make sense is making rules and then breaking them
> opportunistically, whenever they're uncomfortable. If that's a frequent
> occurrence, we should pick different rules, or -- again -- if this is a very
> important patch, we should delay the release for it.
> 
> BTW what about reverting the OpenSBI change? You could still call
> sbi_strncmp() -- rather than strncmp() -- in the "helper" lib.
I replied this in another thread.
> 
> Thanks
> Laszlo


  reply	other threads:[~2020-08-19 14:58 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-14  8:16 Soft Feature Freeze starts now for edk2-stable202008 Liming Gao
2020-08-17 18:14 ` Bret Barkelew
2020-08-17 20:46   ` Laszlo Ersek
2020-08-17 21:33     ` [EXTERNAL] " Bret Barkelew
2020-08-18 10:29       ` Laszlo Ersek
2020-08-18 15:10         ` Bret Barkelew
2020-08-19  9:59           ` Laszlo Ersek
2020-08-19 11:29             ` [edk2-announce] " Abner Chang
2020-08-19 11:48               ` Leif Lindholm
2020-08-19 13:19                 ` Laszlo Ersek
2020-08-19 13:34                   ` Abner Chang
2020-08-19 14:29                     ` Laszlo Ersek
2020-08-19 14:58                       ` Abner Chang [this message]
2020-08-19 16:20                         ` Liming Gao
2020-08-19 17:18                           ` Laszlo Ersek
2020-08-19 13:36                   ` Laszlo Ersek
2020-08-19 14:56                     ` Abner Chang
2020-08-19 13:25                 ` Abner Chang

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=CS1PR8401MB1144794B96BF9DCB4C14B231FF5D0@CS1PR8401MB1144.NAMPRD84.PROD.OUTLOOK.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