* [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
@ 2022-03-29 6:37 Li, Zhihao
2022-03-29 8:39 ` [edk2-devel] " Gerd Hoffmann
2022-04-01 11:35 ` Yao, Jiewen
0 siblings, 2 replies; 5+ messages in thread
From: Li, Zhihao @ 2022-03-29 6:37 UTC (permalink / raw)
To: devel
Cc: Ard Biesheuvel, Jiewen Yao, Jordan Justen, Gerd Hoffmann,
Sebastien Boeuf
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@kernel.org>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Sebastien Boeuf <sebastien.boeuf@intel.com>
Signed-off-by: Zhihao Li <zhihao.li@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/SmmCpuRendezvousLib.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/SmmCpuRendezvousLib.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/SmmCpuRendezvousLib.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/SmmCpuRendezvousLib.inf
[LibraryClasses.common.SMM_CORE]
PcdLib|MdePkg/Library/DxePcdLib/DxePcdLib.inf
--
2.26.2.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [edk2-devel] [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
2022-03-29 6:37 [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally Li, Zhihao
@ 2022-03-29 8:39 ` Gerd Hoffmann
2022-04-01 11:35 ` Yao, Jiewen
1 sibling, 0 replies; 5+ messages in thread
From: Gerd Hoffmann @ 2022-03-29 8:39 UTC (permalink / raw)
To: devel, zhihao.li
Cc: Ard Biesheuvel, Jiewen Yao, Jordan Justen, Sebastien Boeuf
On Tue, Mar 29, 2022 at 02:37:38PM +0800, Li, Zhihao wrote:
> 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@kernel.org>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Sebastien Boeuf <sebastien.boeuf@intel.com>
>
> Signed-off-by: Zhihao Li <zhihao.li@intel.com>
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
2022-03-29 6:37 [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally Li, Zhihao
2022-03-29 8:39 ` [edk2-devel] " Gerd Hoffmann
@ 2022-04-01 11:35 ` Yao, Jiewen
2022-04-01 17:10 ` Li, Zhihao
1 sibling, 1 reply; 5+ messages in thread
From: Yao, Jiewen @ 2022-04-01 11:35 UTC (permalink / raw)
To: Li, Zhihao, devel@edk2.groups.io
Cc: Ard Biesheuvel, Justen, Jordan L, Gerd Hoffmann, Boeuf, Sebastien
Reviewed-by: Jiewen Yao <Jiewen.yao@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@intel.com>
> Sent: Tuesday, March 29, 2022 2:38 PM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Yao, Jiewen
> <jiewen.yao@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Gerd
> Hoffmann <kraxel@redhat.com>; Boeuf, Sebastien <sebastien.boeuf@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@kernel.org>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Sebastien Boeuf <sebastien.boeuf@intel.com>
>
> Signed-off-by: Zhihao Li <zhihao.li@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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
2022-04-01 11:35 ` Yao, Jiewen
@ 2022-04-01 17:10 ` Li, Zhihao
2022-04-02 12:38 ` Yao, Jiewen
0 siblings, 1 reply; 5+ messages in thread
From: Li, Zhihao @ 2022-04-01 17:10 UTC (permalink / raw)
To: Yao, Jiewen, devel@edk2.groups.io
Cc: Ard Biesheuvel, Justen, Jordan L, Gerd Hoffmann, Boeuf, Sebastien
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@intel.com>
Sent: Friday, April 1, 2022 7:35 PM
To: Li, Zhihao <zhihao.li@intel.com>; devel@edk2.groups.io
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Justen, Jordan L <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Boeuf, Sebastien <sebastien.boeuf@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@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@intel.com>
> Sent: Tuesday, March 29, 2022 2:38 PM
> To: devel@edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Yao, Jiewen
> <jiewen.yao@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>;
> Gerd Hoffmann <kraxel@redhat.com>; Boeuf, Sebastien
> <sebastien.boeuf@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@kernel.org>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Sebastien Boeuf <sebastien.boeuf@intel.com>
>
> Signed-off-by: Zhihao Li <zhihao.li@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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally.
2022-04-01 17:10 ` Li, Zhihao
@ 2022-04-02 12:38 ` Yao, Jiewen
0 siblings, 0 replies; 5+ messages in thread
From: Yao, Jiewen @ 2022-04-02 12:38 UTC (permalink / raw)
To: Li, Zhihao, devel@edk2.groups.io
Cc: Ard Biesheuvel, Justen, Jordan L, Gerd Hoffmann, Boeuf, Sebastien
Merged https://github.com/tianocore/edk2/pull/2729
> -----Original Message-----
> From: Li, Zhihao <zhihao.li@intel.com>
> Sent: Saturday, April 2, 2022 1:10 AM
> To: Yao, Jiewen <jiewen.yao@intel.com>; devel@edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Justen, Jordan L
> <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Boeuf,
> Sebastien <sebastien.boeuf@intel.com>
> Subject: RE: [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to
> make it work normally.
>
> 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@intel.com>
> Sent: Friday, April 1, 2022 7:35 PM
> To: Li, Zhihao <zhihao.li@intel.com>; devel@edk2.groups.io
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Justen, Jordan L
> <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>; Boeuf,
> Sebastien <sebastien.boeuf@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@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@intel.com>
> > Sent: Tuesday, March 29, 2022 2:38 PM
> > To: devel@edk2.groups.io
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>; Yao, Jiewen
> > <jiewen.yao@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>;
> > Gerd Hoffmann <kraxel@redhat.com>; Boeuf, Sebastien
> > <sebastien.boeuf@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@kernel.org>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Cc: Gerd Hoffmann <kraxel@redhat.com>
> > Cc: Sebastien Boeuf <sebastien.boeuf@intel.com>
> >
> > Signed-off-by: Zhihao Li <zhihao.li@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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-04-02 12:38 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-03-29 6:37 [PATCH v1 1/1] OvmfPkg: Add dependency of VariableSmm driver to make it work normally Li, Zhihao
2022-03-29 8:39 ` [edk2-devel] " Gerd Hoffmann
2022-04-01 11:35 ` Yao, Jiewen
2022-04-01 17:10 ` Li, Zhihao
2022-04-02 12:38 ` Yao, Jiewen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox