diff options
author | Jeff Fan <jeff.fan@intel.com> | 2017-04-21 11:21:00 +0800 |
---|---|---|
committer | Jeff Fan <jeff.fan@intel.com> | 2017-04-26 08:55:45 +0800 |
commit | 9d64a9fd9ee05f0b0228fc1f8b66398dac85e2c9 (patch) | |
tree | 010d4fa64f3678400cd2d3d34a9fee0398824b46 | |
parent | c934a0a5816e38ca5dbdb25936c4f841958a008e (diff) | |
download | edk2-9d64a9fd9ee05f0b0228fc1f8b66398dac85e2c9.zip edk2-9d64a9fd9ee05f0b0228fc1f8b66398dac85e2c9.tar.gz edk2-9d64a9fd9ee05f0b0228fc1f8b66398dac85e2c9.tar.bz2 |
UefiCpuPkg: Move ProgramVirtualWireMode() to MpInitLib
In PEI phase, BSP did not program vitural wired mode while APs did.
Move program virtual wired mode from CpuDxe to MpInitLib, thus it could benefit
on both CpuDxe and CpuMpPei.
https://bugzilla.tianocore.org/show_bug.cgi?id=496
Cc: Feng Tian <feng.tian@intel.com>
Cc: Michael Kinney <michael.d.kinney@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jeff Fan <jeff.fan@intel.com>
Reviewed-by: Feng Tian <feng.tian@intel.com>
-rw-r--r-- | UefiCpuPkg/CpuDxe/CpuDxe.c | 5 | ||||
-rw-r--r-- | UefiCpuPkg/Library/MpInitLib/MpLib.c | 4 |
2 files changed, 4 insertions, 5 deletions
diff --git a/UefiCpuPkg/CpuDxe/CpuDxe.c b/UefiCpuPkg/CpuDxe/CpuDxe.c index 4a5e282..8680656 100644 --- a/UefiCpuPkg/CpuDxe/CpuDxe.c +++ b/UefiCpuPkg/CpuDxe/CpuDxe.c @@ -1136,11 +1136,6 @@ InitializeCpu ( InitInterruptDescriptorTable ();
//
- // Enable the local APIC for Virtual Wire Mode.
- //
- ProgramVirtualWireMode ();
-
- //
// Install CPU Architectural Protocol
//
Status = gBS->InstallMultipleProtocolInterfaces (
diff --git a/UefiCpuPkg/Library/MpInitLib/MpLib.c b/UefiCpuPkg/Library/MpInitLib/MpLib.c index 03d6c2d..e5e211d 100644 --- a/UefiCpuPkg/Library/MpInitLib/MpLib.c +++ b/UefiCpuPkg/Library/MpInitLib/MpLib.c @@ -1364,6 +1364,10 @@ MpInitLibInitialize ( // Store BSP's MTRR setting
//
MtrrGetAllMtrrs (&CpuMpData->MtrrTable);
+ //
+ // Enable the local APIC for Virtual Wire Mode.
+ //
+ ProgramVirtualWireMode ();
if (OldCpuMpData == NULL) {
if (MaxLogicalProcessorNumber > 1) {
|