[U-Boot] [PATCH v2 3/5] disk: efi: correct the overlap check on GPT header and PTE
Simon Glass
sjg at chromium.org
Sun Oct 22 14:34:02 UTC 2017
On 16 October 2017 at 18:17, Patrick Delaunay <patrick.delaunay at st.com> wrote:
> the partition starting at 0x4400 is refused with overlap error:
> $> gpt write mmc 0 "name=test,start=0x4400,size=0"
> Writing GPT: Partition overlap
> error!
>
> even if the 0x4400 is the first available offset for LBA35 with default
> value:
> - MBR=LBA1
> - GPT header=LBA2
> - PTE= 32 LBAs (128 entry), 3 to 34
>
> And the command to have one partition for all the disk failed also :
> $> gpt write mmc 0 "name=test,size=0"
>
> After the patch :
>
> $> gpt write mmc 0 "name=test,size=0"
> Writing GPT: success!
> $> part list mmc 0
>
> Partition Map for MMC device 0 -- Partition Type: EFI
>
> Part Start LBA End LBA Name
> Attributes
> Type GUID
> Partition GUID
> 1 0x00000022 0x01ce9fde "test"
> attrs: 0x0000000000000000
> type: ebd0a0a2-b9e5-4433-87c0-68b6b72699c7
> type: data
> guid: b4b84b8a-04e3-4000-0036-aff5c9c495b1
>
> And 0x22 = 34 LBA => offset = 0x4400 is accepted as expected
>
> Reviewed-by: Ćukasz Majewski <lukma at denx.de>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> ---
> gpt test are now OK
> ./test/py/test.py -k gpt --build
>
> test/py/tests/test_gpt.py .......
> => 7 passed, 228 deselected in 1.11 seconds
>
> Changes in v2: None
>
> disk/part_efi.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Simon Glass <sjg at chromium.org>
More information about the U-Boot
mailing list