Zhichao, Can you add more information in the commit message on what bug the patch can fix? With that, Reviewed-by: Ray Ni <ray...@intel.com>
> -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Gao, > Zhichao > Sent: Tuesday, July 14, 2020 9:23 AM > To: devel@edk2.groups.io > Cc: Wu, Hao A <hao.a...@intel.com>; Ni, Ray <ray...@intel.com> > Subject: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add > already start check for child hanldes > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2823 > > Treat the EFI_ALREADY_STARTED as EFI_SUCCESS to avoid the partition driver > continuely check next routine function. > > Cc: Hao A Wu <hao.a...@intel.com> > Cc: Ray Ni <ray...@intel.com> > Signed-off-by: Zhichao Gao <zhichao....@intel.com> > Reviewed-by: Hao A Wu <hao.a...@intel.com> > --- > MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c > b/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c > index d1c878ad2e..6a43c3cafb 100644 > --- a/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c > +++ b/MdeModulePkg/Universal/Disk/PartitionDxe/Partition.c > @@ -1276,6 +1276,15 @@ PartitionInstallChildHandle ( > } else { > FreePool (Private->DevicePath); > FreePool (Private); > + > + // > + // if the Status == EFI_ALREADY_STARTED, it means the child handles > + // are already installed. So return EFI_SUCCESS to avoid do the next > + // partition type check. > + // > + if (Status == EFI_ALREADY_STARTED) { > + Status = EFI_SUCCESS; > + } > } > > return Status; > -- > 2.21.0.windows.1 > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#62465): https://edk2.groups.io/g/devel/message/62465 Mute This Topic: https://groups.io/mt/75490642/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
