[LinuxBIOS] patch: extending LAR, and removing elf from linuxbios (it is not needed)
Stefan Reinauer
stepan at coresystems.de
Tue Aug 28 01:55:30 CEST 2007
Peter Stuge wrote:
>> + for(i = 0, entry = (void *)0; ;i++) {
>> + char filename[64];
>> + void *newentry;
>> + sprintf(filename, "normal/payload%d", i);
>> + archive.len = *(u32 *)0xfffffff4;
>> + 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 ;)
The size is explicit btw. The start address is not.
The above code would break on sc520, but that seems dead ;)
--
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de • http://www.coresystems.de/
More information about the coreboot
mailing list