[PATCH v4] MdePkg/Cpuid.h: Define new element in CPUID Leaf(07h) data structure.

[PATCH v4] MdePkg/Cpuid.h: Define new element in CPUID Leaf(07h) data structure.

[Jason Lou]

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3309

Define new element(Hybird) in CPUID_STRUCTURED_EXTENDED_FEATURE_FLAGS
(07h) data structure.

Signed-off-by: Jason Lou <yun.lou@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
---
 MdePkg/Include/Register/Intel/Cpuid.h | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/MdePkg/Include/Register/Intel/Cpuid.h b/MdePkg/Include/Register/Intel/Cpuid.h
index 19af99b6af..6f77e174c1 100644
--- a/MdePkg/Include/Register/Intel/Cpuid.h
+++ b/MdePkg/Include/Register/Intel/Cpuid.h
@@ -6,7 +6,7 @@
   If a register returned is a single 32-bit value, then a data structure is
   not provided for that register.
 
-  Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.<BR>
   SPDX-License-Identifier: BSD-2-Clause-Patent
 
   @par Specification Reference:
@@ -1550,9 +1550,17 @@ typedef union {
     ///
     UINT32  AVX512_4FMAPS:1;
     ///
-    /// [Bit 25:4] Reserved.
+    /// [Bit 14:4] Reserved.
     ///
-    UINT32  Reserved2:22;
+    UINT32  Reserved4:11;
+    ///
+    /// [Bit 15] Hybrid. If 1, the processor is identified as a hybrid part.
+    ///
+    UINT32  Hybrid:1;
+    ///
+    /// [Bit 25:16] Reserved.
+    ///
+    UINT32  Reserved5:10;
     ///
     /// [Bit 26] Enumerates support for indirect branch restricted speculation
     /// (IBRS) and the indirect branch pre-dictor barrier (IBPB). Processors
-- 
2.28.0.windows.1

Re: [edk2-devel] [PATCH v4] MdePkg/Cpuid.h: Define new element in CPUID Leaf(07h) data structure.

[Ni, Ray]

Reviewed-by: Ray Ni <ray.ni@intel.com>

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jason Lou
> Sent: Friday, April 9, 2021 10:01 AM
> To: devel@edk2.groups.io
> Cc: Lou, Yun <yun.lou@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> Liu, Zhiguang <zhiguang.liu@intel.com>; Ni, Ray <ray.ni@intel.com>
> Subject: [edk2-devel] [PATCH v4] MdePkg/Cpuid.h: Define new element in CPUID Leaf(07h) data structure.
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3309
> 
> Define new element(Hybird) in CPUID_STRUCTURED_EXTENDED_FEATURE_FLAGS
> (07h) data structure.
> 
> Signed-off-by: Jason Lou <yun.lou@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> ---
>  MdePkg/Include/Register/Intel/Cpuid.h | 14 +++++++++++---
>  1 file changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git a/MdePkg/Include/Register/Intel/Cpuid.h b/MdePkg/Include/Register/Intel/Cpuid.h
> index 19af99b6af..6f77e174c1 100644
> --- a/MdePkg/Include/Register/Intel/Cpuid.h
> +++ b/MdePkg/Include/Register/Intel/Cpuid.h
> @@ -6,7 +6,7 @@
>    If a register returned is a single 32-bit value, then a data structure is
> 
>    not provided for that register.
> 
> 
> 
> -  Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
> 
> +  Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.<BR>
> 
>    SPDX-License-Identifier: BSD-2-Clause-Patent
> 
> 
> 
>    @par Specification Reference:
> 
> @@ -1550,9 +1550,17 @@ typedef union {
>      ///
> 
>      UINT32  AVX512_4FMAPS:1;
> 
>      ///
> 
> -    /// [Bit 25:4] Reserved.
> 
> +    /// [Bit 14:4] Reserved.
> 
>      ///
> 
> -    UINT32  Reserved2:22;
> 
> +    UINT32  Reserved4:11;
> 
> +    ///
> 
> +    /// [Bit 15] Hybrid. If 1, the processor is identified as a hybrid part.
> 
> +    ///
> 
> +    UINT32  Hybrid:1;
> 
> +    ///
> 
> +    /// [Bit 25:16] Reserved.
> 
> +    ///
> 
> +    UINT32  Reserved5:10;
> 
>      ///
> 
>      /// [Bit 26] Enumerates support for indirect branch restricted speculation
> 
>      /// (IBRS) and the indirect branch pre-dictor barrier (IBPB). Processors
> 
> --
> 2.28.0.windows.1
> 
> 
> 
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#73891): https://edk2.groups.io/g/devel/message/73891
> Mute This Topic: https://groups.io/mt/81959083/1712937
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [ray.ni@intel.com]
> -=-=-=-=-=-=
> 

回复: [edk2-devel] [PATCH v4] MdePkg/Cpuid.h: Define new element in CPUID Leaf(07h) data structure.

[gaoliming]

Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

> -----邮件原件-----
> 发件人: Ni, Ray <ray.ni@intel.com>
> 发送时间: 2021年4月9日 13:02
> 收件人: devel@edk2.groups.io; Lou, Yun <yun.lou@intel.com>
> 抄送: Kinney, Michael D <michael.d.kinney@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> 主题: RE: [edk2-devel] [PATCH v4] MdePkg/Cpuid.h: Define new element in
> CPUID Leaf(07h) data structure.
> 
> Reviewed-by: Ray Ni <ray.ni@intel.com>
> 
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Jason
> Lou
> > Sent: Friday, April 9, 2021 10:01 AM
> > To: devel@edk2.groups.io
> > Cc: Lou, Yun <yun.lou@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> > Liu, Zhiguang <zhiguang.liu@intel.com>; Ni, Ray <ray.ni@intel.com>
> > Subject: [edk2-devel] [PATCH v4] MdePkg/Cpuid.h: Define new element in
> CPUID Leaf(07h) data structure.
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3309
> >
> > Define new element(Hybird) in
> CPUID_STRUCTURED_EXTENDED_FEATURE_FLAGS
> > (07h) data structure.
> >
> > Signed-off-by: Jason Lou <yun.lou@intel.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > Cc: Ray Ni <ray.ni@intel.com>
> > ---
> >  MdePkg/Include/Register/Intel/Cpuid.h | 14 +++++++++++---
> >  1 file changed, 11 insertions(+), 3 deletions(-)
> >
> > diff --git a/MdePkg/Include/Register/Intel/Cpuid.h
> b/MdePkg/Include/Register/Intel/Cpuid.h
> > index 19af99b6af..6f77e174c1 100644
> > --- a/MdePkg/Include/Register/Intel/Cpuid.h
> > +++ b/MdePkg/Include/Register/Intel/Cpuid.h
> > @@ -6,7 +6,7 @@
> >    If a register returned is a single 32-bit value, then a data
structure is
> >
> >    not provided for that register.
> >
> >
> >
> > -  Copyright (c) 2015 - 2019, Intel Corporation. All rights
reserved.<BR>
> >
> > +  Copyright (c) 2015 - 2021, Intel Corporation. All rights
reserved.<BR>
> >
> >    SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> >
> >
> >    @par Specification Reference:
> >
> > @@ -1550,9 +1550,17 @@ typedef union {
> >      ///
> >
> >      UINT32  AVX512_4FMAPS:1;
> >
> >      ///
> >
> > -    /// [Bit 25:4] Reserved.
> >
> > +    /// [Bit 14:4] Reserved.
> >
> >      ///
> >
> > -    UINT32  Reserved2:22;
> >
> > +    UINT32  Reserved4:11;
> >
> > +    ///
> >
> > +    /// [Bit 15] Hybrid. If 1, the processor is identified as a hybrid
part.
> >
> > +    ///
> >
> > +    UINT32  Hybrid:1;
> >
> > +    ///
> >
> > +    /// [Bit 25:16] Reserved.
> >
> > +    ///
> >
> > +    UINT32  Reserved5:10;
> >
> >      ///
> >
> >      /// [Bit 26] Enumerates support for indirect branch restricted
> speculation
> >
> >      /// (IBRS) and the indirect branch pre-dictor barrier (IBPB).
> Processors
> >
> > --
> > 2.28.0.windows.1
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#73891):
> https://edk2.groups.io/g/devel/message/73891
> > Mute This Topic: https://groups.io/mt/81959083/1712937
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub [ray.ni@intel.com]
> > -=-=-=-=-=-=
> >