public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: edk2-devel-01 <edk2-devel@ml01.01.org>
Cc: Igor Mammedov <imammedo@redhat.com>,
	Jeff Fan <jeff.fan@intel.com>,
	Jordan Justen <jordan.l.justen@intel.com>
Subject: [PATCH 0/4] UefiCpuPkg, OvmfPkg: multiprocessing fixes and improvements
Date: Tue, 22 Nov 2016 21:26:15 +0100	[thread overview]
Message-ID: <20161122202619.12594-1-lersek@redhat.com> (raw)

Patches 1 and 2 fix several instances of the same CPUID-related bug in
UefiCpuPkg (BaseXApicLib, BaseXApicX2ApicLib, MpInitLib/Ia32,
MpInitLib/X64).

Patches 3 and 4 are independent from the bugfix, but they relate
nonetheless to multiprocessing, so I'm including them in the same
series. They allow OvmfPkg to fetch the number of boot processors from
QEMU, and to make CpuMpPei / CpuDxe wait in the initial enumeration
exactly as long as it takes for all of the APs to come up. This makes MP
boot much more reliable (and no time is wasted waiting).

(

  Note that I have another patch set pending on the list:

    [edk2] [PATCH v2 0/4] OvmfPkg: broadcast SMIs and dynamically revert
                          to traditional AP sync mode

  That series and this series have a trivial context dependency in
  OvmfPkg/OvmfPkg*.dsc. Because that series consumes an interface from
  QEMU that will be part of the 2.9 release only, not the pending 2.8
  release, that series comes *second*. In comparison, this series works
  as-is, so it comes first. Locally I have rebased the broadcast SMI
  series already (it is trivial); it does not interfere with the review.

)

Repo:   https://github.com/lersek/edk2/
Branch: mpinit

Cc: Igor Mammedov <imammedo@redhat.com>
Cc: Jeff Fan <jeff.fan@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>

Thanks
Laszlo

Laszlo Ersek (4):
  UefiCpuPkg/LocalApicLib: fix feature test for Extended Topology CPUID
    leaf
  UefiCpuPkg/MpInitLib: fix feature test for Extended Topology CPUID
    leaf
  UefiCpuPkg/MpInitLib: allow platforms to provide a known CPU count
    upfront
  OvmfPkg/PlatformPei: set PcdCpuKnownLogicalProcessorNumber for
    MpInitLib

 OvmfPkg/OvmfPkgIa32.dsc                                    |  2 ++
 OvmfPkg/OvmfPkgIa32X64.dsc                                 |  2 ++
 OvmfPkg/OvmfPkgX64.dsc                                     |  2 ++
 OvmfPkg/PlatformPei/Platform.c                             | 22 ++++++++++++++++++++
 OvmfPkg/PlatformPei/PlatformPei.inf                        |  1 +
 UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.c             |  7 +++++--
 UefiCpuPkg/Library/BaseXApicX2ApicLib/BaseXApicX2ApicLib.c |  7 +++++--
 UefiCpuPkg/Library/MpInitLib/DxeMpInitLib.inf              |  1 +
 UefiCpuPkg/Library/MpInitLib/PeiMpInitLib.inf              |  4 ++--
 UefiCpuPkg/Library/MpInitLib/MpLib.c                       | 16 ++++++++++----
 UefiCpuPkg/Library/MpInitLib/Ia32/MpFuncs.nasm             | 21 +++++++++++--------
 UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm              | 21 +++++++++++--------
 UefiCpuPkg/UefiCpuPkg.dec                                  | 11 ++++++++++
 13 files changed, 89 insertions(+), 28 deletions(-)

-- 
2.9.2



             reply	other threads:[~2016-11-22 20:26 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-22 20:26 Laszlo Ersek [this message]
2016-11-22 20:26 ` [PATCH 1/4] UefiCpuPkg/LocalApicLib: fix feature test for Extended Topology CPUID leaf Laszlo Ersek
2016-11-23  5:10   ` Fan, Jeff
2016-11-22 20:26 ` [PATCH 2/4] UefiCpuPkg/MpInitLib: " Laszlo Ersek
2016-11-23  5:10   ` Fan, Jeff
2016-11-22 20:26 ` [PATCH 3/4] UefiCpuPkg/MpInitLib: allow platforms to provide a known CPU count upfront Laszlo Ersek
2016-11-23  5:36   ` Fan, Jeff
2016-11-23 16:04     ` Laszlo Ersek
2016-11-24  1:18       ` Fan, Jeff
2016-11-24  9:36         ` Laszlo Ersek
2016-11-24 13:49           ` Fan, Jeff
2016-11-24 18:32           ` Igor Mammedov
2016-11-24 21:20             ` Laszlo Ersek
2016-11-22 20:26 ` [PATCH 4/4] OvmfPkg/PlatformPei: set PcdCpuKnownLogicalProcessorNumber for MpInitLib Laszlo Ersek
2016-11-23 20:43 ` [PATCH 0/4] UefiCpuPkg, OvmfPkg: multiprocessing fixes and improvements Laszlo Ersek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161122202619.12594-1-lersek@redhat.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox