- [edk2-devel] [RFC][PATCH v1] UefiCpuPkg/MpInitLib DXE: Reduce AP status check interval
Re: [edk2-devel] [RFC][PATCH v1] UefiCpuPkg/MpInitLib DXE: Reduce AP status check interval
toggle quoted messageShow quoted text
From: firstname.lastname@example.org [mailto:email@example.com] On Behalf Of Brian
Sent: Tuesday, March 24, 2020 12:39 AM
To: firstname.lastname@example.org; Ni, Ray; Laszlo Ersek; Wu, Hao A;
Cc: Dong, Eric; Kinney, Michael D; Zeng, Star
Subject: Re: [edk2-devel] [RFC][PATCH v1] UefiCpuPkg/MpInitLib DXE: Reduce
AP status check interval
On 3/23/20 9:37 AM, Ni, Ray wrote:
Adding a PCD means platform integrators need to consider which value to
Thank you. I agree with Laszlo: MP initialization is tricky to scale,
OK. I agree with your suggestion.
Most of the time, they may just use the default PCD value.The patch changes existent behavior; it is not for a newly introduced
Then, why not we add the PCD later when a real case is met?
Because most platforms are not in the edk2 tree, we don't know what
platforms could be regressed by increasing the polling frequency
tenfold. (And remember that the polling action has O(n) cost, where "n"
is the logical processor count.)
Under your suggestion, the expression "real case is met" amounts to
"someone reports a regression" (possibly after the next stable tag,
even). I don't think that's a good idea.
In particular, the patch is motivated by RegisterCpuFeaturesLib -- the
CpuFeaturesInitialize() function -- on some platform(s) that Hao uses.
But there are platforms that don't use RegisterCpuFeaturesLib, and still
and platform dependent. My group builds real machines with thousands of
APs, and we have had to do various tweaks to the MP init. code. Having
a PCD to adjust this timeout will be very useful.
Thanks all for the feedbacks. Please grant me some time to prepare a new
version of the patch.
Brian J. Johnson
Enterprise X86 Lab
Hewlett Packard Enterprise
Join email@example.com to automatically receive all group messages.