[coreboot] [PATCH] fix Multiboot header in libpayload
Uwe Hermann
uwe at hermann-uwe.de
Tue Nov 11 22:04:57 CET 2008
On Tue, Nov 11, 2008 at 09:42:56PM +0100, Robert Millan wrote:
>
> Hi,
>
> Multiboot header in libpayload is at a too high location (>0x2000), which
> breaks the standard (GRUB checks for the header and refuses to load if a
> header could not be found below 0x2000).
>
> Attached patch fixes the problem by reordering the objects in link command.
>
> --
> Robert Millan
>
> The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
> how) you may access your data; but nobody's threatening your freedom: we
> still allow you to remove your data and not access it at all."
>
> Signed-off-by: Robert Millan <rmh at aybabtu.com>
>
> Index: libpayload/bin/lpgcc
> ===================================================================
> --- libpayload/bin/lpgcc (revision 3744)
> +++ libpayload/bin/lpgcc (working copy)
> @@ -101,8 +101,8 @@
> else
> _LIBGCC=`$DEFAULT_CC -m32 -print-libgcc-file-name`
> if [ $DEBUGME -eq 1 ]; then
> - echo "$DEFAULT_CC $_CFLAGS $_LDFLAGS $CMDLINE $_LIBDIR/i386/head.o $_LIBDIR/libpayload.a $_LIBGCC"
> + echo "$DEFAULT_CC $_CFLAGS $_LDFLAGS $_LIBDIR/i386/head.o $CMDLINE $_LIBDIR/libpayload.a $_LIBGCC"
Please also add a comment here, this is highly non-obvious from just
looking at the file's contents otherwise.
> fi
>
> - $DEFAULT_CC $_CFLAGS $_LDFLAGS $CMDLINE $_LIBDIR/i386/head.o $_LIBDIR/libpayload.a $_LIBGCC
> + $DEFAULT_CC $_CFLAGS $_LDFLAGS $_LIBDIR/i386/head.o $CMDLINE $_LIBDIR/libpayload.a $_LIBGCC
> fi
> --
> coreboot mailing list: coreboot at coreboot.org
> http://www.coreboot.org/mailman/listinfo/coreboot
--
http://www.hermann-uwe.de | http://www.holsham-traders.de
http://www.crazy-hacks.org | http://www.unmaintained-free-software.org
More information about the coreboot
mailing list