From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web10.18050.1600245770999671117 for ; Wed, 16 Sep 2020 01:42:51 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=EhRKjHRI; spf=pass (domain: redhat.com, ip: 207.211.31.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1600245770; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Jbg3cAFj+LFGnet6m0B/X1UWwyk6IF8NJhNbZ+/IpQw=; b=EhRKjHRIAemTBYNoXYewRVEDBjHyChkXB9rOSchXh3YKmW2yfmNGgP5+Ozj9ycBtmGb8DN eItOI5Ot+pqDgzJ7nRZMNM1syMU5pV5OzIV/HVuJWXSQDxllML/y6iGT5HwEfAdWm3SC3h 8Vm0y5y8CAljg7ID1BCxA4Ek/o1W7A4= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-323-iIjozTlaONWo-DsX75D-lQ-1; Wed, 16 Sep 2020 04:42:42 -0400 X-MC-Unique: iIjozTlaONWo-DsX75D-lQ-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 82EEB8030AE; Wed, 16 Sep 2020 08:42:40 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-113-213.ams2.redhat.com [10.36.113.213]) by smtp.corp.redhat.com (Postfix) with ESMTP id ED76281C46; Wed, 16 Sep 2020 08:42:38 +0000 (UTC) Subject: development process failure [was: remove TPM related ppi from Depex for Fsp wrapper PEIM driver] To: Chasel Chiu , Jiewen Yao Cc: devel@edk2.groups.io, qi1.zhang@intel.com, Nate DeSimone , Star Zeng , Jian J Wang References: <20200915062109.3225-1-qi1.zhang@intel.com> From: "Laszlo Ersek" Message-ID: <8ff2b344-c725-0b1a-0632-f1d589cbb326@redhat.com> Date: Wed, 16 Sep 2020 10:42:38 +0200 MIME-Version: 1.0 In-Reply-To: <20200915062109.3225-1-qi1.zhang@intel.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0.002 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Content-Language: en-US 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 > Cc: Nate DeSimone > Cc: Star Zeng > Cc: Jiewen Yao > Cc: Jian J Wang > > 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 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