回复: [edk2-devel] [Patch v2 0/2] MdeModulePkg/Core/Dxe: Set MemoryTypeInfo bin range from HOB

["gaoliming via groups.io" <gaoliming=byosoft.com.cn@groups.io>]

Mike:
  Thanks for your update. This patch set is good to me. Reviewed-by: Liming
Gao <gaoliming@byosoft.com.cn>

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Michael D
> Kinney
> 发送时间: 2024年1月29日 11:45
> 收件人: devel@edk2.groups.io
> 抄送: Liming Gao <gaoliming@byosoft.com.cn>; Aaron Li
> <aaron.li@intel.com>; Liu Yun <yun.y.liu@intel.com>; Andrew Fish
> <afish@apple.com>
> 主题: [edk2-devel] [Patch v2 0/2] MdeModulePkg/Core/Dxe: Set
> MemoryTypeInfo bin range from HOB
> 
> New in V2
> =========
> * Break single patch into 2 patches
>   * Moves GCD Initialization before RT data allocations
>   * Set MemoryTypeInfo bin range from HOB feature
> * Update description of Memory Type Information GUID to
>   describe additional use case as Owner GUID in a Resource
>   Descriptor HOB for preferred memory range for Memory Type
>   Information bins.
> 
> Provide an optional method for PEI to declare a specific address
> range to use for the Memory Type Information bins. The current
> algorithm uses heuristics that tends to place the Memory Type
> Information bins in the same location, but memory configuration
> changes across boots or algorithm changes across a firmware
> updates could potentially change the Memory Type Information bin
> location.
> 
> If the HOB List contains a Resource Descriptor HOB that
> describes tested system memory and has an Owner GUID of
> gEfiMemoryTypeInformationGuid, then use the address range
> described by the Resource Descriptor HOB as the preferred
> location of the Memory Type Information bins. If this HOB is
> not detected, then the current behavior is preserved.
> 
> The HOB with an Owner GUID of gEfiMemoryTypeInformationGuid
> is ignored for the following conditions:
> * The HOB with an Owner GUID of gEfiMemoryTypeInformationGuid
>   is smaller than the Memory Type Information bins.
> * The HOB list contains more than one Resource Descriptor HOB
>   with an owner GUID of gEfiMemoryTypeInformationGuid.
> * The Resource Descriptor HOB with an Owner GUID of
>   gEfiMemoryTypeInformationGuid is the same Resource Descriptor
>   HOB that that describes the PHIT memory range.
> 
> Update the DxeMain initialization order to initialize GCD
> services before any runtime allocations are performed.  This
> is required to prevent runtime data fragmentation when the
> UEFI System Table and UEFI Runtime Service Table are allocated
> before both the memory and GCD services are initialized.
> 
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Aaron Li <aaron.li@intel.com>
> Cc: Liu Yun <yun.y.liu@intel.com>
> Cc: Andrew Fish <afish@apple.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> 
> Michael D Kinney (2):
>   MdeModulePkg/Core/Dxe: Initialize GCD before RT memory allocations
>   MdeModulePkg/Core/Dxe: Set MemoryTypeInfo bin range from HOB
> 
>  MdeModulePkg/Core/Dxe/DxeMain.h               |   6 ++
>  MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c       |  23 ++--
>  MdeModulePkg/Core/Dxe/Gcd/Gcd.c               |  72
> ++++++++++++-
>  MdeModulePkg/Core/Dxe/Image/Image.c           |   1 -
>  MdeModulePkg/Core/Dxe/Mem/Page.c              | 101
> ++++++++++++++++++
>  .../Include/Guid/MemoryTypeInformation.h      |  14 ++-
>  6 files changed, 199 insertions(+), 18 deletions(-)
> 
> --
> 2.40.1.windows.1
> 
> 
> 
> 





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