Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-30 Thread Yao, Jiewen
merged

> -Original Message-
> From: Lin, Du 
> Sent: Thursday, May 30, 2024 8:07 PM
> To: Yao, Jiewen ; devel@edk2.groups.io; gaoliming
> 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> ; Lin, Du 
> Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to
> support new resource attributes
> 
> 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 
> Sent: Thursday, May 30, 2024 6:31 PM
> To: Lin, Du ; devel@edk2.groups.io; gaoliming
> 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> 
> 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 
> > Sent: Thursday, May 30, 2024 6:26 PM
> > To: Yao, Jiewen ; devel@edk2.groups.io;
> > gaoliming 
> > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > ; Lin, Du 
> > 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 
> > Sent: Thursday, May 30, 2024 8:00 AM
> > To: devel@edk2.groups.io; Yao, Jiewen ;
> > gaoliming ; Lin, Du 
> > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > 
> > 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: devel@edk2.groups.io  On Behalf Of Yao,
> > > Jiewen
> > > Sent: Monday, May 27, 2024 3:13 PM
> > > To: gaoliming ; devel@edk2.groups.io; Lin,
> > > Du 
> > > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > > 
> > > 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 
> > > > Sent: Monday, May 27, 2024 3:12 PM
> > > > To: devel@edk2.groups.io; Yao, Jiewen ; Lin,
> > > > Du 
> > > > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > > > 
> > > > 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
> > > > > -----邮件原件-----
> > > > > 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> > > > > 发送时间: 2024年5月27日 14:43
> > > > > 收件人: Lin, Du ; devel@edk2.groups.io
> > > > > 抄送: Ard Biesheuvel ; Gerd Hoffmann
> > > > > 
> > > > > 主题: 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.
> > > > >
> > > > >
> > &g

Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-30 Thread Du Lin
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  
Sent: Thursday, May 30, 2024 6:31 PM
To: Lin, Du ; devel@edk2.groups.io; gaoliming 

Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann' 

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 
> Sent: Thursday, May 30, 2024 6:26 PM
> To: Yao, Jiewen ; devel@edk2.groups.io; 
> gaoliming 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> ; Lin, Du 
> 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 
> Sent: Thursday, May 30, 2024 8:00 AM
> To: devel@edk2.groups.io; Yao, Jiewen ; 
> gaoliming ; Lin, Du 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> 
> 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: devel@edk2.groups.io  On Behalf Of Yao, 
> > Jiewen
> > Sent: Monday, May 27, 2024 3:13 PM
> > To: gaoliming ; devel@edk2.groups.io; Lin, 
> > Du 
> > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > 
> > 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 
> > > Sent: Monday, May 27, 2024 3:12 PM
> > > To: devel@edk2.groups.io; Yao, Jiewen ; Lin, 
> > > Du 
> > > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > > 
> > > 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
> > > > -邮件原件-
> > > > 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> > > > 发送时间: 2024年5月27日 14:43
> > > > 收件人: Lin, Du ; devel@edk2.groups.io
> > > > 抄送: Ard Biesheuvel ; Gerd Hoffmann 
> > > > 
> > > > 主题: 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 
> > > > > Sent: Monday, May 27, 2024 2:23 PM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Ard Biesheuvel ; Gerd Hoffmann 
> > > > > ; Yao, Jiewen ; Lin, 
> > > > > Du 
> > > > > 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
> > > > >
> &g

Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-30 Thread Yao, Jiewen
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 
> Sent: Thursday, May 30, 2024 6:26 PM
> To: Yao, Jiewen ; devel@edk2.groups.io; gaoliming
> 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> ; Lin, Du 
> 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 
> Sent: Thursday, May 30, 2024 8:00 AM
> To: devel@edk2.groups.io; Yao, Jiewen ; gaoliming
> ; Lin, Du 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> 
> 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: devel@edk2.groups.io  On Behalf Of Yao,
> > Jiewen
> > Sent: Monday, May 27, 2024 3:13 PM
> > To: gaoliming ; devel@edk2.groups.io; Lin,
> > Du 
> > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > 
> > 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 
> > > Sent: Monday, May 27, 2024 3:12 PM
> > > To: devel@edk2.groups.io; Yao, Jiewen ; Lin,
> > > Du 
> > > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > > 
> > > 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
> > > > -邮件原件-
> > > > 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> > > > 发送时间: 2024年5月27日 14:43
> > > > 收件人: Lin, Du ; devel@edk2.groups.io
> > > > 抄送: Ard Biesheuvel ; Gerd Hoffmann
> > > > 
> > > > 主题: 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 
> > > > > Sent: Monday, May 27, 2024 2:23 PM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > > > > ; Yao, Jiewen ; Lin, Du
> > > > > 
> > > > > 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 
> > > > > Sent: Thursday, May 16, 2024 5:37 PM
> > > > > To: devel@edk2.groups.io; Lin, Du 
> > > > > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > > > > 
> > > > > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list
> > > > > check to support new resource attributes
> > > > >
> > > > > Reviewed-by: Jiewen Yao 
> > > > >
> > > > > > -Original Message-
> > > > > > From: devel@edk2.groups.io  On Behalf Of
> > > > > > Lin,
> > Du
> > > > > > Sent: Thursday, May 9, 2024 1:27 PM
> > > > > > To: devel@edk2.groups.io
> > > > >

Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-30 Thread Du Lin
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  
Sent: Thursday, May 30, 2024 8:00 AM
To: devel@edk2.groups.io; Yao, Jiewen ; gaoliming 
; Lin, Du 
Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann' 

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: devel@edk2.groups.io  On Behalf Of Yao, 
> Jiewen
> Sent: Monday, May 27, 2024 3:13 PM
> To: gaoliming ; devel@edk2.groups.io; Lin, 
> Du 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> 
> 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 
> > Sent: Monday, May 27, 2024 3:12 PM
> > To: devel@edk2.groups.io; Yao, Jiewen ; Lin, 
> > Du 
> > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > 
> > 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
> > > -邮件原件-
> > > 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> > > 发送时间: 2024年5月27日 14:43
> > > 收件人: Lin, Du ; devel@edk2.groups.io
> > > 抄送: Ard Biesheuvel ; Gerd Hoffmann 
> > > 
> > > 主题: 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 
> > > > Sent: Monday, May 27, 2024 2:23 PM
> > > > To: devel@edk2.groups.io
> > > > Cc: Ard Biesheuvel ; Gerd Hoffmann 
> > > > ; Yao, Jiewen ; Lin, Du 
> > > > 
> > > > 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 
> > > > Sent: Thursday, May 16, 2024 5:37 PM
> > > > To: devel@edk2.groups.io; Lin, Du 
> > > > Cc: Ard Biesheuvel ; Gerd Hoffmann 
> > > > 
> > > > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list 
> > > > check to support new resource attributes
> > > >
> > > > Reviewed-by: Jiewen Yao 
> > > >
> > > > > -Original Message-
> > > > > From: devel@edk2.groups.io  On Behalf Of 
> > > > > Lin,
> Du
> > > > > Sent: Thursday, May 9, 2024 1:27 PM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Lin, Du ; Ard Biesheuvel 
> > > > > ; Gerd Hoffmann 
> > > > > ;
> Yao,
> > > > > Jiewen 
> > > > > 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 
> > > > > Cc: Gerd Hoffmann 
> > > > > Cc: Jiewen Yao 
> > > > > Signed-off-by: Du Lin 
> > > > > ---
> > > > >  OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c | 2 ++
> > > > >  1 file changed, 2 insertions(+)
> > > > >
> > > > > diff --git a/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c
> > > > >

Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-29 Thread Yao, Jiewen
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: devel@edk2.groups.io  On Behalf Of Yao, Jiewen
> Sent: Monday, May 27, 2024 3:13 PM
> To: gaoliming ; devel@edk2.groups.io; Lin, Du
> 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> 
> 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 
> > Sent: Monday, May 27, 2024 3:12 PM
> > To: devel@edk2.groups.io; Yao, Jiewen ; Lin, Du
> > 
> > Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> > 
> > 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
> > > -邮件原件-
> > > 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> > > 发送时间: 2024年5月27日 14:43
> > > 收件人: Lin, Du ; devel@edk2.groups.io
> > > 抄送: Ard Biesheuvel ; Gerd Hoffmann
> > > 
> > > 主题: 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 
> > > > Sent: Monday, May 27, 2024 2:23 PM
> > > > To: devel@edk2.groups.io
> > > > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > > > ; Yao, Jiewen ; Lin, Du
> > > > 
> > > > 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 
> > > > Sent: Thursday, May 16, 2024 5:37 PM
> > > > To: devel@edk2.groups.io; Lin, Du 
> > > > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > > > 
> > > > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to
> > > > support new resource attributes
> > > >
> > > > Reviewed-by: Jiewen Yao 
> > > >
> > > > > -Original Message-
> > > > > From: devel@edk2.groups.io  On Behalf Of Lin,
> Du
> > > > > Sent: Thursday, May 9, 2024 1:27 PM
> > > > > To: devel@edk2.groups.io
> > > > > Cc: Lin, Du ; Ard Biesheuvel
> > > > > ; Gerd Hoffmann ;
> Yao,
> > > > > Jiewen 
> > > > > 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 
> > > > > Cc: Gerd Hoffmann 
> > > > > Cc: Jiewen Yao 
> > > > > Signed-off-by: Du Lin 
> > > > > ---
> > > > >  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 (#119368): https://edk2.groups.io/g/devel/message/119368
Mute This Topic: https://groups.io/mt/106379971/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-27 Thread Yao, Jiewen
Thanks. Not urgent. Let’s wait.


> -Original Message-
> From: gaoliming 
> Sent: Monday, May 27, 2024 3:12 PM
> To: devel@edk2.groups.io; Yao, Jiewen ; Lin, Du
> 
> Cc: 'Ard Biesheuvel' ; 'Gerd Hoffmann'
> 
> 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
> > -邮件原件-
> > 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> > 发送时间: 2024年5月27日 14:43
> > 收件人: Lin, Du ; devel@edk2.groups.io
> > 抄送: Ard Biesheuvel ; Gerd Hoffmann
> > 
> > 主题: 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 
> > > Sent: Monday, May 27, 2024 2:23 PM
> > > To: devel@edk2.groups.io
> > > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > > ; Yao, Jiewen ; Lin, Du
> > > 
> > > 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 
> > > Sent: Thursday, May 16, 2024 5:37 PM
> > > To: devel@edk2.groups.io; Lin, Du 
> > > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > > 
> > > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to
> > > support new resource attributes
> > >
> > > Reviewed-by: Jiewen Yao 
> > >
> > > > -Original Message-
> > > > From: devel@edk2.groups.io  On Behalf Of Lin, Du
> > > > Sent: Thursday, May 9, 2024 1:27 PM
> > > > To: devel@edk2.groups.io
> > > > Cc: Lin, Du ; Ard Biesheuvel
> > > > ; Gerd Hoffmann ; Yao,
> > > > Jiewen 
> > > > 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 
> > > > Cc: Gerd Hoffmann 
> > > > Cc: Jiewen Yao 
> > > > Signed-off-by: Du Lin 
> > > > ---
> > > >  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 (#119278): https://edk2.groups.io/g/devel/message/119278
Mute This Topic: https://groups.io/mt/106326545/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




回复: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-27 Thread gaoliming via groups.io
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
> -邮件原件-
> 发件人: devel@edk2.groups.io  代表 Yao, Jiewen
> 发送时间: 2024年5月27日 14:43
> 收件人: Lin, Du ; devel@edk2.groups.io
> 抄送: Ard Biesheuvel ; Gerd Hoffmann
> 
> 主题: 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 
> > Sent: Monday, May 27, 2024 2:23 PM
> > To: devel@edk2.groups.io
> > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > ; Yao, Jiewen ; Lin, Du
> > 
> > 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 
> > Sent: Thursday, May 16, 2024 5:37 PM
> > To: devel@edk2.groups.io; Lin, Du 
> > Cc: Ard Biesheuvel ; Gerd Hoffmann
> > 
> > Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to
> > support new resource attributes
> >
> > Reviewed-by: Jiewen Yao 
> >
> > > -Original Message-
> > > From: devel@edk2.groups.io  On Behalf Of Lin, Du
> > > Sent: Thursday, May 9, 2024 1:27 PM
> > > To: devel@edk2.groups.io
> > > Cc: Lin, Du ; Ard Biesheuvel
> > > ; Gerd Hoffmann ; Yao,
> > > Jiewen 
> > > 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 
> > > Cc: Gerd Hoffmann 
> > > Cc: Jiewen Yao 
> > > Signed-off-by: Du Lin 
> > > ---
> > >  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 (#119277): https://edk2.groups.io/g/devel/message/119277
Mute This Topic: https://groups.io/mt/106326539/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-27 Thread Yao, Jiewen
I have approved it.

What is the process to merge? There is no COMMIT button or PUSH label.


> -Original Message-
> From: Lin, Du 
> Sent: Monday, May 27, 2024 2:23 PM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel ; Gerd Hoffmann
> ; Yao, Jiewen ; Lin, Du
> 
> 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 
> Sent: Thursday, May 16, 2024 5:37 PM
> To: devel@edk2.groups.io; Lin, Du 
> Cc: Ard Biesheuvel ; Gerd Hoffmann
> 
> Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to
> support new resource attributes
> 
> Reviewed-by: Jiewen Yao 
> 
> > -Original Message-
> > From: devel@edk2.groups.io  On Behalf Of Lin, Du
> > Sent: Thursday, May 9, 2024 1:27 PM
> > To: devel@edk2.groups.io
> > Cc: Lin, Du ; Ard Biesheuvel
> > ; Gerd Hoffmann ; Yao,
> > Jiewen 
> > 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 
> > Cc: Gerd Hoffmann 
> > Cc: Jiewen Yao 
> > Signed-off-by: Du Lin 
> > ---
> >  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: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-27 Thread Du Lin
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  
Sent: Thursday, May 16, 2024 5:37 PM
To: devel@edk2.groups.io; Lin, Du 
Cc: Ard Biesheuvel ; Gerd Hoffmann 

Subject: RE: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support 
new resource attributes

Reviewed-by: Jiewen Yao 

> -Original Message-
> From: devel@edk2.groups.io  On Behalf Of Lin, Du
> Sent: Thursday, May 9, 2024 1:27 PM
> To: devel@edk2.groups.io
> Cc: Lin, Du ; Ard Biesheuvel 
> ; Gerd Hoffmann ; Yao, 
> Jiewen 
> 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 
> Cc: Gerd Hoffmann 
> Cc: Jiewen Yao 
> Signed-off-by: Du Lin 
> ---
>  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 (#119274): https://edk2.groups.io/g/devel/message/119274
Mute This Topic: https://groups.io/mt/105996363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-20 Thread Du Lin
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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#119076): https://edk2.groups.io/g/devel/message/119076
Mute This Topic: https://groups.io/mt/105996363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-16 Thread Du Lin
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.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#118980): https://edk2.groups.io/g/devel/message/118980
Mute This Topic: https://groups.io/mt/105996363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-16 Thread Yao, Jiewen
Reviewed-by: Jiewen Yao 

> -Original Message-
> From: devel@edk2.groups.io  On Behalf Of Lin, Du
> Sent: Thursday, May 9, 2024 1:27 PM
> To: devel@edk2.groups.io
> Cc: Lin, Du ; Ard Biesheuvel ;
> Gerd Hoffmann ; Yao, Jiewen 
> 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 
> Cc: Gerd Hoffmann 
> Cc: Jiewen Yao 
> Signed-off-by: Du Lin 
> ---
>  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 (#118955): https://edk2.groups.io/g/devel/message/118955
Mute This Topic: https://groups.io/mt/105996363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




Re: [edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-13 Thread Gerd Hoffmann
On Thu, May 09, 2024 at 01:27:07PM GMT, Du Lin wrote:
> 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 
> Cc: Gerd Hoffmann 
> Cc: Jiewen Yao 
> Signed-off-by: Du Lin 

> +
> EFI_RESOURCE_ATTRIBUTE_ENCRYPTED|
> +
> EFI_RESOURCE_ATTRIBUTE_SPECIAL_PURPOSE |

Acked-by: Gerd Hoffmann 

take care,
  Gerd



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#118859): https://edk2.groups.io/g/devel/message/118859
Mute This Topic: https://groups.io/mt/105996363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-




[edk2-devel] [PATCH] OvmfPkg: Update VMM Hob list check to support new resource attributes

2024-05-08 Thread Du Lin
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 
Cc: Gerd Hoffmann 
Cc: Jiewen Yao 
Signed-off-by: Du Lin 
---
 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 (#118715): https://edk2.groups.io/g/devel/message/118715
Mute This Topic: https://groups.io/mt/105996363/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-