Re: [V1 PATCH 1/1] OvmfPkg: Add INVD case in #VE handler

["Yao, Jiewen" <jiewen.yao@intel.com>]

Reviewed-by: Jiewen Yao <Jiewen.yao@intel.com>

> -----Original Message-----
> From: Ryan Afranji <afranji@google.com>
> Sent: Thursday, December 8, 2022 8:51 AM
> To: devel@edk2.groups.io
> Cc: Nakajima, Jun <jun.nakajima@intel.com>; Xu, Min M
> <min.m.xu@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>; Aktas, Erdem
> <erdemaktas@google.com>; vannapurve@google.com; Ryan Afranji
> <afranji@google.com>
> Subject: [V1 PATCH 1/1] OvmfPkg: Add INVD case in #VE handler
> 
> According to the Intel GHCI specification document section 2.4.1, the
> goal for instructions that do not have a corresponding TDCALL is for the
> handler to treat the instruction as a NOP.
> 
> INVD does not have a corresponding TDCALL. This patch makes the #VE
> handler treat INVD as a NOP.
> 
> Signed-off-by: Ryan Afranji <afranji@google.com>
> ---
>  OvmfPkg/Library/CcExitLib/CcExitVeHandler.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> b/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> index c35f65a649..3798c2bb13 100644
> --- a/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> +++ b/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> @@ -541,6 +541,7 @@ CcExitHandleVe (
>      case EXIT_REASON_MONITOR_INSTRUCTION:
>      case EXIT_REASON_WBINVD:
>      case EXIT_REASON_RDPMC:
> +    case EXIT_REASON_INVD:
>        /* Handle as nops. */
>        break;
> 
> --
> 2.39.0.rc1.256.g54fd8350bd-goog