[edk2-devel] [Patch v3 0/6] UefiCpuPkg: Enable Edkii Mp Services2 Ppi

Dong, Eric posted 6 patches 4 years, 8 months ago
Failed in applying to current master (apply log)
UefiCpuPkg/CpuMpPei/CpuMpPei.c                |  21 +-
UefiCpuPkg/CpuMpPei/CpuMpPei.h                |   3 +-
UefiCpuPkg/CpuMpPei/CpuMpPei.inf              |   2 +
UefiCpuPkg/CpuMpPei/Mp2Pei.c                  | 417 ++++++++++++++++++
UefiCpuPkg/Include/Library/MpInitLib.h        |  36 +-
UefiCpuPkg/Include/Ppi/MpServices2.h          | 279 ++++++++++++
UefiCpuPkg/Library/MpInitLib/DxeMpLib.c       |   5 +-
UefiCpuPkg/Library/MpInitLib/MpLib.c          |  61 ++-
UefiCpuPkg/Library/MpInitLib/MpLib.h          |   6 +-
UefiCpuPkg/Library/MpInitLib/PeiMpLib.c       |   5 +-
UefiCpuPkg/Library/MpInitLibUp/MpInitLibUp.c  |  35 ++
.../CpuFeaturesInitialize.c                   |   2 +-
.../DxeRegisterCpuFeaturesLib.c               |   6 +-
.../PeiRegisterCpuFeaturesLib.c               |  66 ++-
.../PeiRegisterCpuFeaturesLib.inf             |   1 +
.../RegisterCpuFeatures.h                     |   2 +-
UefiCpuPkg/UefiCpuPkg.dec                     |   3 +
17 files changed, 909 insertions(+), 41 deletions(-)
create mode 100644 UefiCpuPkg/CpuMpPei/Mp2Pei.c
create mode 100644 UefiCpuPkg/Include/Ppi/MpServices2.h
[edk2-devel] [Patch v3 0/6] UefiCpuPkg: Enable Edkii Mp Services2 Ppi
Posted by Dong, Eric 4 years, 8 months ago
v3 changes:
1. Fix function header definition conflict issue.
2. Update copyright.

V2 changes:
1. Enable new API for MpInitLibUp.
2. Remove useless parameter FailedCpuList.
3. Rename StartupAllAPsWorker to StartupAllCpusWorker.

V1 Changes:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1973

CpuFeatures solution introduces a policy which write msr bases on scope info. With this policy enabled, for the msr which is package scope, CpuFeature PEIM driver boots hang.
The reason for this failure is current CPU MP Ppi not supports start bsp and AP at the same time.

This patch serial introduce new API named StartupAllCPUs. And it add new ppi namded Edkii Mp Services2 Ppi, include EFI MP Serivces PPI and this new API.

With this new PPi, CPU features PEI solution can start all CPUs at the same time.

Because this patch only add one new API, i did belew tests:
1. Test StartupAllCpus with timeout == 0 pass.
2. Test StartupAllCpus with timeout != 0 pass.

Cc: Ray Ni <ray.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Chandana Kumar <chandana.c.kumar@intel.com>
Cc: Star Zeng <star.zeng@intel.com>


Eric Dong (6):
  UefiCpuPkg/Include/MpInitLib.h: Add MpInitLibStartupAllCPUs API.
  UefiCpuPkg/MpInitLib: Add MpInitLibStartupAllCPUs API.
  UefiCpuPkg/MpInitLibUp: Add MpInitLibStartupAllCPUs API.
  UefiCpuPkg: Add new EDKII_PEI_MP_SERVICES2_PPI
  UefiCpuPkg/CpuMpPei: Produce EDKII_PEI_MP_SERVICES2_PPI.
  UefiCpuPkg/RegisterCpuFeaturesLib: Start all processors
    simultaneously.

 UefiCpuPkg/CpuMpPei/CpuMpPei.c                |  21 +-
 UefiCpuPkg/CpuMpPei/CpuMpPei.h                |   3 +-
 UefiCpuPkg/CpuMpPei/CpuMpPei.inf              |   2 +
 UefiCpuPkg/CpuMpPei/Mp2Pei.c                  | 417 ++++++++++++++++++
 UefiCpuPkg/Include/Library/MpInitLib.h        |  36 +-
 UefiCpuPkg/Include/Ppi/MpServices2.h          | 279 ++++++++++++
 UefiCpuPkg/Library/MpInitLib/DxeMpLib.c       |   5 +-
 UefiCpuPkg/Library/MpInitLib/MpLib.c          |  61 ++-
 UefiCpuPkg/Library/MpInitLib/MpLib.h          |   6 +-
 UefiCpuPkg/Library/MpInitLib/PeiMpLib.c       |   5 +-
 UefiCpuPkg/Library/MpInitLibUp/MpInitLibUp.c  |  35 ++
 .../CpuFeaturesInitialize.c                   |   2 +-
 .../DxeRegisterCpuFeaturesLib.c               |   6 +-
 .../PeiRegisterCpuFeaturesLib.c               |  66 ++-
 .../PeiRegisterCpuFeaturesLib.inf             |   1 +
 .../RegisterCpuFeatures.h                     |   2 +-
 UefiCpuPkg/UefiCpuPkg.dec                     |   3 +
 17 files changed, 909 insertions(+), 41 deletions(-)
 create mode 100644 UefiCpuPkg/CpuMpPei/Mp2Pei.c
 create mode 100644 UefiCpuPkg/Include/Ppi/MpServices2.h

-- 
2.21.0.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#44289): https://edk2.groups.io/g/devel/message/44289
Mute This Topic: https://groups.io/mt/32581673/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-