diff options
author | Eric Dong <eric.dong@intel.com> | 2018-07-24 22:29:53 +0800 |
---|---|---|
committer | Eric Dong <eric.dong@intel.com> | 2018-07-26 16:54:14 +0800 |
commit | 2da3e96cb7aa4510f5b5086efca0c2188c055751 (patch) | |
tree | 54a2b58bf8d3540a46beac06ec46510a05445240 /BaseTools/Source/Python/Workspace/WorkspaceCommon.py | |
parent | 2a5997f899bc0bb2851d3ba7168b78b36444e6f0 (diff) | |
download | edk2-2da3e96cb7aa4510f5b5086efca0c2188c055751.tar.gz edk2-2da3e96cb7aa4510f5b5086efca0c2188c055751.tar.bz2 edk2-2da3e96cb7aa4510f5b5086efca0c2188c055751.zip |
UefiCpuPkg/MpInitLib: Remove StartCount and volatile definition.
The patch includes below changes:
(1) It removes "volatile" from RunningCount, because only the BSP modifies it.
(2) When we detect a timeout in CheckAllAPs(), and collect the list of failed CPUs, the size of the list is derived from the following difference, before the patch:
StartCount - FinishedCount
where "StartCount" is set by the BSP at startup, and FinishedCount is incremented by the APs themselves.
Here the patch replaces this difference with
StartCount - RunningCount
that is, the difference is no more calculated from the BSP's startup counter and the AP's shared finish counter, but from the RunningCount measurement that the BSP does itself, in CheckAllAPs().
(3) Finally, the patch changes the meaning of RunningCount. Before the patch, we have:
- StartCount: the number of APs the BSP stars up,
- RunningCount: the number of finished APs that the BSP collected
After the patch, StartCount is removed, and RunningCount is *redefined* as the following difference:
OLD_StartCount - OLD_RunningCount
Giving the number of APs that the BSP started up but hasn't collected yet.
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Tested-by: Laszlo Ersek <lersek@redhat.com>
Diffstat (limited to 'BaseTools/Source/Python/Workspace/WorkspaceCommon.py')
0 files changed, 0 insertions, 0 deletions