<div dir="ltr"><br><br><div class="gmail_quote">On Wed, Oct 15, 2008 at 5:44 PM, Carl-Daniel Hailfinger <span dir="ltr"><<a href="mailto:c-d.hailfinger.devel.2006@gmx.net">c-d.hailfinger.devel.2006@gmx.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="Ih2E3d">On 16.10.2008 01:13, Peter Stuge wrote:<br>
> ron minnich wrote:<br>
><br>
>> Myles, try setting -mregparm=0 and see if life is better.<br>
>><br>
><br>
> Good find.<br>
><br>
<br>
</div>__attribute__((stdcall)) will do this for you.</blockquote><div><br>It doesn't work for me.  I tried it in various places, but it doesn't change the way the parameters are handled.<br><br>void __attribute__ ((stdcall)) callbiosint(void)<br>
__attribute__ ((stdcall)) void callbiosint(void)<br>void  callbiosint(void) __attribute__ ((stdcall))<br><br>biosint: INT# 0x18<br>biosint: eax 0x2e ebx 0x10000 ecx 0xfe4 edx 0xcf11c<br>biosint: ebp 0xc0000000 esp 0xd0000 edi 0x1a esi 0x0<br>
biosint:  ip 0x1022   cs 0xf  flags 0x2067<br>BIOSINT: Unsupport int #0x18<br><br>Thanks,<br>Myles<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Changing compilation mode makes code to -mregparm=0 makes the code<br>
bigger and slower and it will also possibly consume more stack which we<br>
don't have in MP setups.<br>
<div class="Ih2E3d"><br>
<br>
>> I vote we get rid of this type of Cleverness. It's just not<br>
>> performance critical in a bios. We're not an OS and we should keep<br>
>> it simple. I don't think we'll live or die on 3 on-stack variables.<br>
>><br>
><br>
> If it isn't more relevant because of CAR then away it goes.<br>
><br>
<br>
</div>See above.<br>
<br>
Regards,<br>
Carl-Daniel<br>
<font color="#888888"><br>
--<br>
<a href="http://www.hailfinger.org/" target="_blank">http://www.hailfinger.org/</a><br>
</font><div><div></div><div class="Wj3C7c"><br>
<br>
--<br>
coreboot mailing list: <a href="mailto:coreboot@coreboot.org">coreboot@coreboot.org</a><br>
<a href="http://www.coreboot.org/mailman/listinfo/coreboot" target="_blank">http://www.coreboot.org/mailman/listinfo/coreboot</a><br>
</div></div></blockquote></div><br></div>