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

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

Merged https://github.com/tianocore/edk2/pull/3741

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao,
> Jiewen
> Sent: Friday, December 9, 2022 2:50 PM
> To: Ryan Afranji <afranji@google.com>; devel@edk2.groups.io
> Cc: Nakajima, Jun <jun.nakajima@intel.com>; Xu, Min M
> <min.m.xu@intel.com>; Aktas, Erdem <erdemaktas@google.com>;
> vannapurve@google.com
> Subject: Re: [edk2-devel] [V1 PATCH 1/1] OvmfPkg: Add INVD case in #VE
> handler
> 
> 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
> 
> 
> 
> 
>