> +    //
> +    // 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/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to