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 (#119387): https://edk2.groups.io/g/devel/message/119387 Mute This Topic: https://groups.io/mt/106379971/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
