[v1] Resolve dependency from MmUnblockMemoryLib

[edk2-devel] [edk2-staging][PATCH V1 0/5] Resolve dependency from MmUnblockMemoryLib

Posted by Kun Qin 3 years, 1 month ago

The merging of commit range c5740f360636..59a3ccb09e7a on edk2 master has
separated Tcg2Smm driver into 2 functional componnents, Tcg2Acpi and Tcg2
MM driver. They also introduced dependency of MmUnblockMemoryLib on
VariableSmmRuntimeDxe as well as the new Tcg2Acpi driver.

This patch series is sent to resolve the dependency mentioned above.

Patch v1 branch: https://github.com/kuqin12/edk2-platforms/tree/unblock_dependency_v1

Cc: Chasel Chiu <chasel.chiu@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Kelly Steele <kelly.steele@intel.com>
Cc: Zailiang Sun <zailiang.sun@intel.com>
Cc: Yi Qian <yi.qian@intel.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Leif Lindholm <leif@nuviainc.com>

Kun Qin (5):
  MinPlatformPkg: CoreCommonLib: Added new library for
    VariableSmmRuntimeDxe
  MinPlatformPkg: Core Include Files: Added Tcg2Acpi driver after
    separation
  QuarkPlatformPkg: Quark DSC File: Added new library for
    VariableSmmRuntimeDxe
  Vlv2TbltDevicePkg: PlatformPkg DSC: Added library for
    VariableSmmRuntimeDxe
  Socionext: DeveloperBox DSC File: Added library for
    VariableSmmRuntimeDxe

 Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc           | 1 +
 Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc          | 1 +
 Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf | 3 ++-
 Platform/Intel/QuarkPlatformPkg/Quark.dsc                             | 1 +
 Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc                  | 1 +
 Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc                   | 1 +
 Platform/Socionext/DeveloperBox/DeveloperBox.dsc                      | 2 ++
 7 files changed, 9 insertions(+), 1 deletion(-)

-- 
2.30.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72645): https://edk2.groups.io/g/devel/message/72645
Mute This Topic: https://groups.io/mt/81243531/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] [edk2-staging][PATCH V1 0/5] Resolve dependency from MmUnblockMemoryLib

Posted by Thomas Abraham 3 years, 1 month ago

On 3/11/21 1:37 AM, Kun Qin via groups.io wrote: 
> The merging of commit range c5740f360636..59a3ccb09e7a on edk2 master has
> separated Tcg2Smm driver into 2 functional componnents, Tcg2Acpi and Tcg2
> MM driver. They also introduced dependency of MmUnblockMemoryLib on
> VariableSmmRuntimeDxe as well as the new Tcg2Acpi driver.
> 
> This patch series is sent to resolve the dependency mentioned above.
> 
> Patch v1 branch: https://github.com/kuqin12/edk2-
> platforms/tree/unblock_dependency_v1
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Kelly Steele <kelly.steele@intel.com>
> Cc: Zailiang Sun <zailiang.sun@intel.com>
> Cc: Yi Qian <yi.qian@intel.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> 
> Kun Qin (5):
>   MinPlatformPkg: CoreCommonLib: Added new library for
>     VariableSmmRuntimeDxe
>   MinPlatformPkg: Core Include Files: Added Tcg2Acpi driver after
>     separation
>   QuarkPlatformPkg: Quark DSC File: Added new library for
>     VariableSmmRuntimeDxe
>   Vlv2TbltDevicePkg: PlatformPkg DSC: Added library for
>     VariableSmmRuntimeDxe
>   Socionext: DeveloperBox DSC File: Added library for
>     VariableSmmRuntimeDxe
> 
>  Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc           | 1 +
>  Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc          | 1 +
>  Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf | 3 ++-
>  Platform/Intel/QuarkPlatformPkg/Quark.dsc                             | 1 +
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc                  | 1 +
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc                   | 1 +
>  Platform/Socionext/DeveloperBox/DeveloperBox.dsc                      | 2 ++
>  7 files changed, 9 insertions(+), 1 deletion(-)

Can this series be extended to resolve the dependency in Platform/Arm/SgiPkg as well? The change being applied in this patch series was tested with SgiPkg it does resolve the dependency.

Thanks,
Thomas.



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72660): https://edk2.groups.io/g/devel/message/72660
Mute This Topic: https://groups.io/mt/81243531/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-