public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition
@ 2023-12-08  5:37 ManickamX Srinivasan
  2023-12-08  5:48 ` Zhiguang Liu
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: ManickamX Srinivasan @ 2023-12-08  5:37 UTC (permalink / raw)
  To: devel
  Cc: ManickamX Srinivasan, Michael D Kinney, Liming Gao, Zhiguang Liu,
	T V Krishnamoorthy

New memory type as defined in UEFI standard v2.10

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
---
 MdePkg/Include/Uefi/UefiSpec.h | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/MdePkg/Include/Uefi/UefiSpec.h b/MdePkg/Include/Uefi/UefiSpec.h
index 7dfe35b499..d583ee17d0 100644
--- a/MdePkg/Include/Uefi/UefiSpec.h
+++ b/MdePkg/Include/Uefi/UefiSpec.h
@@ -110,6 +110,21 @@ typedef enum {
 //
 #define EFI_MEMORY_RUNTIME  0x8000000000000000ULL
 
+//
+// If this flag is set, the memory region is
+// described with additional ISA-specific memory attributes 
+// as specified in EFI_MEMORY_ISA_MASK.
+//
+#define EFI_MEMORY_ISA_VALID 0x4000000000000000ULL
+
+//
+// Defines the bits reserved for describing optional ISA-specific cacheability 
+// attributes that are not covered by the standard UEFI Memory Attributes cacheability
+// bits (EFI_MEMORY_UC, EFI_MEMORY_WC, EFI_MEMORY_WT, EFI_MEMORY_WB and EFI_MEMORY_UCE).
+// See Calling Conventions for further ISA-specific enumeration of these bits.
+//
+#define EFI_MEMORY_ISA_MASK 0x0FFFF00000000000ULL
+
 //
 // Attributes bitmasks, grouped by type
 //
-- 
2.30.2.windows.1



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



^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition
  2023-12-08  5:37 [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition ManickamX Srinivasan
@ 2023-12-08  5:48 ` Zhiguang Liu
  2023-12-08 20:23 ` Michael D Kinney
  2023-12-12  0:27 ` [edk2-devel] 回复: " gaoliming via groups.io
  2 siblings, 0 replies; 5+ messages in thread
From: Zhiguang Liu @ 2023-12-08  5:48 UTC (permalink / raw)
  To: ManickamX Srinivasan, devel

[-- Attachment #1: Type: text/plain, Size: 420 bytes --]

Reviewed-by: Zhiguang Liu < zhiguang.liu@intel.com >


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



[-- Attachment #2: Type: text/html, Size: 908 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition
  2023-12-08  5:37 [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition ManickamX Srinivasan
  2023-12-08  5:48 ` Zhiguang Liu
@ 2023-12-08 20:23 ` Michael D Kinney
  2023-12-09  3:21   ` Michael D Kinney
  2023-12-12  0:27 ` [edk2-devel] 回复: " gaoliming via groups.io
  2 siblings, 1 reply; 5+ messages in thread
From: Michael D Kinney @ 2023-12-08 20:23 UTC (permalink / raw)
  To: Srinivasan, ManickamX, devel@edk2.groups.io
  Cc: Gao, Liming, Liu, Zhiguang, T V, Krishnamoorthy,
	Kinney, Michael D

Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>
> Sent: Thursday, December 7, 2023 9:37 PM
> To: devel@edk2.groups.io
> Cc: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>; Kinney,
> Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; T V,
> Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Subject: [PATCH v3] MdePkg: Add a new memory type definition
> 
> New memory type as defined in UEFI standard v2.10
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> ---
>  MdePkg/Include/Uefi/UefiSpec.h | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> b/MdePkg/Include/Uefi/UefiSpec.h
> index 7dfe35b499..d583ee17d0 100644
> --- a/MdePkg/Include/Uefi/UefiSpec.h
> +++ b/MdePkg/Include/Uefi/UefiSpec.h
> @@ -110,6 +110,21 @@ typedef enum {
>  //
>  #define EFI_MEMORY_RUNTIME  0x8000000000000000ULL
> 
> +//
> +// If this flag is set, the memory region is
> +// described with additional ISA-specific memory attributes
> +// as specified in EFI_MEMORY_ISA_MASK.
> +//
> +#define EFI_MEMORY_ISA_VALID 0x4000000000000000ULL
> +
> +//
> +// Defines the bits reserved for describing optional ISA-specific
> cacheability
> +// attributes that are not covered by the standard UEFI Memory
> Attributes cacheability
> +// bits (EFI_MEMORY_UC, EFI_MEMORY_WC, EFI_MEMORY_WT, EFI_MEMORY_WB and
> EFI_MEMORY_UCE).
> +// See Calling Conventions for further ISA-specific enumeration of
> these bits.
> +//
> +#define EFI_MEMORY_ISA_MASK 0x0FFFF00000000000ULL
> +
>  //
>  // Attributes bitmasks, grouped by type
>  //
> --
> 2.30.2.windows.1



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



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition
  2023-12-08 20:23 ` Michael D Kinney
@ 2023-12-09  3:21   ` Michael D Kinney
  0 siblings, 0 replies; 5+ messages in thread
From: Michael D Kinney @ 2023-12-09  3:21 UTC (permalink / raw)
  To: Srinivasan, ManickamX, devel@edk2.groups.io
  Cc: Gao, Liming, Liu, Zhiguang, T V, Krishnamoorthy,
	Kinney, Michael D

Merged: https://github.com/tianocore/edk2/pull/5128


> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Friday, December 8, 2023 12:23 PM
> To: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>;
> devel@edk2.groups.io
> Cc: Gao, Liming <gaoliming@byosoft.com.cn>; Liu, Zhiguang
> <zhiguang.liu@intel.com>; T V, Krishnamoorthy
> <krishnamoorthy.t.v@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Subject: RE: [PATCH v3] MdePkg: Add a new memory type definition
> 
> Reviewed-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> > -----Original Message-----
> > From: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>
> > Sent: Thursday, December 7, 2023 9:37 PM
> > To: devel@edk2.groups.io
> > Cc: Srinivasan, ManickamX <manickamx.srinivasan@intel.com>; Kinney,
> > Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>; T
> V,
> > Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> > Subject: [PATCH v3] MdePkg: Add a new memory type definition
> >
> > New memory type as defined in UEFI standard v2.10
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> > Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> > ---
> >  MdePkg/Include/Uefi/UefiSpec.h | 15 +++++++++++++++
> >  1 file changed, 15 insertions(+)
> >
> > diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> > b/MdePkg/Include/Uefi/UefiSpec.h
> > index 7dfe35b499..d583ee17d0 100644
> > --- a/MdePkg/Include/Uefi/UefiSpec.h
> > +++ b/MdePkg/Include/Uefi/UefiSpec.h
> > @@ -110,6 +110,21 @@ typedef enum {
> >  //
> >  #define EFI_MEMORY_RUNTIME  0x8000000000000000ULL
> >
> > +//
> > +// If this flag is set, the memory region is
> > +// described with additional ISA-specific memory attributes
> > +// as specified in EFI_MEMORY_ISA_MASK.
> > +//
> > +#define EFI_MEMORY_ISA_VALID 0x4000000000000000ULL
> > +
> > +//
> > +// Defines the bits reserved for describing optional ISA-specific
> > cacheability
> > +// attributes that are not covered by the standard UEFI Memory
> > Attributes cacheability
> > +// bits (EFI_MEMORY_UC, EFI_MEMORY_WC, EFI_MEMORY_WT, EFI_MEMORY_WB
> and
> > EFI_MEMORY_UCE).
> > +// See Calling Conventions for further ISA-specific enumeration of
> > these bits.
> > +//
> > +#define EFI_MEMORY_ISA_MASK 0x0FFFF00000000000ULL
> > +
> >  //
> >  // Attributes bitmasks, grouped by type
> >  //
> > --
> > 2.30.2.windows.1



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



^ permalink raw reply	[flat|nested] 5+ messages in thread

* [edk2-devel] 回复: [PATCH v3] MdePkg: Add a new memory type definition
  2023-12-08  5:37 [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition ManickamX Srinivasan
  2023-12-08  5:48 ` Zhiguang Liu
  2023-12-08 20:23 ` Michael D Kinney
@ 2023-12-12  0:27 ` gaoliming via groups.io
  2 siblings, 0 replies; 5+ messages in thread
From: gaoliming via groups.io @ 2023-12-12  0:27 UTC (permalink / raw)
  To: 'ManickamX Srinivasan', devel
  Cc: 'Michael D Kinney', 'Zhiguang Liu',
	'T V Krishnamoorthy'

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

> -----邮件原件-----
> 发件人: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> 发送时间: 2023年12月8日 13:37
> 收件人: devel@edk2.groups.io
> 抄送: ManickamX Srinivasan <manickamx.srinivasan@intel.com>; Michael D
> Kinney <michael.d.kinney@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Zhiguang Liu <zhiguang.liu@intel.com>; T V
> Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> 主题: [PATCH v3] MdePkg: Add a new memory type definition
> 
> New memory type as defined in UEFI standard v2.10
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: T V Krishnamoorthy <krishnamoorthy.t.v@intel.com>
> Signed-off-by: ManickamX Srinivasan <manickamx.srinivasan@intel.com>
> ---
>  MdePkg/Include/Uefi/UefiSpec.h | 15 +++++++++++++++
>  1 file changed, 15 insertions(+)
> 
> diff --git a/MdePkg/Include/Uefi/UefiSpec.h
> b/MdePkg/Include/Uefi/UefiSpec.h
> index 7dfe35b499..d583ee17d0 100644
> --- a/MdePkg/Include/Uefi/UefiSpec.h
> +++ b/MdePkg/Include/Uefi/UefiSpec.h
> @@ -110,6 +110,21 @@ typedef enum {
>  //
>  #define EFI_MEMORY_RUNTIME  0x8000000000000000ULL
> 
> +//
> +// If this flag is set, the memory region is
> +// described with additional ISA-specific memory attributes
> +// as specified in EFI_MEMORY_ISA_MASK.
> +//
> +#define EFI_MEMORY_ISA_VALID 0x4000000000000000ULL
> +
> +//
> +// Defines the bits reserved for describing optional ISA-specific
cacheability
> +// attributes that are not covered by the standard UEFI Memory Attributes
> cacheability
> +// bits (EFI_MEMORY_UC, EFI_MEMORY_WC, EFI_MEMORY_WT,
> EFI_MEMORY_WB and EFI_MEMORY_UCE).
> +// See Calling Conventions for further ISA-specific enumeration of these
bits.
> +//
> +#define EFI_MEMORY_ISA_MASK 0x0FFFF00000000000ULL
> +
>  //
>  // Attributes bitmasks, grouped by type
>  //
> --
> 2.30.2.windows.1





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



^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2023-12-12  0:27 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-08  5:37 [edk2-devel] [PATCH v3] MdePkg: Add a new memory type definition ManickamX Srinivasan
2023-12-08  5:48 ` Zhiguang Liu
2023-12-08 20:23 ` Michael D Kinney
2023-12-09  3:21   ` Michael D Kinney
2023-12-12  0:27 ` [edk2-devel] 回复: " gaoliming via groups.io

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox