[edk2-devel] Soft Feature Freeze start date delays to 2020-08-24 for edk2-stable202008
On 08/26/20 03:19, Yao, Jiewen wrote:
To clarify below:The series in question has three SecurityPkg patches:
[PATCH v4 1/8] SecurityPkg/TcgEventLogRecordLib: add new lib for firmware measurement
[PATCH v4 5/8] SecurityPkg/dsc: add FvEventLogRecordLib
[PATCH v4 7/8] SecurityPkg/Tcg2: handle PRE HASH and LOG ONLY
As I'm writing this, *none* of the listed patches have any kind of
Reviewed-by or Acked-by, either included in the patches themselves, or
posted in response to them.
I request to check in to stable202008, if possible.We can do that only if (a) we extend the SFF deadline again, and (b)
each of the SecurityPkg patches receives at least an Acked-by from one
of the SecurityPkg maintainers, until the new deadline.
I'm certainly not against the idea. I don't mind if the release slips
some more; it's OK to say that we're not ready to release yet. The point
is, as long as we're doing more work for completing the release, we
should prolong the stabilization period as well (opportunity for people
On 08/26/20 12:16, Yao, Jiewen wrote:
HI LaszloYou are correct:
I have two comments on this.
First, because you authored the IntelFsp2WrapperPkg patches in the
series, you cannot R-b them (you cannot R-b your own patches, even if
they are resent by someone else). However, that's not necessary: the
IntelFsp2WrapperPkg is maintained by Chasel Chiu, and Chasel did review
those patches, under v4, in the end.
Second, the v4 submitter, Qi Zhang, should have picked up your R-b from
under v3, and included them in the v4 posting. (Assuming the v3->v4
changes were exactly as you requested, under v3.)
Qi sent v4 series in August 17, with only naming change from FvEventLogRecordLib to TcgEventLogRecordLib.OK. In this case, Qi should have posted the v4 SecurityPkg patches with
your R-b *already* present.
Jian replied "[PATCH v3 0/8] Need add a FSP binary measurement" with reviewed-by on V3 patch series in August 18.That's correct too:
This means that Qi should have sent v4 with Jian's R-b on *every* patch.
So I treat this patch series is qualified to check in (since V4 adopted my comment). But please let me know if there is any misunderstanding.No, you are entirely correct. I was misled because v4 was not posted
correctly, with regard to the feedback tags given under v3.
So, what remains to do now is this: until the HFF (2020-08-28) we can,
and should, merge v4 of the series. As follows:
- apply Jian's R-b from under v3 to every patch in the series
- apply your R-b from under v3 to the patches you did *not* author (that
is, apply the tag to the SecurityPkg patches, plus to
"IntelFsp2WrapperPkg/dsc: add HashLib, Tpm2CommandLib and Tpm2DeviceLib")
- apply Chasel Chiu's R-b from under v4 to the IntelFsp2WrapperPkg patches.
This will make the series fully reviewed, and mergeable.
Note that Chasel requested a copyright year update when pushing, here:
<https://edk2.groups.io/g/devel/message/64382>. Given that Chasel
(maintainer/reviewer), Jiewen (original author) and Qi (poster) all work
for Intel, and the (C) notice in question is Intel's, I think that *any*
maintainer can satisfy Chasel's request, when merging the series.
So, I think I'll go ahead and merge v4. Thank you for the v3 pointers.
When I am about to merge, I am told that we are in SFF and I cannot check in.I'm going to merge the series for you, given the amount of work needed
for collecting the feedback tags.