On Mittwoch, 26. Juli 2017 22:25:29 CEST Rob Clark wrote: > On Wed, Jul 26, 2017 at 4:10 PM, Alexander Graf <ag...@suse.de> wrote: > > On 26.07.17 20:34, Rob Clark wrote: > >> When booting shim -> fallback -> shim -> grub -> linux the memory map is > >> a bit larger than the size linux passes in on the first call. But in > >> the EFI_BUFFER_TOO_SMALL case we were not passing back the updated size > >> to linux so it would loop forever. > >> > >> Signed-off-by: Rob Clark <robdcl...@gmail.com> > > > > The spec is actually very explicit about this case. It says in the > > EFI_BUFFER_TOO_SMALL case, we *have* to return the map size. > > yes, that is what I fixed. We *weren't* returning the required buffer > size before :-)
Sigh, yes, this was correct in the first 3 versions of the patch series, but unfortunately broken in v4 which was actually committed ... See: https://lists.denx.de/pipermail/u-boot/2016-October/268766.html Actually, the map_size variable is no longer needed, if you assign to *memory_map_size directly. Anyway, this patch is: Reviewed-by: Stefan Brüns <stefan.bru...@rwth-aachen.de> Kind regards, Stefan _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
