Re: [edk2-devel] [PATCH v1 0/1] Introduce DxeMmUnblockMemoryLib Interface

["Wu, Hao A" <hao.a.wu@intel.com>]

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Kun Qin
> Sent: Wednesday, February 3, 2021 6:16 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.wu@intel.com>;
> Dong, Eric <eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Yao, Jiewen
> <jiewen.yao@intel.com>
> Subject: [edk2-devel] [PATCH v1 0/1] Introduce DxeMmUnblockMemoryLib
> Interface
> 
> The interface proposed in this patch series intends to provide an abstraction
> layer for DXE drivers to request certain memory regions to be accessible from
> inside MM environment that applies total memory blockage.
> 
> This abstraction could pave way for models such as Standalone MM to
> manage memory resources without having knowledge of DXE memory map
> inside MM environment.
> 
> Example usages of it can be NVS region in Tcg2Smm and runtime variable
> cache regions in VariableSmmRuntimeDxe.


My thought is that it might be more helpful if the whole implementation proposal
to address BZ-3168 can be provided before reviewing the interfaces for the new
library (or the library itself).

Hello Jiewen,

Do you have comments on the approach on implementing the BZ-3168
(https://bugzilla.tianocore.org/show_bug.cgi?id=3168) feature?
Thanks in advance.

Best Regards,
Hao Wu


> 
> Patch v1 branch: https://github.com/kuqin12/edk2/tree/unblock_mem_v1
> 
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> 
> Kun Qin (1):
>   MdeModulePkg: DxeMmUnblockMemoryLib: Added definition and null
>     instance
> 
> 
> MdeModulePkg/Library/DxeMmUnblockMemoryLib/DxeMmUnblockMemo
> ryLibNull.c   | 40 ++++++++++++++++++++
>  MdeModulePkg/Include/Library/DxeMmUnblockMemoryLib.h                     |
> 40 ++++++++++++++++++++
> 
> MdeModulePkg/Library/DxeMmUnblockMemoryLib/DxeMmUnblockMemo
> ryLibNull.inf | 29 ++++++++++++++
>  MdeModulePkg/MdeModulePkg.dec                                            |  5 +++
>  MdeModulePkg/MdeModulePkg.dsc                                            |  2 +
>  5 files changed, 116 insertions(+)
>  create mode 100644
> MdeModulePkg/Library/DxeMmUnblockMemoryLib/DxeMmUnblockMemo
> ryLibNull.c
>  create mode 100644
> MdeModulePkg/Include/Library/DxeMmUnblockMemoryLib.h
>  create mode 100644
> MdeModulePkg/Library/DxeMmUnblockMemoryLib/DxeMmUnblockMemo
> ryLibNull.inf
> 
> --
> 2.30.0.windows.1
> 
> 
> 
> 
>