I just add push label for it. 

 

发件人: Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh@amd.com> 
发送时间: 2024年6月4日 23:46
收件人: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io; Chang,
Abner <Abner.Chang@amd.com>
抄送: michael.d.kinney@intel.com; 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 Liming,

 

Sorry for the late response. I already created a pull request for it.

https://github.com/tianocore/edk2/pull/5737

 

Regards,

Neo

  _____  

From: gaoliming <gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn>
>
Sent: Wednesday, May 15, 2024 8:08 PM
To: devel@edk2.groups.io <mailto:devel@edk2.groups.io>
<devel@edk2.groups.io <mailto:devel@edk2.groups.io> >; Chang, Abner
<Abner.Chang@amd.com <mailto:Abner.Chang@amd.com> >; Hsueh, Hong-Chih (Neo)
<Hong-Chih.Hsueh@amd.com <mailto:Hong-Chih.Hsueh@amd.com> >
Cc: michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com>
<michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com> >; zhiguang.
liu@intel.com <mailto:zhiguang.liu@intel.com>  <zhiguang.liu@intel.com
<mailto:zhiguang.liu@intel.com> >; He, Jiangang <Jiangang.He@amd.com
<mailto: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 <mailto: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 <mailto:devel@edk2.groups.io>
<devel@edk2.groups.io <mailto:devel@edk2.groups.io> > 代表 Chang, Abner via
> groups.io
> 发送时间: 2024年5月16日 8:50
> 收件人: Hsueh, Hong-Chih (Neo) <Hong-Chih.Hsueh@amd.com <mailto:Hong-Chih.
Hsueh@amd.com> >;
> devel@edk2.groups.io <mailto:devel@edk2.groups.io> 
> 抄送: michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com> ;
gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> ;
> zhiguang.liu@intel.com <mailto:zhiguang.liu@intel.com> ; He, Jiangang
<Jiangang.He@amd.com <mailto: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 <mailto:Hong-Chih.
Hsueh@amd.com> >
> > Sent: Thursday, May 2, 2024 3:31 AM
> > To: devel@edk2.groups.io <mailto:devel@edk2.groups.io> 
> > Cc: michael.d.kinney@intel.com <mailto:michael.d.kinney@intel.com> ;
gaoliming@byosoft.com.cn <mailto:gaoliming@byosoft.com.cn> ;
> > zhiguang.liu@intel.com <mailto:zhiguang.liu@intel.com> ; He, Jiangang
<Jiangang.He@amd.com <mailto:Jiangang.He@amd.com> >; Chang,
> > Abner <Abner.Chang@amd.com <mailto:Abner.Chang@amd.com> >; Hsueh,
Hong-Chih (Neo) <Hong-
> > Chih.Hsueh@amd.com <mailto:Chih.Hsueh@amd.com> >
> > Subject: [PATCH] MdePkg: Remove non-ASCII characters from header file
> >
> > Cc: Jiangang He <jiangang.he@amd.com <mailto:jiangang.he@amd.com> >
> > Signed-off-by: Neo Hsueh <Hong-Chih.Hsueh@amd.com
<mailto: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 (#119469): https://edk2.groups.io/g/devel/message/119469
Mute This Topic: https://groups.io/mt/106500097/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-