[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