[edk2-devel] [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
Li, Zhihao
zhihao.li at intel.com
Fri Apr 1 17:10:24 UTC 2022
Yes, some patches in other Pkg.
Patch in UefiCpuPkg have been merged.
It provide a mode allow cpu which in SMM mode work without waiting for other APs enter SMM mode.
And define a new service that can be used by SMI handler to optionally wait for other APs to complete SMM rendezvous.
Patch in UefiPayloadPkg in review.
Use VariableSmm driver so that need to add SmmCpuRendezvousLib dependency.
Patch in MdeModulePkg is blocked.
VariableSmm/VariableStandaloneMM driver needs use new service complete SMM rendezvous.
Patch in SecurityPkg is blocked.
TcgSmm/Tcg2Smm driver need use new service complete SMM rendezvous.
Thanks a lot.
-----Original Message-----
From: Yao, Jiewen <jiewen.yao at intel.com>
Sent: Friday, April 1, 2022 7:35 PM
To: Li, Zhihao <zhihao.li at intel.com>; devel at edk2.groups.io
Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>; Justen, Jordan L <jordan.l.justen at intel.com>; Gerd Hoffmann <kraxel at redhat.com>; Boeuf, Sebastien <sebastien.boeuf at intel.com>
Subject: RE: [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
Reviewed-by: Jiewen Yao <Jiewen.yao at intel.com>
Is this a standalone patch? Or one of a series patch?
I think it should be later. But, please enlighten me.
> -----Original Message-----
> From: Li, Zhihao <zhihao.li at intel.com>
> Sent: Tuesday, March 29, 2022 2:38 PM
> To: devel at edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>; Yao, Jiewen
> <jiewen.yao at intel.com>; Justen, Jordan L <jordan.l.justen at intel.com>;
> Gerd Hoffmann <kraxel at redhat.com>; Boeuf, Sebastien
> <sebastien.boeuf at intel.com>
> Subject: [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver
> to make it work normally.
>
> REF： https://bugzilla.tianocore.org/show_bug.cgi?id=3861
>
> UefiCpuPkg define a new Protocol with the new services
> SmmWaitForAllProcessor(), which can be used by SMI handler to
> optionally wait for other APs to complete SMM rendezvous in relaxed AP
> mode.
>
> VariableSmm driver need use SmmCpuRendezvousLib, So add
> SmmCpuRendezvousLib in OvmfPkg.
>
> Cc: Ard Biesheuvel <ardb+tianocore at kernel.org>
> Cc: Jiewen Yao <jiewen.yao at intel.com>
> Cc: Jordan Justen <jordan.l.justen at intel.com>
> Cc: Gerd Hoffmann <kraxel at redhat.com>
> Cc: Sebastien Boeuf <sebastien.boeuf at intel.com>
>
> Signed-off-by: Zhihao Li <zhihao.li at intel.com>
> ---
> OvmfPkg/CloudHv/CloudHvX64.dsc | 1 +
> OvmfPkg/OvmfPkgIa32.dsc | 3 ++-
> OvmfPkg/OvmfPkgIa32X64.dsc | 3 ++-
> OvmfPkg/OvmfPkgX64.dsc | 3 ++-
> 4 files changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/OvmfPkg/CloudHv/CloudHvX64.dsc
> b/OvmfPkg/CloudHv/CloudHvX64.dsc index 8ac9227c5f50..c5f8314a37b6
> 100644
> --- a/OvmfPkg/CloudHv/CloudHvX64.dsc
> +++ b/OvmfPkg/CloudHv/CloudHvX64.dsc
> @@ -429,6 +429,7 @@
> !endif
>
> BaseCryptLib|CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf
>
> PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf
>
> +
> SmmCpuRendezvousLib|UefiCpuPkg/Library/SmmCpuRendezvousLib/SmmCpuR
> endezvousLib.inf
>
>
>
> [LibraryClasses.common.SMM_CORE]
>
> PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf
>
> diff --git a/OvmfPkg/OvmfPkgIa32.dsc b/OvmfPkg/OvmfPkgIa32.dsc index
> 29eea82571c5..e6122118e07e 100644
> --- a/OvmfPkg/OvmfPkgIa32.dsc
> +++ b/OvmfPkg/OvmfPkgIa32.dsc
> @@ -1,7 +1,7 @@
> ## @file
>
> # EFI/Framework Open Virtual Machine Firmware (OVMF) platform
>
> #
>
> -# Copyright (c) 2006 - 2021, Intel Corporation. All rights
> reserved.<BR>
>
> +# Copyright (c) 2006 - 2022, Intel Corporation. All rights
> +reserved.<BR>
>
> # (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
>
> # Copyright (c) Microsoft Corporation.
>
> #
>
> @@ -427,6 +427,7 @@
> !endif
>
> BaseCryptLib|CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf
>
> PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf
>
> +
> SmmCpuRendezvousLib|UefiCpuPkg/Library/SmmCpuRendezvousLib/SmmCpuR
> endezvousLib.inf
>
>
>
> [LibraryClasses.common.SMM_CORE]
>
> PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf
>
> diff --git a/OvmfPkg/OvmfPkgIa32X64.dsc b/OvmfPkg/OvmfPkgIa32X64.dsc
> index 56d3c49ab21a..05c16aa3fd3e 100644
> --- a/OvmfPkg/OvmfPkgIa32X64.dsc
> +++ b/OvmfPkg/OvmfPkgIa32X64.dsc
> @@ -1,7 +1,7 @@
> ## @file
>
> # EFI/Framework Open Virtual Machine Firmware (OVMF) platform
>
> #
>
> -# Copyright (c) 2006 - 2021, Intel Corporation. All rights
> reserved.<BR>
>
> +# Copyright (c) 2006 - 2022, Intel Corporation. All rights
> +reserved.<BR>
>
> # (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
>
> # Copyright (c) Microsoft Corporation.
>
> #
>
> @@ -431,6 +431,7 @@
> !endif
>
> BaseCryptLib|CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf
>
> PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf
>
> +
> SmmCpuRendezvousLib|UefiCpuPkg/Library/SmmCpuRendezvousLib/SmmCpuR
> endezvousLib.inf
>
>
>
> [LibraryClasses.common.SMM_CORE]
>
> PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf
>
> diff --git a/OvmfPkg/OvmfPkgX64.dsc b/OvmfPkg/OvmfPkgX64.dsc index
> f0924c0f9d0a..ea42bee22cb5 100644
> --- a/OvmfPkg/OvmfPkgX64.dsc
> +++ b/OvmfPkg/OvmfPkgX64.dsc
> @@ -1,7 +1,7 @@
> ## @file
>
> # EFI/Framework Open Virtual Machine Firmware (OVMF) platform
>
> #
>
> -# Copyright (c) 2006 - 2021, Intel Corporation. All rights
> reserved.<BR>
>
> +# Copyright (c) 2006 - 2022, Intel Corporation. All rights
> +reserved.<BR>
>
> # (C) Copyright 2016 Hewlett Packard Enterprise Development LP<BR>
>
> # Copyright (c) Microsoft Corporation.
>
> #
>
> @@ -432,6 +432,7 @@
> !endif
>
> BaseCryptLib|CryptoPkg/Library/BaseCryptLib/SmmCryptLib.inf
>
> PciLib|OvmfPkg/Library/DxePciLibI440FxQ35/DxePciLibI440FxQ35.inf
>
> +
> SmmCpuRendezvousLib|UefiCpuPkg/Library/SmmCpuRendezvousLib/SmmCpuR
> endezvousLib.inf
>
>
>
> [LibraryClasses.common.SMM_CORE]
>
> PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf
>
> --
> 2.26.2.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88351): https://edk2.groups.io/g/devel/message/88351
Mute This Topic: https://groups.io/mt/90103054/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list