Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-25 Thread Laszlo Ersek
: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to >> control AP status check interval >> >> Mike, Hao, >> I searched all edk2 code using regex pattern "Pcd.*Timeout.*", >> "Pcd.*Timer.*", >> "Pcd.*Delay.*" >> and fo

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Laszlo Ersek
On 03/24/20 16:59, Kinney, Michael D wrote: > How was the milliseconds units selected? I suggested msecs for continuity with the pre-patch unit: #define AP_CHECK_INTERVAL (EFI_TIMER_PERIOD_MILLISECONDS (100)) > We have other PCDs that provide timer intervals in > 100ns unit and my

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Wu, Hao A
, 2020 11:59 PM > > To: Ni, Ray ; Zeng, Star ; Wu, Hao A > ; > > devel@edk2.groups.io; Kinney, Michael D > > Cc: Dong, Eric ; Laszlo Ersek ; > Brian J . Johnson > > Subject: RE: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to > control AP status check inte

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Ni, Ray
> ; > devel@edk2.groups.io; Kinney, Michael D > Cc: Dong, Eric ; Laszlo Ersek ; Brian > J . Johnson > Subject: RE: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to > control AP status check interval > > How was the milliseconds units selected? > > We ha

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Ni, Ray
A ; devel@edk2.groups.io > Cc: Dong, Eric ; Ni, Ray ; Laszlo > Ersek ; Kinney, Michael D > ; Brian J . Johnson ; > Zeng, Star > Subject: RE: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to > control AP status check interval > > > -Original Message-

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Zeng, Star
essage- > > > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf > > > Of Wu, Hao A > > > Sent: Tuesday, March 24, 2020 2:33 PM > > > To: devel@edk2.groups.io > > > Cc: Wu, Hao A ; Dong, Eric > > > ; Ni, Ray ; Laszlo Ersek > &g

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Wu, Hao A
2020 2:33 PM > > To: devel@edk2.groups.io > > Cc: Wu, Hao A ; Dong, Eric ; Ni, > > Ray ; Laszlo Ersek ; Kinney, Michael > > D ; Zeng, Star ; Brian J . > > Johnson > > Subject: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to > > control AP statu

Re: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Zeng, Star
.groups.io > Cc: Wu, Hao A ; Dong, Eric ; Ni, > Ray ; Laszlo Ersek ; Kinney, Michael > D ; Zeng, Star ; Brian J . > Johnson > Subject: [edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to > control AP status check interval > > REF:https://bugzilla.tianocore.o

[edk2-devel] [PATCH v2] UefiCpuPkg/MpInitLib DXE: Add PCD to control AP status check interval

2020-03-24 Thread Wu, Hao A
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2627 The commit will introduce a static PCD to specify the periodic interval for checking the AP status when MP services StartupAllAPs() and StartupThisAP() are being executed in a non-blocking manner. Or in other words, specifies the interval