Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-15 Thread Ni, Ray
:33 AM > > To: devel@edk2.groups.io; Gao, Zhichao > > Cc: Wu, Hao A ; Ni, Ray > > Subject: Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add > > already start check for child hanldes > > > > On Tue, Jul 14, 2020 at 09:22:59AM +0800, Gao,

Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-15 Thread Gao, Zhichao
think we should update the invalid MBR check logic. Thanks, Zhichao > -Original Message- > From: Gary Lin > Sent: Thursday, July 16, 2020 11:33 AM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Wu, Hao A ; Ni, Ray > Subject: Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg

Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-15 Thread Gary Lin
On Tue, Jul 14, 2020 at 09:22:59AM +0800, Gao, Zhichao via groups.io wrote: > 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. Hi Zhichao, I just found that this patch

Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-13 Thread Ni, Ray
Looks good to me! Thanks!! > -Original Message- > From: Gao, Zhichao > Sent: Tuesday, July 14, 2020 1:56 PM > To: Ni, Ray ; devel@edk2.groups.io > Cc: Wu, Hao A > Subject: RE: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add > already start check for ch

Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-13 Thread Gao, Zhichao
4 AM > To: devel@edk2.groups.io; Gao, Zhichao > Cc: Wu, Hao A > Subject: RE: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add > already start check for child hanldes > > Zhichao, > Can you add more information in the commit message on what bug the patch > can fix?

Re: [edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-13 Thread Ni, Ray
Cc: Wu, Hao A ; Ni, Ray > 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 >

[edk2-devel] [PATCH V3 3/3] MdeModulePkg/PartitionDxe: Add already start check for child hanldes

2020-07-13 Thread Gao, Zhichao
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 Cc: Ray Ni Signed-off-by: Zhichao Gao Reviewed-by: Hao A Wu --- MdeModulePkg/Universal/Disk/PartitionDxe/Pa