[edk2-devel] [PATCH V1 0/2] Refactor TDX MmioExit

Min Xu posted 2 patches 1 year, 4 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
OvmfPkg/Library/CcExitLib/CcExitLib.inf     |   1 +
OvmfPkg/Library/CcExitLib/CcExitVcHandler.c | 697 +++-----------------
OvmfPkg/Library/CcExitLib/CcExitVeHandler.c | 498 +++++++++-----
OvmfPkg/Library/CcExitLib/CcInstruction.c   | 454 +++++++++++++
OvmfPkg/Library/CcExitLib/CcInstruction.h   | 197 ++++++
OvmfPkg/Library/CcExitLib/SecCcExitLib.inf  |   1 +
6 files changed, 1082 insertions(+), 766 deletions(-)
create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.c
create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.h
[edk2-devel] [PATCH V1 0/2] Refactor TDX MmioExit
Posted by Min Xu 1 year, 4 months ago
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4169

The previous TDX MmioExit doesn't handle the Mmio instructions correctly
in some scenarios. This patch-set refactors the implementation to fix the
issues.

Before the refactoring, common X86 instruction codes in CcExitVcHandler.c
are moved to separate files (CcInstruction.h / CcInstruction.c) so that
these codes can be re-used in TDX.

Cc: Erdem Aktas <erdemaktas@google.com>
Cc: James Bottomley <jejb@linux.ibm.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Tom Lendacky <thomas.lendacky@amd.com>
Cc: Ryan Afranji <afranji@google.com>
Reported-by: Ryan Afranji <afranji@google.com>
Signed-off-by: Min Xu <min.m.xu@intel.com>

Min M Xu (2):
  OvmfPkg/CcExitLib: Move common X86 instruction code to separate file
  OvmfPkg/CcExitLib: Refactor TDX MmioExit

 OvmfPkg/Library/CcExitLib/CcExitLib.inf     |   1 +
 OvmfPkg/Library/CcExitLib/CcExitVcHandler.c | 697 +++-----------------
 OvmfPkg/Library/CcExitLib/CcExitVeHandler.c | 498 +++++++++-----
 OvmfPkg/Library/CcExitLib/CcInstruction.c   | 454 +++++++++++++
 OvmfPkg/Library/CcExitLib/CcInstruction.h   | 197 ++++++
 OvmfPkg/Library/CcExitLib/SecCcExitLib.inf  |   1 +
 6 files changed, 1082 insertions(+), 766 deletions(-)
 create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.c
 create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.h

-- 
2.29.2.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97798): https://edk2.groups.io/g/devel/message/97798
Mute This Topic: https://groups.io/mt/95934163/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH V1 0/2] Refactor TDX MmioExit
Posted by Min Xu 1 year, 4 months ago
Code: https://github.com/mxu9/edk2/tree/TdxMmioExit.v1

> -----Original Message-----
> From: Xu, Min M <min.m.xu@intel.com>
> Sent: Thursday, December 29, 2022 4:56 PM
> To: devel@edk2.groups.io
> Cc: Xu, Min M <min.m.xu@intel.com>; Aktas, Erdem
> <erdemaktas@google.com>; James Bottomley <jejb@linux.ibm.com>; Yao,
> Jiewen <jiewen.yao@intel.com>; Gerd Hoffmann <kraxel@redhat.com>;
> Tom Lendacky <thomas.lendacky@amd.com>; Ryan Afranji
> <afranji@google.com>
> Subject: [PATCH V1 0/2] Refactor TDX MmioExit
> 
> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4169
> 
> The previous TDX MmioExit doesn't handle the Mmio instructions correctly in
> some scenarios. This patch-set refactors the implementation to fix the issues.
> 
> Before the refactoring, common X86 instruction codes in CcExitVcHandler.c
> are moved to separate files (CcInstruction.h / CcInstruction.c) so that these
> codes can be re-used in TDX.
> 
> Cc: Erdem Aktas <erdemaktas@google.com>
> Cc: James Bottomley <jejb@linux.ibm.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Tom Lendacky <thomas.lendacky@amd.com>
> Cc: Ryan Afranji <afranji@google.com>
> Reported-by: Ryan Afranji <afranji@google.com>
> Signed-off-by: Min Xu <min.m.xu@intel.com>
> 
> Min M Xu (2):
>   OvmfPkg/CcExitLib: Move common X86 instruction code to separate file
>   OvmfPkg/CcExitLib: Refactor TDX MmioExit
> 
>  OvmfPkg/Library/CcExitLib/CcExitLib.inf     |   1 +
>  OvmfPkg/Library/CcExitLib/CcExitVcHandler.c | 697 +++-----------------
> OvmfPkg/Library/CcExitLib/CcExitVeHandler.c | 498 +++++++++-----
>  OvmfPkg/Library/CcExitLib/CcInstruction.c   | 454 +++++++++++++
>  OvmfPkg/Library/CcExitLib/CcInstruction.h   | 197 ++++++
>  OvmfPkg/Library/CcExitLib/SecCcExitLib.inf  |   1 +
>  6 files changed, 1082 insertions(+), 766 deletions(-)  create mode 100644
> OvmfPkg/Library/CcExitLib/CcInstruction.c
>  create mode 100644 OvmfPkg/Library/CcExitLib/CcInstruction.h
> 
> --
> 2.29.2.windows.2



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