public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Fan Jeff <vanjeff_919@hotmail.com>
To: Eric Dong <eric.dong@intel.com>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: 答复: [Patch 0/2] Enhance collect AP Count logic
Date: Tue, 24 Oct 2017 07:31:39 +0000	[thread overview]
Message-ID: <CY1PR19MB0283CF8C8124B6F5F76A067ED7470@CY1PR19MB0283.namprd19.prod.outlook.com> (raw)
In-Reply-To: <1508743358-3640-1-git-send-email-eric.dong@intel.com>

Reviewed-by: Jeff Fan <vanjeff_919@hotmail.com>

发件人: Eric Dong<mailto:eric.dong@intel.com>
发送时间: 2017年10月23日 15:22
收件人: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>
主题: [edk2] [Patch 0/2] Enhance collect AP Count logic

Current logic always waiting for a specific value to collect all APs
count. This logic may caused some platforms cost too much time to
wait for time out.

This patch series add new logic to collect APs count. It adds new variable
NumApsExecuting to detect whether all APs have finished initialization.
Each AP let NumApsExecuting++ when begin to initialize itself and let
NumApsExecuting-- when it finish the initialization. BSP base on whether
NumApsExecuting == 0 to finished the collect AP process.

Because current code already use NumApsExecuting variable, so add another
patch to change the variable name for the current code for better
understanding.


Eric Dong (2):
  UefiCpuPkg/MpInitLib: Change AP Index variable name.
  UefiCpuPkg/MpInitLib: Enhance waiting for AP initialization logic.

 UefiCpuPkg/Library/MpInitLib/Ia32/MpEqu.inc    |  3 ++-
 UefiCpuPkg/Library/MpInitLib/Ia32/MpFuncs.nasm | 10 ++++++++--
 UefiCpuPkg/Library/MpInitLib/MpLib.c           | 24 ++++++++++++++++--------
 UefiCpuPkg/Library/MpInitLib/MpLib.h           |  3 ++-
 UefiCpuPkg/Library/MpInitLib/X64/MpEqu.inc     |  5 +++--
 UefiCpuPkg/Library/MpInitLib/X64/MpFuncs.nasm  | 10 ++++++++--
 6 files changed, 39 insertions(+), 16 deletions(-)

--
2.7.0.windows.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


      parent reply	other threads:[~2017-10-24  7:27 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-23  7:22 [Patch 0/2] Enhance collect AP Count logic Eric Dong
2017-10-23  7:22 ` [Patch 1/2] UefiCpuPkg/MpInitLib: Change AP Index variable name Eric Dong
2017-10-24  6:00   ` Ni, Ruiyu
2017-10-23  7:22 ` [Patch 2/2] UefiCpuPkg/MpInitLib: Enhance waiting for AP initialization logic Eric Dong
2017-10-24  6:02   ` Ni, Ruiyu
2017-10-24  6:27     ` 答复: " Fan Jeff
2017-10-24  7:18       ` Ni, Ruiyu
2017-10-24  7:32         ` 答复: " Fan Jeff
2017-10-24 10:15   ` Laszlo Ersek
2017-10-24 14:24     ` 答复: " Fan Jeff
2017-10-24 16:29       ` Laszlo Ersek
2017-10-24 15:23     ` Dong, Eric
2017-10-24 15:40       ` Dong, Eric
2017-10-24 17:40       ` Laszlo Ersek
2017-10-24 22:30         ` Brian J. Johnson
2017-10-25  5:35           ` 答复: " Fan Jeff
2017-10-25  5:32         ` Fan Jeff
2017-10-25  5:42         ` Dong, Eric
2017-10-25 15:07           ` Laszlo Ersek
2017-10-26  1:13             ` Dong, Eric
2017-10-26 20:48               ` Brian J. Johnson
2017-10-27  1:31                 ` Dong, Eric
2017-10-24  7:31 ` Fan Jeff [this message]

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=CY1PR19MB0283CF8C8124B6F5F76A067ED7470@CY1PR19MB0283.namprd19.prod.outlook.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