On 18.09.2008 23:59, Carl-Daniel Hailfinger wrote:
> On 18.09.2008 22:09, Robert Millan wrote:
>   
>> --- arch/x86/stage1.c        (revision 867)
>> +++ arch/x86/stage1.c        (working copy)
>> @@ -139,6 +140,14 @@
>>  }
>>  #endif /* CONFIG_PAYLOAD_ELF_LOADER */
>>  
>> +
>> +int run_address_multiboot(void *f)
>> +{
>> +    int ret;
>> +    __asm__ __volatile__ ("call *%3" : "=a" (ret) : "a" (MB_MAGIC2), "b" 
>> (0xf0000), "c" (f));
>> +    return ret;
>> +}
>> +
>>   
>>     
>
> The function above belongs in multiboot.c
>   

I forgot to mention that the clobber list is incomplete, possibly
leading to corruption if a payload returns.

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/


--
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to