mbox series

[edk2,v4,0/4] MdePkg, MdeModulePkg: Enforce page allocation granularity

Message ID 1488553894-3520-1-git-send-email-ard.biesheuvel@linaro.org
Headers show
Series MdePkg, MdeModulePkg: Enforce page allocation granularity | expand

Message

Ard Biesheuvel March 3, 2017, 3:11 p.m. UTC
This is now a 4-piece series, so that the allocation granularity used by
PEI core, which is defined per-architecture in the UEFI spec, is not defined
locally but centrally for all modules in all phases.

Ard Biesheuvel (4):
  MdePkg/ProcessorBind: add defines for page allocation granularity
  MdeModulePkg/DxeCore: switch to MdePkg allocation granularity macros
  MdeModulePkg/PiSmmCore: switch to MdePkg allocation granularity macros
  MdeModulePkg/PeiCore: honour minimal runtime allocation granularity

 MdeModulePkg/Core/Dxe/DxeMain.h                     | 25 -------------
 MdeModulePkg/Core/Dxe/Mem/Page.c                    | 22 +++++------
 MdeModulePkg/Core/Dxe/Mem/Pool.c                    |  8 ++--
 MdeModulePkg/Core/Dxe/Misc/MemoryAttributesTable.c  |  2 +-
 MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c       |  4 +-
 MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c        |  8 ++--
 MdeModulePkg/Core/Pei/Memory/MemoryServices.c       | 39 +++++++++++++++++++-
 MdeModulePkg/Core/PiSmmCore/MemoryAttributesTable.c |  6 +--
 MdeModulePkg/Core/PiSmmCore/Page.c                  |  4 +-
 MdeModulePkg/Core/PiSmmCore/PiSmmCore.h             |  6 ---
 MdePkg/Include/AArch64/ProcessorBind.h              |  6 +++
 MdePkg/Include/Arm/ProcessorBind.h                  |  6 +++
 MdePkg/Include/Ebc/ProcessorBind.h                  |  6 +++
 MdePkg/Include/Ia32/ProcessorBind.h                 |  6 +++
 MdePkg/Include/Ipf/ProcessorBind.h                  |  6 +++
 MdePkg/Include/X64/ProcessorBind.h                  |  6 +++
 16 files changed, 100 insertions(+), 60 deletions(-)

-- 
2.7.4

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Comments

Yao, Jiewen March 3, 2017, 3:31 p.m. UTC | #1
Thank you.
All reviewed-by: Jiewen.yao@intel.com

> -----Original Message-----

> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]

> Sent: Friday, March 3, 2017 11:12 PM

> To: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com>; Kinney,

> Michael D <michael.d.kinney@intel.com>

> Cc: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>; Ard

> Biesheuvel <ard.biesheuvel@linaro.org>

> Subject: [PATCH v4 0/4] MdePkg, MdeModulePkg: Enforce page allocation

> granularity

> 

> This is now a 4-piece series, so that the allocation granularity used by

> PEI core, which is defined per-architecture in the UEFI spec, is not defined

> locally but centrally for all modules in all phases.

> 

> Ard Biesheuvel (4):

>   MdePkg/ProcessorBind: add defines for page allocation granularity

>   MdeModulePkg/DxeCore: switch to MdePkg allocation granularity macros

>   MdeModulePkg/PiSmmCore: switch to MdePkg allocation granularity macros

>   MdeModulePkg/PeiCore: honour minimal runtime allocation granularity

> 

>  MdeModulePkg/Core/Dxe/DxeMain.h                     | 25 -------------

>  MdeModulePkg/Core/Dxe/Mem/Page.c                    | 22 +++++------

>  MdeModulePkg/Core/Dxe/Mem/Pool.c                    |  8 ++--

>  MdeModulePkg/Core/Dxe/Misc/MemoryAttributesTable.c  |  2 +-

>  MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c       |  4 +-

>  MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c        |  8 ++--

>  MdeModulePkg/Core/Pei/Memory/MemoryServices.c       | 39

> +++++++++++++++++++-

>  MdeModulePkg/Core/PiSmmCore/MemoryAttributesTable.c |  6 +--

>  MdeModulePkg/Core/PiSmmCore/Page.c                  |  4 +-

>  MdeModulePkg/Core/PiSmmCore/PiSmmCore.h             |  6 ---

>  MdePkg/Include/AArch64/ProcessorBind.h              |  6 +++

>  MdePkg/Include/Arm/ProcessorBind.h                  |  6 +++

>  MdePkg/Include/Ebc/ProcessorBind.h                  |  6 +++

>  MdePkg/Include/Ia32/ProcessorBind.h                 |  6 +++

>  MdePkg/Include/Ipf/ProcessorBind.h                  |  6 +++

>  MdePkg/Include/X64/ProcessorBind.h                  |  6 +++

>  16 files changed, 100 insertions(+), 60 deletions(-)

> 

> --

> 2.7.4


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Gao, Liming March 6, 2017, 2:33 a.m. UTC | #2
Reviewed-by: Liming Gao <liming.gao@intel.com>


>-----Original Message-----

>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]

>Sent: Friday, March 03, 2017 11:12 PM

>To: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com>; Kinney,

>Michael D <michael.d.kinney@intel.com>

>Cc: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>;

>Ard Biesheuvel <ard.biesheuvel@linaro.org>

>Subject: [PATCH v4 0/4] MdePkg, MdeModulePkg: Enforce page allocation

>granularity

>

>This is now a 4-piece series, so that the allocation granularity used by

>PEI core, which is defined per-architecture in the UEFI spec, is not defined

>locally but centrally for all modules in all phases.

>

>Ard Biesheuvel (4):

>  MdePkg/ProcessorBind: add defines for page allocation granularity

>  MdeModulePkg/DxeCore: switch to MdePkg allocation granularity macros

>  MdeModulePkg/PiSmmCore: switch to MdePkg allocation granularity macros

>  MdeModulePkg/PeiCore: honour minimal runtime allocation granularity

>

> MdeModulePkg/Core/Dxe/DxeMain.h                     | 25 -------------

> MdeModulePkg/Core/Dxe/Mem/Page.c                    | 22 +++++------

> MdeModulePkg/Core/Dxe/Mem/Pool.c                    |  8 ++--

> MdeModulePkg/Core/Dxe/Misc/MemoryAttributesTable.c  |  2 +-

> MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c       |  4 +-

> MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c        |  8 ++--

> MdeModulePkg/Core/Pei/Memory/MemoryServices.c       | 39

>+++++++++++++++++++-

> MdeModulePkg/Core/PiSmmCore/MemoryAttributesTable.c |  6 +--

> MdeModulePkg/Core/PiSmmCore/Page.c                  |  4 +-

> MdeModulePkg/Core/PiSmmCore/PiSmmCore.h             |  6 ---

> MdePkg/Include/AArch64/ProcessorBind.h              |  6 +++

> MdePkg/Include/Arm/ProcessorBind.h                  |  6 +++

> MdePkg/Include/Ebc/ProcessorBind.h                  |  6 +++

> MdePkg/Include/Ia32/ProcessorBind.h                 |  6 +++

> MdePkg/Include/Ipf/ProcessorBind.h                  |  6 +++

> MdePkg/Include/X64/ProcessorBind.h                  |  6 +++

> 16 files changed, 100 insertions(+), 60 deletions(-)

>

>--

>2.7.4


_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Ard Biesheuvel March 6, 2017, 3:14 p.m. UTC | #3
On 6 March 2017 at 03:33, Gao, Liming <liming.gao@intel.com> wrote:
> Reviewed-by: Liming Gao <liming.gao@intel.com>

>


Thanks all. Pushed to master

>>-----Original Message-----

>>From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]

>>Sent: Friday, March 03, 2017 11:12 PM

>>To: edk2-devel@lists.01.org; Gao, Liming <liming.gao@intel.com>; Kinney,

>>Michael D <michael.d.kinney@intel.com>

>>Cc: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>;

>>Ard Biesheuvel <ard.biesheuvel@linaro.org>

>>Subject: [PATCH v4 0/4] MdePkg, MdeModulePkg: Enforce page allocation

>>granularity

>>

>>This is now a 4-piece series, so that the allocation granularity used by

>>PEI core, which is defined per-architecture in the UEFI spec, is not defined

>>locally but centrally for all modules in all phases.

>>

>>Ard Biesheuvel (4):

>>  MdePkg/ProcessorBind: add defines for page allocation granularity

>>  MdeModulePkg/DxeCore: switch to MdePkg allocation granularity macros

>>  MdeModulePkg/PiSmmCore: switch to MdePkg allocation granularity macros

>>  MdeModulePkg/PeiCore: honour minimal runtime allocation granularity

>>

>> MdeModulePkg/Core/Dxe/DxeMain.h                     | 25 -------------

>> MdeModulePkg/Core/Dxe/Mem/Page.c                    | 22 +++++------

>> MdeModulePkg/Core/Dxe/Mem/Pool.c                    |  8 ++--

>> MdeModulePkg/Core/Dxe/Misc/MemoryAttributesTable.c  |  2 +-

>> MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c       |  4 +-

>> MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c        |  8 ++--

>> MdeModulePkg/Core/Pei/Memory/MemoryServices.c       | 39

>>+++++++++++++++++++-

>> MdeModulePkg/Core/PiSmmCore/MemoryAttributesTable.c |  6 +--

>> MdeModulePkg/Core/PiSmmCore/Page.c                  |  4 +-

>> MdeModulePkg/Core/PiSmmCore/PiSmmCore.h             |  6 ---

>> MdePkg/Include/AArch64/ProcessorBind.h              |  6 +++

>> MdePkg/Include/Arm/ProcessorBind.h                  |  6 +++

>> MdePkg/Include/Ebc/ProcessorBind.h                  |  6 +++

>> MdePkg/Include/Ia32/ProcessorBind.h                 |  6 +++

>> MdePkg/Include/Ipf/ProcessorBind.h                  |  6 +++

>> MdePkg/Include/X64/ProcessorBind.h                  |  6 +++

>> 16 files changed, 100 insertions(+), 60 deletions(-)

>>

>>--

>>2.7.4

>

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel