I have tested on the OVMF with linux iso image like Ubuntu, Fedora and RedHat.
It can enumerate under the shell and load the grub efi application. Thanks, Zhichao > -----Original Message----- > From: Wu, Hao A <hao.a...@intel.com> > Sent: Thursday, July 9, 2020 1:04 PM > To: Gao, Zhichao <zhichao....@intel.com>; devel@edk2.groups.io > Cc: Ni, Ray <ray...@intel.com>; Laszlo Ersek <ler...@redhat.com> > Subject: RE: [PATCH V2 0/3] MdeModulePkg/PartitionDxe: Fix the partition check > issue > > > -----Original Message----- > > From: Gao, Zhichao <zhichao....@intel.com> > > Sent: Wednesday, July 8, 2020 10:27 AM > > To: devel@edk2.groups.io > > Cc: Wu, Hao A <hao.a...@intel.com>; Ni, Ray <ray...@intel.com>; Laszlo > > Ersek <ler...@redhat.com> > > Subject: [PATCH V2 0/3] MdeModulePkg/PartitionDxe: Fix the partition > > check issue > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2823 > > > > V1: > > Separate the UDF from the partition rountine array and do the check > > for every media. > > > > V2: > > Drop V1 because it is a bug: there should not be two partition types > > in one media. > > 1. Correct the LastBlock value in MBR handler. It should be the number > > of sectors (512 bytes). > > 2. Skip the MBR check if the MBR is added for the Windows > > comaptiblity. We treat such media as ElTorito and do the check. > > 3. Fix the partition check bug: One partition type returns already > > started should be treated as success to avoid multi partition type be > > installed into same media. > > > Hello Zhichao, > > The changes look good to me. > May I know what tests have been done for the series? Thanks. > > Best Regards, > Hao Wu > > > > > > Cc: Hao A Wu <hao.a...@intel.com> > > Cc: Ray Ni <ray...@intel.com> > > Cc: Laszlo Ersek <ler...@redhat.com> > > Signed-off-by: Zhichao Gao <zhichao....@intel.com> > > > > Zhichao Gao (3): > > MdeModulePkg/PartitionDxe: Correct the MBR last block value > > MdeModulePkg/PartitionDxe: Skip the MBR that add for CD-ROM > > MdeModulePkg/PartitionDxe: Add already start check for child hanldes > > > > .../Universal/Disk/PartitionDxe/Mbr.c | 42 +++++++++++++++---- > > .../Universal/Disk/PartitionDxe/Partition.c | 9 ++++ > > 2 files changed, 44 insertions(+), 7 deletions(-) > > > > -- > > 2.21.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#62323): https://edk2.groups.io/g/devel/message/62323 Mute This Topic: https://groups.io/mt/75369396/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
