From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f176.google.com (mail-pl1-f176.google.com [209.85.214.176]) by mx.groups.io with SMTP id smtpd.web10.2304.1668798698399491264 for ; Fri, 18 Nov 2022 11:11:38 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@gmail.com header.s=20210112 header.b=pl2/DVxC; spf=pass (domain: gmail.com, ip: 209.85.214.176, mailfrom: pedro.falcato@gmail.com) Received: by mail-pl1-f176.google.com with SMTP id b21so5368116plc.9 for ; Fri, 18 Nov 2022 11:11:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=p6or42ogWDwBPh6Rs3+daETbSVckXypD4/HHvZJFUNs=; b=pl2/DVxC9ab5P8fFlv3fc/YJM82TGCXyiUPO2RrKXjXUOmb86BYwQhGx1fbCUWjgXg YjVYIra+WGd5E4dshJQWHfXfJsaiw12Lxf67X/By9iPV+aUkBEYZn9X5PJNLujqBfvbW Dl8ipM7d4JA4wzILRovCmOVUKUaN+jDFmf9RCWmifsGY7JOmMIqXolQf2HkqrLhNR8jw C3a+xaIe+Ha+qvlTpmfQmjNd3fDWXLDNp2y10vX6+u+JrojTwZUvTideZikNR+hb18ec SrIWjrG0APlkbjrhHZbK67Chhh94sgLI3mPh8JTjJB7GSUetGSIOXFayKorHxbQxYM3+ JuJw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=p6or42ogWDwBPh6Rs3+daETbSVckXypD4/HHvZJFUNs=; b=SoZZ5ZZ4PWREGFXIuWkJRHemSdTfaZ2sk/WXgUJThGzLdeYjM0ony8W3VukgdyZB5U /DpoO1+lo7AK9doAjYf0ylYB8GZEfHEG87FCiCfYu82c0lWBrX3DfgBj+YvO9quKO6dS ynodD2S2M44yT3uMCPySR1rDD5kE0fhLAfw0ar2jUhGW5/upcjRhpX1Cl1dkQR/Vem8u CuNdb+sZWijcYnPB7kUaURiwtTFOP0A8v+9iQBckOVz/XTiIrI1gBIY0MXrz5lHaXCUu 33OePCCgTRkwDmyXFGs0IoNXvS2VuX1MvRzZ6ygPbztXgAU6mZRtgY4MecG4G8LdXtAg 9O/w== X-Gm-Message-State: ANoB5pmVWRbB+eNqZ/ynTHI3GKNxM2wHDb69eLbETuUC/B4eKq7S9mww r+TX9x/4MjsWHk1kd0OF8qIRF4vJDPrOHx4JAUo= X-Google-Smtp-Source: AA0mqf5zWV2MjOIoO5hkbGeGyuxm1x96jTWy9Sv4Y70XBCMTftCqHL4TMJTdOG5+FtIUvjod4C8n4JBIWNSXR0A/RD8= X-Received: by 2002:a17:90a:ea05:b0:20a:a1a8:3719 with SMTP id w5-20020a17090aea0500b0020aa1a83719mr9152272pjy.225.1668798697791; Fri, 18 Nov 2022 11:11:37 -0800 (PST) MIME-Version: 1.0 References: <20221117060123.209-1-jackx.lin@intel.com> In-Reply-To: From: "Pedro Falcato" Date: Fri, 18 Nov 2022 19:11:26 +0000 Message-ID: Subject: Re: [edk2-devel] [edk2-platforms: PATCH] BIOS needs to present cores in order of relative performance in MADT To: "Lin, JackX" Cc: "devel@edk2.groups.io" , "Chiu, Chasel" , "Desimone, Nathaniel L" , "Oram, Isaac W" , "Gao, Liming" , "Dong, Eric" , "Kuo, Donald" , "Kumar, Chandana C" Content-Type: multipart/alternative; boundary="00000000000095660805edc37963" --00000000000095660805edc37963 Content-Type: text/plain; charset="UTF-8" On Fri, Nov 18, 2022 at 8:35 AM Lin, JackX 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 --00000000000095660805edc37963 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable

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

Hi Pedro,

=C2=A0

Yes, it is for =C2=A0some specific test tools.

=C2=A0

There are different performance between CPU cores, a= nd the big cores are better than small cores.

Some legacy tools are executing with the highest per= formance CPU cores, In early CPU design, it is usually the first one.

=C2=A0

We have to put them at the front of all CPU cores, o= therwise some tests cannot pass.

Due to avoid this happened, we need to ensure CPU co= res are ordered by big core first.


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

Thanks,
=
Pedro
--00000000000095660805edc37963--