Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-11 Thread Yao, Jiewen
Merged https://github.com/tianocore/edk2/pull/2877 > -Original Message- > From: Xu, Min M > Sent: Monday, May 9, 2022 8:45 PM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Brijesh > Singh > ; Aktas, Erdem ; James > Bottomley ; Yao, Jiewen ; Tom > Lendacky ; Gerd Hoffmann >

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-11 Thread Yao, Jiewen
Acked-by: Jiewen Yao > -Original Message- > From: Xu, Min M > Sent: Monday, May 9, 2022 8:45 PM > To: devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Brijesh > Singh > ; Aktas, Erdem ; James > Bottomley ; Yao, Jiewen ; Tom > Lendacky ; Gerd Hoffmann > Subject: RE: [PATCH V2 0/6]

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-10 Thread Min Xu
On May 10, 2022 5:27 PM, Gerd Hoffmann wrote: > On Mon, May 09, 2022 at 12:44:58PM +, Xu, Min M wrote: > > Gerd & Tom > > What are your comments about this patch-set? > > > > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3918 > > > > > > Above BZ reports an issue that commit 88da06ca

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-10 Thread Lendacky, Thomas via groups.io
On 5/9/22 18:37, Xu, Min M wrote: On May 10, 2022 1:30 AM, Tom Lendacky wrote: On 5/9/22 07:44, Xu, Min M wrote: Gerd & Tom What are your comments about this patch-set? Hi Min, This appears to resolve the issue. I was able to boot a 64 vCPU guest in legacy, SEV, SEV-ES and SEV-SNP modes

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-10 Thread Lendacky, Thomas via groups.io
On 5/9/22 18:37, Xu, Min M wrote: On May 10, 2022 1:30 AM, Tom Lendacky wrote: On 5/9/22 07:44, Xu, Min M wrote: Gerd & Tom What are your comments about this patch-set? Hi Min, This appears to resolve the issue. I was able to boot a 64 vCPU guest in legacy, SEV, SEV-ES and SEV-SNP modes

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-10 Thread Gerd Hoffmann
On Mon, May 09, 2022 at 12:44:58PM +, Xu, Min M wrote: > Gerd & Tom > What are your comments about this patch-set? > > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3918 > > > > Above BZ reports an issue that commit 88da06ca triggers ASSERT in some > > scenario. This patch-set is to

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-09 Thread Min Xu
On May 10, 2022 1:30 AM, Tom Lendacky wrote: > > On 5/9/22 07:44, Xu, Min M wrote: > > Gerd & Tom > > What are your comments about this patch-set? > > Hi Min, > > This appears to resolve the issue. I was able to boot a 64 vCPU guest in > legacy, SEV, SEV-ES and SEV-SNP modes without any

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-09 Thread Lendacky, Thomas via groups.io
On 5/9/22 07:44, Xu, Min M wrote: Gerd & Tom What are your comments about this patch-set? Hi Min, This appears to resolve the issue. I was able to boot a 64 vCPU guest in legacy, SEV, SEV-ES and SEV-SNP modes without any asserts. I'm assuming that you were able to see the ASSERTs on your

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-09 Thread Min Xu
Gerd & Tom What are your comments about this patch-set? > -Original Message- > From: Xu, Min M > Sent: Saturday, May 7, 2022 9:36 AM > To: devel@edk2.groups.io > Cc: Xu, Min M ; Dong, Eric ; Ni, > Ray ; Brijesh Singh ; Aktas, > Erdem ; James Bottomley ; > Yao, Jiewen ; Tom Lendacky > ;

Re: [edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-06 Thread Ni, Ray
Reviewed-by: Ray Ni > -Original Message- > From: Xu, Min M > Sent: Saturday, May 7, 2022 9:36 AM > To: devel@edk2.groups.io > Cc: Xu, Min M ; Dong, Eric ; Ni, Ray > ; Brijesh Singh > ; Aktas, Erdem ; James > Bottomley ; Yao, Jiewen > ; Tom Lendacky ; Gerd Hoffmann > > Subject:

[edk2-devel] [PATCH V2 0/6] Support 2 CpuMpPei/CpuDxe in One image

2022-05-06 Thread Min Xu
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3918 Above BZ reports an issue that commit 88da06ca triggers ASSERT in some scenario. This patch-set is to fix this issue. As commit 88da06ca describes TDVF BSP and APs are simplied and it can simply use MpInitLibUp instead of MpInitLib. To