On 10/10/19 09:38, Laszlo Ersek wrote: > On 10/10/19 04:52, Ni, Ray wrote: >> Laszlo, >> Can you add comments in the code you changed to describe the two different >> behaviors? > > It's described in the DEC file, near the PCD: > > + # nonzero - PcdCpuApInitTimeOutInMicroSeconds is ignored. The > initial > + # AP detection finishes when the detected CPU count (BSP > + # plus APs) reaches the value of this PCD.<BR> > > (1) Do you want me to repeat this explanation in the code as well? > > (2) Do you want me to emphasize in the explanation that, if at least one > processor fails to check in, the boot is going to block forever, by design? > > (3) Emphasize this in the DEC file only, in the code only, or in both?
... I think I've figured it out, let me send a v2 of just this patch. (The rest of the series (for OvmfPkg) will likely be dropped, due to an alternative proposal from Igor. The UefiCpuPkg facility will remain the same however.) Thanks! Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#48711): https://edk2.groups.io/g/devel/message/48711 Mute This Topic: https://groups.io/mt/34441228/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
