[coreboot] VbTryLoadKernel restriction on 512 byte sector size

[Aaron Durbin]

On Tue, Apr 3, 2018 at 10:26 AM,  <mturney at codeaurora.org> wrote:
> Folks,
> Can anyone on this list expound on why VbTryLoadKernel() performs a
> sanity-check on bytes_per_lba != 512?

>------->-------/*
>------->------- * Sanity-check what we can. FWIW, VbTryLoadKernel() is always
>------->------- * called with only a single bit set in get_info_flags.
>------->------- *
>------->------- * Ensure 512-byte sectors and non-trivially sized disk (for
>------->------- * cgptlib) and that we got a partition with only the flags we
>------->------- * asked for.
>------->------- */

Seems to do with cgptlib having a 512 byte assumption. I'm not sure
what would break if that check was removed. We'd have to look at
cgptlib and other uses. Please file a Google issue tracker bug so we
can properly drive this.

>
> We are working on supporting a board that will require bytes_per_lba == 4K.
> Cheers,
> T.mike
>
> --
> coreboot mailing list: coreboot at coreboot.org
> https://mail.coreboot.org/mailman/listinfo/coreboot