Hi Zhichao, On 07/08/20 04:27, Zhichao Gao wrote: > 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. > > 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(-) >
I've skimmed the patches briefly -- I've realized that I don't know enough to usefully review them. I'll defer to Hao and Ray. Thanks Laszlo -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#62242): https://edk2.groups.io/g/devel/message/62242 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] -=-=-=-=-=-=-=-=-=-=-=-
