Any plans to fix payload builds on 64-bit machines?

[Sean McNeil]

All the recent changes to how the compiler is defined have now caused 
payloads like FILO and SeaBIOS to not build on X86_64 machines. Is 
anyone working to fix this?

Cheers,
Sean

[David Hendricks]

On Tue, May 20, 2014 at 6:31 PM, Sean McNeil  wrote:
> All the recent changes to how the compiler is defined have now caused
> payloads like FILO and SeaBIOS to not build on X86_64 machines. Is anyone
> working to fix this?

I just tried building images with each of those as payloads and they seemed
to compile fine for me. Try "rm -f .xcompile" before compiling and see if
that helps.

If that doesn't work, can you post more details such as the git version
you're on and output of "make" ?

[Patrick Georgi]

Am 21.05.2014 03:31, schrieb Sean McNeil:
> All the recent changes to how the compiler is defined have now caused
> payloads like FILO and SeaBIOS to not build on X86_64 machines. Is
> anyone working to fix this?

--- a/src/arch/x86/Makefile.inc
+++ b/src/arch/x86/Makefile.inc
@@ -347,6 +347,7 @@ seabios:
                        CC="$(CC_x86_32)" LD="$(LD_x86_32)"
OBJDUMP="$(OBJDUMP_x86_32)" \
                        OBJCOPY="$(OBJCOPY_x86_32)"
STRIP="$(STRIP_x86_32)" \
                        AS="$(AS_x86_32)" \
+                       CFLAGS="$(CLAGS_x86_32)" \
                        CONFIG_SEABIOS_MASTER=$(CONFIG_SEABIOS_MASTER) \
                        CONFIG_SEABIOS_STABLE=$(CONFIG_SEABIOS_STABLE) \

CONFIG_SEABIOS_THREAD_OPTIONROMS=$(CONFIG_SEABIOS_THREAD_OPTIONROMS) \

might do, or

CC="$(CC_x86_32) $(CFLAGS_x86_x32)" two lines earlier.

I can't test since my host compiler _really_ is broken for
firmware-style build tasks.

Regards,
Patrick

[Carl-Daniel Hailfinger]

Am 21.05.2014 22:13 schrieb Patrick Georgi:
> Am 21.05.2014 03:31, schrieb Sean McNeil:
> > All the recent changes to how the compiler is defined have now caused
> > payloads like FILO and SeaBIOS to not build on X86_64 machines. Is
> > anyone working to fix this?
>
> --- a/src/arch/x86/Makefile.inc
> +++ b/src/arch/x86/Makefile.inc
> @@ -347,6 +347,7 @@ seabios:
>                         CC="$(CC_x86_32)" LD="$(LD_x86_32)"
> OBJDUMP="$(OBJDUMP_x86_32)" \
>                         OBJCOPY="$(OBJCOPY_x86_32)"
> STRIP="$(STRIP_x86_32)" \
>                         AS="$(AS_x86_32)" \
>                   CFLAGS="$(CLAGS_x86_32)" \

Typo, the "f" in CFLAGS_x86_32 is missing.
>                     CONFIG_SEABIOS_MASTER=$(CONFIG_SEABIOS_MASTER) \
>                     CONFIG_SEABIOS_STABLE=$(CONFIG_SEABIOS_STABLE) \
>
> CONFIG_SEABIOS_THREAD_OPTIONROMS=$(CONFIG_SEABIOS_THREAD_OPTIONROMS) \
>
> might do, or
>
> CC="$(CC_x86_32) $(CFLAGS_x86_x32)" two lines earlier.
>
> I can't test since my host compiler _really_ is broken for
> firmware-style build tasks.

Regards,
Carl-Daniel