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 without any asserts.

I'm assuming that you were able to see the ASSERTs on your end and
validate, too?

Yes. I enable a 4 vCPU legacy guest and can see the ASSERTs. But it appears in 
a random rate so it missed in the CI.

Yeah, with a low number of vCPUs, the crashes were random. When I upped the count to 64 vCPUs that could all run in parallel (running on an EPYC server box) it happened on (almost) every boot.

But glad that you were able to observe it and create this fix.

Thanks, Min!

Tom


Thanks
Min


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


Reply via email to