VbTryLoadKernel restriction on 512 byte sector size

Folks,
Can anyone on this list expound on why VbTryLoadKernel() performs a 
sanity-check on bytes_per_lba != 512?

We are working on supporting a board that will require bytes_per_lba == 
4K.
Cheers,
T.mike

[Aaron Durbin]

On Tue, Apr 3, 2018 at 10:26 AM,   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