Re: [edk2-devel] [edk2-platforms] Intel/MinPlatformPkg: resolve MmUnblockMemoryLib

["Kun Qin" <kuqin12@gmail.com>]

Hi Zhiguang,

I have already sent this patch series to resolve dependencies in 
edk2-platform (although the change is slightly different):
https://edk2.groups.io/g/devel/message/72645

Specifically:
https://edk2.groups.io/g/devel/message/72646

Could you please let me know if the change above resolves the dependency 
issue for you?

The patch series have 2 other patches not being reviewed yet. I plan to 
send out another round tomorrow to include reviewed-by tags. But please 
let me know if you prefer your change below to check in first, I will 
just drop my patch #1 when sending v2.

Thanks,
Kun

On 03/16/2021 02:06, Zhiguang Liu wrote:
> The below Edk2 patch makes VariableSmmRuntimeDxe begin to consume MmUnblockMemoryLib.
> It cause multiple platforms build failure.
>      f463dbadede138dc96a66dae6f361c54f0b3093c
>      MdeModulePkg: VariableSmmRuntimeDxe: Added request unblock memory interface
> 
> This change added NULL MmUnblockMemoryLib instance in MinPlatformPkg dsc include files
> 
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> 
> Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
>   Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> index fa9098d525..ee91dd8bd6 100644
> --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeLib.dsc
> @@ -116,6 +116,7 @@
>   !endif
> 
>     BaseCryptLib|CryptoPkg/Library/BaseCryptLib/RuntimeCryptLib.inf
> 
>     VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLibRuntimeDxe.inf
> 
> +  MmUnblockMemoryLib|MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.inf
> 
>   
> 
>   [LibraryClasses.common.UEFI_DRIVER]
> 
>     Tpm2DeviceLib|SecurityPkg/Library/Tpm2DeviceLibTcg2/Tpm2DeviceLibTcg2.inf
>