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

Peter Stuge peter at stuge.se
Wed Aug 29 23:39:07 CEST 2007


On Tue, Aug 28, 2007 at 01:55:30AM +0200, Stefan Reinauer wrote:
> Peter Stuge wrote:
> >> +		archive.start =(void *)(0UL-archive.len);
> >>     
> >
> > Is 0UL good also on 64bit? (Even if it's truncated and "saved" when
> > assigned to archive.start I'd prefer having the size explicitly.)
> >   
> There will not be a 64bit version of the above code within the next
> decade. But who says never ;)

My point was that if this is built on amd64 0UL is 64 bit, right?
But archive.start is u32 - and we should be more explicit about what
we want to accomplish.


//Peter




More information about the coreboot mailing list