[LinuxBIOS] patch: extending LAR, and removing elf from linuxbios (it is not needed)

Stefan Reinauer stepan at coresystems.de
Wed Aug 29 09:59:35 CEST 2007


* ron minnich <rminnich at gmail.com> [070828 17:26]:
> > > void *entry;
> > > void *loadaddress;
> >
> > Not sure how these pointers sneaked in here. They do break portability
> > and cross compilability. Compiling LinuxBIOS on a 64bit host without
> > compiling lar 32bit is not possible with such things in the header.
> 
> I think you guys have missed the point. These are in the mem_file
> struct, and that struct is by definition a per-machine struct. There
> is NO cross-compatibility issue with mem_file -- see include/lar.h.
> mem_file is not even defined in util/lar/lar.h. The lar utility
> doesn't know what it is.
 
Yes, sorry, I missed it.

 
> > What was that linker auto rename trick that Marc mentioned recently?
> 
> The naming is not the issue. The issue is getting gcc to do abs calls,
> not relative calls, for some symbols.

Sure. But still the naming is an issue. The question is: How can we call
it printk on both sides of the rubicon. And having LD rename some of the
symbols for us seems like a simple solution.

Stefan

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.dehttp://www.coresystems.de/




More information about the coreboot mailing list