From: "Ni, Ruiyu" <ruiyu.ni@Intel.com>
To: Eric Dong <eric.dong@intel.com>, edk2-devel@lists.01.org
Cc: Michael D Kinney <michael.d.kinney@intel.com>,
Laszlo Ersek <lersek@redhat.com>
Subject: Re: [Patch 00/14] Update MSR definitions.
Date: Tue, 18 Sep 2018 09:59:58 +0800 [thread overview]
Message-ID: <952599c7-1c36-56c5-a3d2-4510609a0c98@Intel.com> (raw)
In-Reply-To: <20180918014330.28336-1-eric.dong@intel.com>
On 9/18/2018 9:43 AM, Eric Dong wrote:
> Current MSR definition are follow the SDM 2016-09 version. The latest
> SDM is 2018-05. This patch serial update the MSR related definition to
> follow the latest SDM 2018-05 version. MSR related defintion are saved
> at UefiCpuPkg\Include\Register\.
>
> The changes for this serial includes:
> 1. Add new MSR definition and file.
> 2. Remove old MSR definition which not defined in new SDM.
> 3. Change MSR name to follow new SDM, keep old one for compatibility.
> 4. Change MSR data structure definition to follow new SDM.
> 5. Update comments to follow the new SDM, mainly related to chapter info.
>
> Below changes are incompatible changes:
> 2. Remove old MSR definition which not defined in new SDM.
> For this one, i search edk2 codebase, not found any code uses it. so no
> impact for edk2 codebase. Detail changes see patch 9 ~ 11.
>
> 4. Change MSR data structure definition to follow new SDM.
> For this one, new data structure just change the original reserved bits to
> valid bits, should have no impact for the current code. Detail see patch 8
> and patch 14
>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Ruiyu Ni <ruiyu.ni@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Eric Dong <eric.dong@intel.com>
>
>
> Eric Dong (14):
> UefiCpuPkg/Include/Register/Msr: Update reference spec info.
> UefiCpuPkg/Include/Register/Msr/GoldmontPlusMsr.h: Add new MSR file
> for goldmont plus microarchitecture.
> UefiCpuPkg/Include/Register/Msr/SilvermontMsr.h: Add new MSR.
> UefiCpuPkg/Include/Register/Msr/*.h: Add new MSR.
> UefiCpuPkg/Include/Register/Msr/XeonPhiMsr.h: Add new MSR.
> UefiCpuPkg/Include/Register/Msr/SkylakeMsr.h: Add new MSRs.
> UefiCpuPkg/Include/Register/ArchitecturalMsr.h: Add new MSR.
> UefiCpuPkg/Include/Register/ArchitecturalMsr.h: Change structure
> definition.
> UefiCpuPkg/Include/Register/Msr/Core2Msr.h: Remove old MSR.
> UefiCpuPkg/Include/Register/Msr/P6Msr.h: Remove old MSR.
> UefiCpuPkg/Include/Register/Msr/CoreMsr.h: Remove old MSR.
> UefiCpuPkg/Include/Register/Msr/SkylakeMsr.h: Add new MSR name and
> keep old one.
> UefiCpuPkg/Include/Register/Msr/GoldmontMsr.h: Add new MSR name and
> keep old one.
> UefiCpuPkg/Include/Register/Msr/XeonPhiMsr.h: Change structure
> definition.
>
> UefiCpuPkg/Include/Register/ArchitecturalMsr.h | 130 +-
> UefiCpuPkg/Include/Register/Msr.h | 7 +-
> UefiCpuPkg/Include/Register/Msr/AtomMsr.h | 28 +-
> UefiCpuPkg/Include/Register/Msr/BroadwellMsr.h | 62 +-
> UefiCpuPkg/Include/Register/Msr/Core2Msr.h | 102 +-
> UefiCpuPkg/Include/Register/Msr/CoreMsr.h | 74 +-
> UefiCpuPkg/Include/Register/Msr/GoldmontMsr.h | 88 +-
> UefiCpuPkg/Include/Register/Msr/GoldmontPlusMsr.h | 272 ++++
> UefiCpuPkg/Include/Register/Msr/HaswellEMsr.h | 62 +-
> UefiCpuPkg/Include/Register/Msr/HaswellMsr.h | 34 +-
> UefiCpuPkg/Include/Register/Msr/IvyBridgeMsr.h | 8 +-
> UefiCpuPkg/Include/Register/Msr/NehalemMsr.h | 52 +-
> UefiCpuPkg/Include/Register/Msr/P6Msr.h | 60 +-
> UefiCpuPkg/Include/Register/Msr/Pentium4Msr.h | 202 +--
> UefiCpuPkg/Include/Register/Msr/PentiumMMsr.h | 22 +-
> UefiCpuPkg/Include/Register/Msr/PentiumMsr.h | 12 +-
> UefiCpuPkg/Include/Register/Msr/SandyBridgeMsr.h | 49 +-
> UefiCpuPkg/Include/Register/Msr/SilvermontMsr.h | 100 +-
> UefiCpuPkg/Include/Register/Msr/SkylakeMsr.h | 1602 ++++++++++++++++++++-
> UefiCpuPkg/Include/Register/Msr/Xeon5600Msr.h | 8 +-
> UefiCpuPkg/Include/Register/Msr/XeonDMsr.h | 84 +-
> UefiCpuPkg/Include/Register/Msr/XeonE7Msr.h | 6 +-
> UefiCpuPkg/Include/Register/Msr/XeonPhiMsr.h | 326 ++++-
> 23 files changed, 2813 insertions(+), 577 deletions(-)
> create mode 100644 UefiCpuPkg/Include/Register/Msr/GoldmontPlusMsr.h
>
Acked-by: Ruiyu Ni <ruiyu.ni@intel.com>
--
Thanks,
Ray
next prev parent reply other threads:[~2018-09-18 1:59 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-18 1:43 [Patch 00/14] Update MSR definitions Eric Dong
2018-09-18 1:43 ` [Patch 01/14] UefiCpuPkg/Include/Register/Msr: Update reference spec info Eric Dong
2018-09-18 1:43 ` [Patch 02/14] UefiCpuPkg/Include/Register/Msr/GoldmontPlusMsr.h: Add new MSR file for goldmont plus microarchitecture Eric Dong
2018-09-18 1:43 ` [Patch 03/14] UefiCpuPkg/Include/Register/Msr/SilvermontMsr.h: Add new MSR Eric Dong
2018-09-18 1:43 ` [Patch 04/14] UefiCpuPkg/Include/Register/Msr/*.h: " Eric Dong
2018-09-18 1:43 ` [Patch 05/14] UefiCpuPkg/Include/Register/Msr/XeonPhiMsr.h: " Eric Dong
2018-09-18 1:43 ` [Patch 06/14] UefiCpuPkg/Include/Register/Msr/SkylakeMsr.h: Add new MSRs Eric Dong
2018-09-18 1:43 ` [Patch 07/14] UefiCpuPkg/Include/Register/ArchitecturalMsr.h: Add new MSR Eric Dong
2018-09-18 1:43 ` [Patch 08/14] UefiCpuPkg/Include/Register/ArchitecturalMsr.h: Change structure definition Eric Dong
2018-09-18 1:43 ` [Patch 09/14] UefiCpuPkg/Include/Register/Msr/Core2Msr.h: Remove old MSR Eric Dong
2018-09-18 1:43 ` [Patch 10/14] UefiCpuPkg/Include/Register/Msr/P6Msr.h: " Eric Dong
2018-09-18 1:43 ` [Patch 11/14] UefiCpuPkg/Include/Register/Msr/CoreMsr.h: " Eric Dong
2018-09-18 1:43 ` [Patch 12/14] UefiCpuPkg/Include/Register/Msr/SkylakeMsr.h: Add new MSR name and keep old one Eric Dong
2018-09-18 1:43 ` [Patch 13/14] UefiCpuPkg/Include/Register/Msr/GoldmontMsr.h: " Eric Dong
2018-09-18 1:43 ` [Patch 14/14] UefiCpuPkg/Include/Register/Msr/XeonPhiMsr.h: Change structure definition Eric Dong
2018-09-18 1:59 ` Ni, Ruiyu [this message]
2018-09-18 9:01 ` [Patch 00/14] Update MSR definitions Laszlo Ersek
2018-09-21 7:40 ` Dong, Eric
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=952599c7-1c36-56c5-a3d2-4510609a0c98@Intel.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