Hi,

Thanks to all the reviewers helping through this patch series. Each individual patch has received reviewed-by tag in this v6 version. It has also just passed all CI build tests here: Unblock mem v6 by kuqin12 · Pull Request #1473 · tianocore/edk2 (github.com)<https://github.com/tianocore/edk2/pull/1473> Could one of the maintainers help to merge these patches into the mainline when you have a chance?

Please let me know if there is anything needed from me to merge in these patches. Thanks in advance!

Regards,
Kun

From: Kun Qin<mailto:kun.q@outlook.com>
Sent: Thursday, March 4, 2021 20:13
To: devel@edk2.groups.io<mailto:devel@edk2.groups.io>
Cc: Michael D Kinney<mailto:michael.d.kinney@intel.com>; Liming Gao<mailto:gaoliming@byosoft.com.cn>; Zhiguang Liu<mailto:zhiguang.liu@intel.com>; Jiewen Yao<mailto:jiewen.yao@intel.com>; Jian J Wang<mailto:jian.j.wang@intel.com>; Hao A Wu<mailto:hao.a.wu@intel.com>; Laszlo Ersek<mailto:lersek@redhat.com>; Ard Biesheuvel<mailto:ardb+tianocore@kernel.org>; Jordan Justen<mailto:jordan.l.justen@intel.com>; Qi Zhang<mailto:qi1.zhang@intel.com>; Rahul Kumar<mailto:rahul1.kumar@intel.com>
Subject: [edk2-devel] [PATCH v6 0/7] Add MmUnblockMemoryLib Interface and Usages

This patch series is a follow up of previous submission:
https://edk2.groups.io/g/devel/message/72442

v6 patches mainly focus on feedback for reviewed commits in v5 patches,
including:
a. Adding "Reviewed-by" and "Acked-by" tags for applicable patch;
b. Updating library class description for newly added interface;

Patch v6 branch: https://github.com/kuqin12/edk2/tree/unblock_mem_v6

Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Qi Zhang <qi1.zhang@intel.com>
Cc: Rahul Kumar <rahul1.kumar@intel.com>

Kun Qin (7):
  MdePkg: MmUnblockMemoryLib: Added definition and null instance
  OvmfPkg: resolve MmUnblockMemoryLib (mainly for VariableSmmRuntimeDxe)
  MdeModulePkg: VariableSmmRuntimeDxe: Added request unblock memory
    interface
  SecurityPkg: Tcg2Smm: Switching from gSmst to gMmst
  SecurityPkg: Tcg2Smm: Separate Tcg2Smm into 2 modules
  SecurityPkg: Tcg2Smm: Added support for Standalone Mm
  SecurityPkg: Tcg2Acpi: Added unblock memory interface for NVS region

 MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmmRuntimeDxe.c   |  42 +
 MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.c           |  44 +
 SecurityPkg/Tcg/{Tcg2Smm/Tcg2Smm.c => Tcg2Acpi/Tcg2Acpi.c}           | 362 ++++-----
 SecurityPkg/Tcg/Tcg2Smm/Tcg2MmDependencyDxe.c                        |  48 ++
 SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c                                    | 857 ++++----------------
 SecurityPkg/Tcg/Tcg2Smm/Tcg2StandaloneMm.c                           |  71 ++
 SecurityPkg/Tcg/Tcg2Smm/Tcg2TraditionalMm.c                          |  82 ++
 MdeModulePkg/MdeModulePkg.dsc                                        |   1 +
 MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmmRuntimeDxe.inf |   1 +
 MdePkg/Include/Library/MmUnblockMemoryLib.h                          |  44 +
 MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.inf         |  34 +
 MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.uni         |  21 +
 MdePkg/MdePkg.dec                                                    |   5 +
 MdePkg/MdePkg.dsc                                                    |   1 +
 OvmfPkg/OvmfPkgIa32.dsc                                              |   3 +
 OvmfPkg/OvmfPkgIa32X64.dsc                                           |   3 +
 OvmfPkg/OvmfPkgX64.dsc                                               |   3 +
 SecurityPkg/Include/Guid/TpmNvsMm.h                                  |  68 ++
 SecurityPkg/SecurityPkg.ci.yaml                                      |   1 +
 SecurityPkg/SecurityPkg.dec                                          |   8 +
 SecurityPkg/SecurityPkg.dsc                                          |  12 +
 SecurityPkg/Tcg/{Tcg2Smm/Tcg2Smm.inf => Tcg2Acpi/Tcg2Acpi.inf}       |  35 +-
 SecurityPkg/Tcg/{Tcg2Smm => Tcg2Acpi}/Tpm.asl                        |   0
 SecurityPkg/Tcg/Tcg2Smm/Tcg2MmDependencyDxe.inf                      |  43 +
 SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.h                                    | 121 +--
 SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.inf                                  |  27 +-
 SecurityPkg/Tcg/Tcg2Smm/{Tcg2Smm.inf => Tcg2StandaloneMm.inf}        |  50 +-
 27 files changed, 952 insertions(+), 1035 deletions(-)
 create mode 100644 MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.c
 copy SecurityPkg/Tcg/{Tcg2Smm/Tcg2Smm.c => Tcg2Acpi/Tcg2Acpi.c} (72%)
 create mode 100644 SecurityPkg/Tcg/Tcg2Smm/Tcg2MmDependencyDxe.c
 create mode 100644 SecurityPkg/Tcg/Tcg2Smm/Tcg2StandaloneMm.c
 create mode 100644 SecurityPkg/Tcg/Tcg2Smm/Tcg2TraditionalMm.c
 create mode 100644 MdePkg/Include/Library/MmUnblockMemoryLib.h
 create mode 100644 MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.inf
 create mode 100644 MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.uni
 create mode 100644 SecurityPkg/Include/Guid/TpmNvsMm.h
 copy SecurityPkg/Tcg/{Tcg2Smm/Tcg2Smm.inf => Tcg2Acpi/Tcg2Acpi.inf} (76%)
 rename SecurityPkg/Tcg/{Tcg2Smm => Tcg2Acpi}/Tpm.asl (100%)
 create mode 100644 SecurityPkg/Tcg/Tcg2Smm/Tcg2MmDependencyDxe.inf
 copy SecurityPkg/Tcg/Tcg2Smm/{Tcg2Smm.inf => Tcg2StandaloneMm.inf} (52%)

--
2.30.0.windows.1