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. V3: patch #1: Remove the useless variable and add the reason and impact of the fix. patch #2: Remove useless condition and make the comments more clear. Cc: Hao A Wu <hao.a...@intel.com> Cc: Ray Ni <ray...@intel.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 | 50 +++++++++++++++---- .../Universal/Disk/PartitionDxe/Partition.c | 9 ++++ 2 files changed, 48 insertions(+), 11 deletions(-) -- 2.21.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#62459): https://edk2.groups.io/g/devel/message/62459 Mute This Topic: https://groups.io/mt/75490638/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
