romcc fixes for AMD64
Eric W. Biederman
ebiederman at lnxi.com
Tue Sep 16 13:30:01 CEST 2003
Stefan Reinauer <stepan at suse.de> writes:
> Hi,
>
> with the attached fix romcc is able to compile all of auto.c
> for the freebios2 amd64 targets. To keep all changes clean
> and visible, they're encapsulated in ifdefs.
>
> Note: This fix alone does not make LinuxBIOS compile on AMD64
> completely yet. GCC and ld need the -m32 in some places at least
> to compile everything.
The -m32 can easily be added in the compiler over rides so that is simple
enough.
> Is there any need in going 64bit for the LinuxBIOS C payload
> on Opteron? Since the kernel does long mode switching itself
> this should not be the case, but maybe there are different
> opinions.
There is not any need yet. But there are certainly advantages.
So far keeping it 32bit has made things simpler. Ideally I'd
like to be able to do either.
> Next fail I saw after romcc passed auto.c was the following:
> /home/stepan/freebios2/src/superio/NSC/pc87360/superio.c:61: warning:
> `pnp_read_enable' defined but not used
>
> /tmp/ccLriZXb.s: Assembler messages:
> /tmp/ccLriZXb.s:243: Error: Incorrect register `%rdx' used with `l' suffix
> /tmp/ccLriZXb.s:245: Error: Incorrect register `%rdx' used with `l' suffix
> make: *** [superio.o] Error 1
Hmm. This looks like an inline assembler problem.
Eric
More information about the coreboot
mailing list