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


Reply via email to