From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.24, mailfrom: eric.dong@intel.com) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by groups.io with SMTP; Wed, 24 Jul 2019 00:07:32 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 24 Jul 2019 00:07:31 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.64,302,1559545200"; d="scan'208";a="172211071" Received: from ydong10-win10.ccr.corp.intel.com ([10.239.158.133]) by orsmga003.jf.intel.com with ESMTP; 24 Jul 2019 00:07:28 -0700 From: "Dong, Eric" To: devel@edk2.groups.io Cc: Ray Ni , Laszlo Ersek , Chandana Kumar , Star Zeng Subject: [Patch v3 0/6] UefiCpuPkg: Enable Edkii Mp Services2 Ppi Date: Wed, 24 Jul 2019 15:07:21 +0800 Message-Id: <20190724070727.14976-1-eric.dong@intel.com> X-Mailer: git-send-email 2.21.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 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 Cc: Laszlo Ersek Cc: Chandana Kumar Cc: Star Zeng 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