[edk2-devel] [PATCH V2 2/3] MdeModulePkg/PartitionDxe: Skip the MBR that add for CD-ROM

[Ni, Ray]

> +    //
> +    // If the MBR with partition entry contains itself, i.e. start with LBA0,
> +    // and have the same size with the media, we treat it as a El Torito partition.
> +    //
> +    if ((StartingLBA == 0) &&
> +        (SizeInLBA != 0) &&
1. Can this check "SizeInLBA != 0" be removed?

> +      DEBUG ((DEBUG_INFO, "PartitionValidMbr: The MBR table has partition
> entry start at sector 0.\n"));
2. Can the debug message be refined as below?
" PartitionValidMbr: MBR table has partition entry covering the ENTIRE disk. Don't treat it as a valid MBR."

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#62393): https://edk2.groups.io/g/devel/message/62393
Mute This Topic: https://groups.io/mt/75369398/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-