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] -=-=-=-=-=-=-=-=-=-=-=-