public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Laszlo Ersek" <lersek@redhat.com>
To: Chasel Chiu <chasel.chiu@intel.com>, Jiewen Yao <jiewen.yao@intel.com>
Cc: devel@edk2.groups.io, qi1.zhang@intel.com,
	Nate DeSimone <nathaniel.l.desimone@intel.com>,
	Star Zeng <star.zeng@intel.com>,
	Jian J Wang <jian.j.wang@intel.com>
Subject: development process failure [was: remove TPM related ppi from Depex for Fsp wrapper PEIM driver]
Date: Wed, 16 Sep 2020 10:42:38 +0200	[thread overview]
Message-ID: <8ff2b344-c725-0b1a-0632-f1d589cbb326@redhat.com> (raw)
In-Reply-To: <20200915062109.3225-1-qi1.zhang@intel.com>

Jiewen, Chasel,

On 09/15/20 08:21, Qi Zhang wrote:
> Some open board are TPM disabled. So the boot may hang because
>  these PPIs can't arrive. And gEdkiiTcgPpiGuid will be notified where
>   it is used. So we need to remove these PPIs from Depex for Fsp wrapper
>   PEI and PeiTpmMeasurementLib.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> 
> Qi Zhang (2):
>   IntelFsp2WrapperPkg: remove gPeiTpmInitializationDonePpiGuid from
>     Depex
>   SecurityPkg/PeiTpmMeasurementLib: remove gEfiTpmDeviceSelectedGuid
> 
>  IntelFsp2WrapperPkg/FspmWrapperPeim/FspmWrapperPeim.inf        | 3 +--
>  IntelFsp2WrapperPkg/FspsWrapperPeim/FspsWrapperPeim.inf        | 3 +--
>  .../Library/PeiTpmMeasurementLib/PeiTpmMeasurementLib.inf      | 3 +--
>  3 files changed, 3 insertions(+), 6 deletions(-)
> 

Please adopt a *much more* disciplined approach when merging patch series.


(1) When you merge a patch set, please report back on the list. Identify
both the pull request URL, and the commit reange.

In this case, the pull request was

  https://github.com/tianocore/edk2/pull/930

and the commit range is a62fb4229d14..7bcb021a6d54.


(2) The associated Bugzilla:

  https://bugzilla.tianocore.org/show_bug.cgi?id=2963

has been completely neglected, by both submitter and maintainers.

- The original BZ report is *absolute trash*.

- No URL into the mailing list archive has been captured in the BZ,
about the posted series.

- The BZ status is still CONFIRMED.

- No mention of the pull request, or the resultant commit, range in the
BZ ticket.


(3) The github pull request at
<https://github.com/tianocore/edk2/pull/930> does contain *any*
indication of the bugzilla ticket, or the cover letter on the list.

Basically we have random artifacts in three different places (Bugzilla,
github.com, mailing list), and nobody of the involved parties
(reviewers, maintainers, constributors) on this patch set have made
*any* effort to cross-reference them. We now have to hunt down
everything separately.


(4) Worst of all, the subject line of commit 414d7d11e6ea contains a
Unicode code point called FULLWIDTH COLON (U+FF1A) rather than a normal
colon (U+003A).

Compare:

- bad (current):           IntelFsp2WrapperPkg: remove [...]
- good (should have been): IntelFsp2WrapperPkg: remove [...]

It makes absolutely no sense to use non-ASCII code points in subject
lines, for something as trivial as a colon.


I've been here for 8-9 years now and it's incredibly frustrating that I
*still* have to whine about basic stuff like this on a regular basis.

I don't even know whom I should CC at Intel (management or otherwise) to
see an improvement in attitude here.

I guess this community cannot be saved.

Laszlo


  parent reply	other threads:[~2020-09-16  8:42 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-15  6:21 [PATCH 0/2] remove TPM related ppi from Depex for Fsp wrapper PEIM driver Qi Zhang
2020-09-15  6:21 ` [PATCH 1/2] IntelFsp2WrapperPkg: remove gPeiTpmInitializationDonePpiGuid from Depex Qi Zhang
2020-09-16  2:07   ` Chiu, Chasel
2020-09-15  6:21 ` [PATCH 2/2] SecurityPkg/PeiTpmMeasurementLib: remove gEfiTpmDeviceSelectedGuid Qi Zhang
2020-09-16  2:09   ` [edk2-devel] " Yao, Jiewen
2020-09-16  8:42 ` Laszlo Ersek [this message]
2020-09-16  9:31   ` development process failure [was: remove TPM related ppi from Depex for Fsp wrapper PEIM driver] Yao, Jiewen
2020-09-16 12:13     ` Laszlo Ersek
2020-09-16 12:30       ` Yao, Jiewen
2020-09-16  9:34 ` [PATCH 0/2] remove TPM related ppi from Depex for Fsp wrapper PEIM driver Yao, Jiewen

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=8ff2b344-c725-0b1a-0632-f1d589cbb326@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