[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.dehttp://www.coresystems.de/





More information about the coreboot mailing list