Hi Jiewen, Thanks for your patience. I rebased this PR again since CI failed after "merge branch". And it is ready to go now.
Per my limited understanding, it should be OK to add the "push" label to merge a PR to upstream even the PR shows "This branch is out-of-date with the base branch" as long as there are no merge conflicts. - Example of adding "push" label to an out-of-date PR: https://github.com/tianocore/edk2/pull/5461 - Example of adding "push" label to an up-to-date PR: https://github.com/tianocore/edk2/pull/5460 BRs, Lin, Du -----Original Message----- From: Yao, Jiewen <[email protected]> Sent: Thursday, May 30, 2024 6:31 PM To: Lin, Du <[email protected]>; [email protected]; gaoliming <[email protected]> Cc: 'Ard Biesheuvel' <[email protected]>; 'Gerd Hoffmann' <[email protected]> Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes It is out of date again. When I click rebase, I notice git uses "merge branch". It seems not what we want. I am not sure how to handle it. Next time, please tell me ASAP once you submit a new PR. > -----Original Message----- > From: Lin, Du <[email protected]> > Sent: Thursday, May 30, 2024 6:26 PM > To: Yao, Jiewen <[email protected]>; [email protected]; > gaoliming <[email protected]> > Cc: 'Ard Biesheuvel' <[email protected]>; 'Gerd Hoffmann' > <[email protected]>; Lin, Du <[email protected]> > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check > to support new resource attributes > > Thanks Jiewen. A new PR has been submitted for this: > https://github.com/tianocore/edk2/pull/5699. > > BRs, > Lin, Du > > -----Original Message----- > From: Yao, Jiewen <[email protected]> > Sent: Thursday, May 30, 2024 8:00 AM > To: [email protected]; Yao, Jiewen <[email protected]>; > gaoliming <[email protected]>; Lin, Du <[email protected]> > Cc: 'Ard Biesheuvel' <[email protected]>; 'Gerd Hoffmann' > <[email protected]> > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check > to support new resource attributes > > Lin Du > Just FYI: The CI failed after I rebase. > > If possible, I recommend to submit a new PR to get it resolved. > Then I can approve again. > > > > -----Original Message----- > > From: [email protected] <[email protected]> On Behalf Of Yao, > > Jiewen > > Sent: Monday, May 27, 2024 3:13 PM > > To: gaoliming <[email protected]>; [email protected]; Lin, > > Du <[email protected]> > > Cc: 'Ard Biesheuvel' <[email protected]>; 'Gerd Hoffmann' > > <[email protected]> > > Subject: Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check > > to support new resource attributes > > > > Thanks. Not urgent. Let’s wait. > > > > > > > -----Original Message----- > > > From: gaoliming <[email protected]> > > > Sent: Monday, May 27, 2024 3:12 PM > > > To: [email protected]; Yao, Jiewen <[email protected]>; Lin, > > > Du <[email protected]> > > > Cc: 'Ard Biesheuvel' <[email protected]>; 'Gerd Hoffmann' > > > <[email protected]> > > > Subject: 回复: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list > > > check to support new resource attributes > > > > > > Jiewen: > > > If the patch is urgent to be merged, I will help add push label > > > to merge it with current process. > > > > > > If the patch is not urgent, it can be merged after TianoCore > > > Code Review is switched from email to GitHub Pull Requests on US Tuesday. > > > > > > Thanks > > > Liming > > > > -----邮件原件----- > > > > 发件人: [email protected] <[email protected]> 代表 Yao, Jiewen > > > > 发送时间: 2024年5月27日 14:43 > > > > 收件人: Lin, Du <[email protected]>; [email protected] > > > > 抄送: Ard Biesheuvel <[email protected]>; Gerd Hoffmann > > > > <[email protected]> > > > > 主题: Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check > > > > to > > > support > > > > new resource attributes > > > > > > > > I have approved it. > > > > > > > > What is the process to merge? There is no COMMIT button or PUSH label. > > > > > > > > > > > > > -----Original Message----- > > > > > From: Lin, Du <[email protected]> > > > > > Sent: Monday, May 27, 2024 2:23 PM > > > > > To: [email protected] > > > > > Cc: Ard Biesheuvel <[email protected]>; Gerd Hoffmann > > > > > <[email protected]>; Yao, Jiewen <[email protected]>; Lin, > > > > > Du <[email protected]> > > > > > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list > > > > > check to support new resource attributes > > > > > > > > > > Thanks for the review and approval. Could you please help > > > > > merge this > > > patch to > > > > > the upstream? A pull request has been created for this patch: > > > > > https://github.com/tianocore/edk2/pull/5644. Thanks. > > > > > > > > > > BRs, > > > > > Lin, Du > > > > > > > > > > -----Original Message----- > > > > > From: Yao, Jiewen <[email protected]> > > > > > Sent: Thursday, May 16, 2024 5:37 PM > > > > > To: [email protected]; Lin, Du <[email protected]> > > > > > Cc: Ard Biesheuvel <[email protected]>; Gerd Hoffmann > > > > > <[email protected]> > > > > > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list > > > > > check to support new resource attributes > > > > > > > > > > Reviewed-by: Jiewen Yao <[email protected]> > > > > > > > > > > > -----Original Message----- > > > > > > From: [email protected] <[email protected]> On Behalf > > > > > > Of Lin, > > Du > > > > > > Sent: Thursday, May 9, 2024 1:27 PM > > > > > > To: [email protected] > > > > > > Cc: Lin, Du <[email protected]>; Ard Biesheuvel > > > > > > <[email protected]>; Gerd Hoffmann > > > > > > <[email protected]>; > > Yao, > > > > > > Jiewen <[email protected]> > > > > > > Subject: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list > > > > > > check to support new resource attributes > > > > > > > > > > > > Encrypted and Special Purpose resource attributes are > > > > > > introduced in PI > > > > > > 1.8 Specification. This patch is to update VMM Hob list > > > > > > integrity check to recognise these resource attributes. > > > > > > > > > > > > Cc: Ard Biesheuvel <[email protected]> > > > > > > Cc: Gerd Hoffmann <[email protected]> > > > > > > Cc: Jiewen Yao <[email protected]> > > > > > > Signed-off-by: Du Lin <[email protected]> > > > > > > --- > > > > > > OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c | 2 ++ > > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > > > diff --git a/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c > > > > > > b/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c > > > > > > index b6085eab44..19e9b1bf54 100644 > > > > > > --- a/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c > > > > > > +++ b/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c > > > > > > @@ -643,6 +643,8 @@ ValidateHobList ( > > > > > > > > > > > > EFI_RESOURCE_ATTRIBUTE_PERSISTABLE | > > > > > > > > > > > > EFI_RESOURCE_ATTRIBUTE_READ_ONLY_PROTECTED | > > > > > > > > > > > > EFI_RESOURCE_ATTRIBUTE_READ_ONLY_PROTECTABLE | > > > > > > + > > > > > > + EFI_RESOURCE_ATTRIBUTE_ENCRYPTED| > > > > > > + > > > > > > EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE | > > > > > > > > > > > > EFI_RESOURCE_ATTRIBUTE_MORE_RELIABLE))) != 0) > > > > > > { > > > > > > DEBUG ((DEBUG_ERROR, "HOB: Unknow > > > > > > ResourceDescriptor ResourceAttribute type. Type: 0x%08x\n", > > > > > > Hob.ResourceDescriptor- > > > > > > >ResourceAttribute)); > > > > > > -- > > > > > > 2.44.0.windows.1 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119393): https://edk2.groups.io/g/devel/message/119393 Mute This Topic: https://groups.io/mt/106379971/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
