public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Pedro Falcato" <pedro.falcato@gmail.com>
To: "Lin, JackX" <jackx.lin@intel.com>
Cc: "devel@edk2.groups.io" <devel@edk2.groups.io>,
	"Chiu, Chasel" <chasel.chiu@intel.com>,
	 "Desimone, Nathaniel L" <nathaniel.l.desimone@intel.com>,
	"Oram, Isaac W" <isaac.w.oram@intel.com>,
	 "Gao, Liming" <gaoliming@byosoft.com.cn>,
	"Dong, Eric" <eric.dong@intel.com>,
	 "Kuo, Donald" <donald.kuo@intel.com>,
	"Kumar, Chandana C" <chandana.c.kumar@intel.com>
Subject: Re: [edk2-devel] [edk2-platforms: PATCH] BIOS needs to present cores in order of relative performance in MADT
Date: Fri, 18 Nov 2022 19:11:26 +0000	[thread overview]
Message-ID: <CAKbZUD0L8v0OgFCLbMt5fwKnT9t+5L54phWebuu7Y_0NXBusdQ@mail.gmail.com> (raw)
In-Reply-To: <DM6PR11MB3738309A5A1ED89656443773F1099@DM6PR11MB3738.namprd11.prod.outlook.com>

[-- Attachment #1: Type: text/plain, Size: 906 bytes --]

On Fri, Nov 18, 2022 at 8:35 AM Lin, JackX <jackx.lin@intel.com> wrote:

> Hi Pedro,
>
>
>
> Yes, it is for  some specific test tools.
>
>
>
> There are different performance between CPU cores, and the big cores are
> better than small cores.
>
> Some legacy tools are executing with the highest performance CPU cores, In
> early CPU design, it is usually the first one.
>
>
>
> We have to put them at the front of all CPU cores, otherwise some tests
> cannot pass.
>
> Due to avoid this happened, we need to ensure CPU cores are ordered by big
> core first.
>

Thank you for the thorough description. Could you please (or whoever is
pushing this) add this to the commit message?
I've been following the evolution of this specific file over the last few
months with mild interest and getting things properly documented should be
pretty important!
Might help avoid any future breakage too :D

Thanks,
Pedro

[-- Attachment #2: Type: text/html, Size: 1807 bytes --]

  reply	other threads:[~2022-11-18 19:11 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-17  6:01 [edk2-platforms: PATCH] BIOS needs to present cores in order of relative performance in MADT JackX Lin
2022-11-17  6:06 ` [edk2-devel] " Michael D Kinney
2022-11-17 14:09 ` Pedro Falcato
2022-11-18  8:34   ` JackX Lin
2022-11-18 19:11     ` Pedro Falcato [this message]
2022-11-22  3:35       ` Ni, Ray
  -- strict thread matches above, loose matches on Subject: below --
2022-11-16  6:29 JackX Lin
2022-11-17  3:45 ` [edk2-devel] " Michael D Kinney
2022-11-17  3:54   ` JackX Lin
2022-11-16  2:52 JackX Lin
2022-11-16  3:50 ` [edk2-devel] " Michael D Kinney
2022-11-16  3:59   ` JackX Lin

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=CAKbZUD0L8v0OgFCLbMt5fwKnT9t+5L54phWebuu7Y_0NXBusdQ@mail.gmail.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