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

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao,
> Jiewen
> Sent: Wednesday, January 18, 2023 8:16 AM
> To: Xu, Min M <min.m...@intel.com>; devel@edk2.groups.io
> Cc: Aktas, Erdem <erdemak...@google.com>; James Bottomley
> <j...@linux.ibm.com>; Gerd Hoffmann <kra...@redhat.com>; Tom
> Lendacky <thomas.lenda...@amd.com>; Michael Roth
> <michael.r...@amd.com>
> Subject: Re: [edk2-devel] [PATCH V2 1/1] OvmfPkg/CcExitLib: Initialize Status
> in IoExit
> 
> Reviewed-by: Jiewen Yao <jiewen....@intel.com>
> 
> > -----Original Message-----
> > From: Xu, Min M <min.m...@intel.com>
> > Sent: Wednesday, January 18, 2023 7:53 AM
> > To: devel@edk2.groups.io
> > Cc: Xu, Min M <min.m...@intel.com>; Aktas, Erdem
> > <erdemak...@google.com>; James Bottomley <j...@linux.ibm.com>; Yao,
> > Jiewen <jiewen....@intel.com>; Gerd Hoffmann <kra...@redhat.com>;
> > Tom Lendacky <thomas.lenda...@amd.com>; Michael Roth
> > <michael.r...@amd.com>
> > Subject: [PATCH V2 1/1] OvmfPkg/CcExitLib: Initialize Status in IoExit
> >
> > From: Min M Xu <min.m...@intel.com>
> >
> > Status should be initialized otherwise it may return unexpected value.
> >
> > Cc: Erdem Aktas <erdemak...@google.com>
> > Cc: James Bottomley <j...@linux.ibm.com>
> > Cc: Jiewen Yao <jiewen....@intel.com>
> > Cc: Gerd Hoffmann <kra...@redhat.com>
> > Cc: Tom Lendacky <thomas.lenda...@amd.com>
> > Cc: Michael Roth <michael.r...@amd.com>
> > Reviewed-by: Jiewen Yao <jiewen....@intel.com>
> > Signed-off-by: Min Xu <min.m...@intel.com>
> > ---
> >  OvmfPkg/Library/CcExitLib/CcExitVeHandler.c | 9 +++++----
> >  1 file changed, 5 insertions(+), 4 deletions(-)
> >
> > diff --git a/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> > b/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> > index 30d547d5fe55..872f772a5ac8 100644
> > --- a/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> > +++ b/OvmfPkg/Library/CcExitLib/CcExitVeHandler.c
> > @@ -105,10 +105,11 @@ IoExit (
> >    UINT64   RepCnt;
> >    UINT64   Status;
> >
> > -  Val   = 0;
> > -  Write = Veinfo->ExitQualification.Io.Direction ? FALSE : TRUE;
> > -  Size  = Veinfo->ExitQualification.Io.Size + 1;
> > -  Port  = Veinfo->ExitQualification.Io.Port;
> > +  Val    = 0;
> > +  Status = 0;
> > +  Write  = Veinfo->ExitQualification.Io.Direction ? FALSE : TRUE;
> > +  Size   = Veinfo->ExitQualification.Io.Size + 1;
> > +  Port   = Veinfo->ExitQualification.Io.Port;
> >
> >    if (Veinfo->ExitQualification.Io.String) {
> >      //
> > --
> > 2.29.2.windows.2
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#98766): https://edk2.groups.io/g/devel/message/98766
Mute This Topic: https://groups.io/mt/96343921/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to