public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Chang, Abner via groups.io" <abner.chang=amd.com@groups.io>
To: gaoliming <gaoliming@byosoft.com.cn>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>,
	"Hsueh, Hong-Chih (Neo)" <Hong-Chih.Hsueh@amd.com>
Cc: "michael.d.kinney@intel.com" <michael.d.kinney@intel.com>,
	"zhiguang.liu@intel.com" <zhiguang.liu@intel.com>,
	"He, Jiangang" <Jiangang.He@amd.com>
Subject: Re: [edk2-devel] [PATCH] MdePkg: Remove non-ASCII characters from header file (BZ# 4775)
Date: Thu, 16 May 2024 01:12:29 +0000	[thread overview]
Message-ID: <LV8PR12MB9452D590B20503B66B64B6B7EAED2@LV8PR12MB9452.namprd12.prod.outlook.com> (raw)
In-Reply-To: <044b01daa72d$9a0819c0$ce184d40$@byosoft.com.cn>

[AMD Official Use Only - AMD Internal Distribution Only]

Thanks Liming! We can merge it after the stable release.

Abner

> -----Original Message-----
> From: gaoliming <gaoliming@byosoft.com.cn>
> Sent: Thursday, May 16, 2024 9:09 AM
> To: devel@edk2.groups.io; Chang, Abner <Abner.Chang@amd.com>; Hsueh,
> Hong-Chih (Neo) <Hong-Chih.Hsueh@amd.com>
> Cc: michael.d.kinney@intel.com; zhiguang.liu@intel.com; He, Jiangang
> <Jiangang.He@amd.com>
> Subject: 回复: [edk2-devel] [PATCH] MdePkg: Remove non-ASCII characters
> from header file (BZ# 4775)
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Abner:
>   This change is good to me. Reviewed-by: Liming Gao
> <gaoliming@byosoft.com.cn>
>
>   But, this is not critical issue. So, I suggest to merge it after the stable tag.
>
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Chang, Abner
> via
> > groups.io
> > 发送时间: 2024年5月16日 8:50
> > 收件人: Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh@amd.com>;
> > devel@edk2.groups.io
> > 抄送: michael.d.kinney@intel.com; gaoliming@byosoft.com.cn;
> > zhiguang.liu@intel.com; He, Jiangang <Jiangang.He@amd.com>
> > 主题: Re: [edk2-devel] [PATCH] MdePkg: Remove non-ASCII characters from
> > header file (BZ# 4775)
> >
> > [AMD Official Use Only - AMD Internal Distribution Only]
> >
> > Hi Mike, Liming and Zhiguang,
> > Could you please check this patch sent two weeks ago? The corresponding
> BZ
> > ticket is 4775. We overlooked tracking this issue and missed the 202405
> stable
> > release. As this impacts the build, do you think we can have a quick review
> and
> > approve it; having this change pulled in 202405 stable release? Otherwise
> we will
> > have to wait until next stable release.
> >
> > Thanks
> > Abner
> >
> > > -----Original Message-----
> > > From: Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh@amd.com>
> > > Sent: Thursday, May 2, 2024 3:31 AM
> > > To: devel@edk2.groups.io
> > > Cc: michael.d.kinney@intel.com; gaoliming@byosoft.com.cn;
> > > zhiguang.liu@intel.com; He, Jiangang <Jiangang.He@amd.com>; Chang,
> > > Abner <Abner.Chang@amd.com>; Hsueh, Hong-Chih (Neo) <Hong-
> > > Chih.Hsueh@amd.com>
> > > Subject: [PATCH] MdePkg: Remove non-ASCII characters from header file
> > >
> > > Cc: Jiangang He <jiangang.he@amd.com>
> > > Signed-off-by: Neo Hsueh <Hong-Chih.Hsueh@amd.com>
> > > ---
> > >  MdePkg/Include/Register/Amd/Cpuid.h              | 4 ++--
> > >  MdePkg/Include/Register/Intel/ArchitecturalMsr.h | 8 ++++----
> > >  2 files changed, 6 insertions(+), 6 deletions(-)
> > >
> > > diff --git a/MdePkg/Include/Register/Amd/Cpuid.h
> > > b/MdePkg/Include/Register/Amd/Cpuid.h
> > > index add43c40aa..51fa9f235c 100644
> > > --- a/MdePkg/Include/Register/Amd/Cpuid.h
> > > +++ b/MdePkg/Include/Register/Amd/Cpuid.h
> > > @@ -46,9 +46,9 @@ CPUID Signature Information
> > >    CPUID Extended Topology Enumeration
> > >
> > >    @note
> > > -  Reference: AMD64 Architecture Programmer’s Manual Volume 3:
> General-
> > > Purpose and System Instructions,
> > > +  Reference: AMD64 Architecture Programmer's Manual Volume 3:
> General-
> > > Purpose and System Instructions,
> > >               Revision 3.35 Appendix E,
> > > -  E.4.24 Function 8000_0026—Extended CPU Topology:
> > > +  E.4.24 Function 8000_0026-Extended CPU Topology:
> > >      CPUID Fn8000_0026 reports extended topology information for logical
> > > processors, including
> > >      asymmetric and heterogenous topology descriptions. Individual logical
> > > processors may report
> > >      different values in systems with asynchronous and heterogeneous
> > > topologies.
> > > diff --git a/MdePkg/Include/Register/Intel/ArchitecturalMsr.h
> > > b/MdePkg/Include/Register/Intel/ArchitecturalMsr.h
> > > index 756e7c86ec..4715c59dc4 100644
> > > --- a/MdePkg/Include/Register/Intel/ArchitecturalMsr.h
> > > +++ b/MdePkg/Include/Register/Intel/ArchitecturalMsr.h
> > > @@ -5733,9 +5733,9 @@ typedef union {
> > >      /// [Bit 7:4] TME Policy/Encryption Algorithm: Only algorithms
> > enumerated
> > > in
> > >      /// IA32_TME_CAPABILITY are allowed.
> > >      /// For example:
> > > -    ///   0000 – AES-XTS-128.
> > > -    ///   0001 – AES-XTS-128 with integrity.
> > > -    ///   0010 – AES-XTS-256.
> > > +    ///   0000 - AES-XTS-128.
> > > +    ///   0001 - AES-XTS-128 with integrity.
> > > +    ///   0010 - AES-XTS-256.
> > >      ///   Other values are invalid.
> > >      ///
> > >      UINT32    TmePolicy : 4;
> > > @@ -5756,7 +5756,7 @@ typedef union {
> > >      /// Similar to enumeration, this is an encoded value.
> > >      /// Writing a value greater than MK_TME_MAX_KEYID_BITS will result in
> > > #GP.
> > >      /// Writing a non-zero value to this field will #GP if bit 1 of EAX
> (Hardware
> > > -    /// Encryption Enable) is not also set to ‘1, as encryption hardware must
> > be
> > > +    /// Encryption Enable) is not also set to 1, as encryption hardware must
> > be
> > >      /// enabled to use MKTME.
> > >      /// Example: To support 255 keys, this field would be set to a value of 8.
> > >      ///
> > > --
> > > 2.40.0.windows.1
> >
> >
> >
> > 
> >
>
>



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#118939): https://edk2.groups.io/g/devel/message/118939
Mute This Topic: https://groups.io/mt/106126690/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  reply	other threads:[~2024-05-16  1:12 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-01 19:31 [edk2-devel] [PATCH] MdePkg: Remove non-ASCII characters from header file Hsueh, Hong-Chih (Neo) via groups.io
2024-05-16  0:49 ` [edk2-devel] [PATCH] MdePkg: Remove non-ASCII characters from header file (BZ# 4775) Chang, Abner via groups.io
2024-05-16  1:08   ` 回复: " gaoliming via groups.io
2024-05-16  1:12     ` Chang, Abner via groups.io [this message]
2024-06-04 15:45     ` Hsueh, Hong-Chih (Neo) via groups.io
2024-06-05  9:38       ` 回复: " gaoliming via groups.io

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=LV8PR12MB9452D590B20503B66B64B6B7EAED2@LV8PR12MB9452.namprd12.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