Cloning RHEL-8 BZ to RHEL-9. +++ This bug was initially created as a clone of Bug #1982176 +++ Issue detected while testing bug 1904267: (In reply to Laszlo Ersek from bug 1904267 comment #57) > (In reply to Eduardo Habkost from bug 1904267 comment #54) > > (In reply to Brian Payton from bug 1904267 comment #30) > > > Simple failure with 1024 vcpus and 8TB > > > > For reference, this is the failure on debugcon: > > > > [...] > > GetMicrocodePatchInfoFromHob: Microcode patch cache HOB is not found. > > CpuMpPei: 5-Level Paging = 0 > > Register PPI Notify: 8F9D4825-797D-48FC-8471-845025792EF6 > > > > ASSERT_EFI_ERROR (Status = Out of Resources) > > ASSERT > > /builddir/build/BUILD/edk2-ca407c7246bf/UefiCpuPkg/CpuMpPei/CpuBist.c(186): > > !EFI_ERROR (Status) > > > > > > > > And this is the failing code: > > > > BistInformationSize = sizeof (EFI_SEC_PLATFORM_INFORMATION_RECORD2) + > > sizeof (EFI_SEC_PLATFORM_INFORMATION_CPU) * > > NumberOfProcessors; > > Status = PeiServicesAllocatePool ( > > (UINTN) BistInformationSize, > > (VOID **) &PlatformInformationRecord2 > > ); > > ASSERT_EFI_ERROR (Status); > > > > Maybe we're hitting some limit on allocation sizes? I don't know what's the > > size of EFI_SEC_PLATFORM_INFORMATION_RECORD2 and > > EFI_SEC_PLATFORM_INFORMATION_CPU. > > This is an edk2 design limitation. > > Please file an edk2 bug for RHEL-8, and clone it for RHEL-9. > > Meanwhile I've sent an upstream problem report: > > * [edk2-devel] CPU count limitation in CpuMpPei BIST processing > > > https://listman.redhat.com/archives/edk2-devel-archive/2021-June/msg01493. > html > http://mid.mail-archive.com/ffa9d7db-b670-8b88-758f-4785c8d05d40@redhat.com > https://edk2.groups.io/g/devel/message/77376
Pawel, Gerd, Is the fix required to resolve this issue already upstream? Thanks
(In reply to Nitesh Narayan Lal from comment #11) > Pawel, Gerd, > Is the fix required to resolve this issue already upstream? No. Possible solutions have been discussed (see linked mails in the original description of this bug), nothing merged so far, Pawel is busy creating a patch.
(In reply to Gerd Hoffmann from comment #12) > (In reply to Nitesh Narayan Lal from comment #11) > > Pawel, Gerd, > > Is the fix required to resolve this issue already upstream? > > No. Possible solutions have been discussed (see linked mails in the > original description of this bug), nothing merged so far, Pawel is busy > creating a patch. Understood, thank you for the update. I am clearing the needinfo for Pawel.
Adding Vivek. Vivek,this BZ has been state for a while.. Can we get an update of where it stands?
(In reply to Klaus Heinrich Kiwi from comment #22) > Adding Vivek. > > Vivek,this BZ has been state for a while.. Can we get an update of where it > stands? We discussed status of this bug this Tuesday. Following were latest updates on this. - Pawel continues to work on this. - This needs changes in qemu as well. Gerd has provided a patch to Pawel for qemu so that he can test firmware changes. - Pawel and Gerd to discuss if changing data structure size is going to break something else in the system. Gerd is on sick leave, so I guess this part can make progress once Gerd is back. Pawel, do you have changes ready and tested so that Gerd (and others) can review it?
Hi All, sorry I make you waiting. I do not have code ready for this, just some raw draft. Will discuss this with Gerd on Wednesday if possible (Tuesday is public holiday in Poland). For last 2 weeks both me and Gerd were on PTO which delayed our sync also.
There is no merged code for this upstream yet. So resetting it back to "ASSIGNED" state.
Miroslav, is it some script which sets the status automatically from Assigned to POST. What we have currently there is just some Draft and its not about to be merged.
(In reply to Vivek Goyal from comment #38) > Miroslav, is it some script which sets the status automatically from > Assigned to POST. What we have currently there is just some Draft and its > not about to be merged. Yes, it's done by our gitlab checker. However, it is expect name in form "DRAFT: ...." to identify MR as draft. As the MR uses format "[draft] ..." it is not properly identified as draft MR. I updated MR to be in form we expect.
QE (pre verify): Set 'Verified:Tested,SanityOnly' as gating/tier1 test pass.
For the record: The exact limit are: 1022 (old, without fix) and 8190 (new, with fix).
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory (Important: edk2 security, bug fix, and enhancement update), and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2023:2165