I agree.<br>
<br>
The cache_as_ram_auto.c should be the same to all Opteron MB except for <br>
1 spd rom related.<br>
2 superio related<br>
<br>
Other thing is need to consider apic id lifting.<br>
<br>
So i consider change that to <br>
<br>
#include "auto_1.c"<br>
int someting spd and superio related<br>
#include "auto_2.c"<br>
<br>
<br>
YH<br><br><div><span class="gmail_quote">On 9/6/05, <b class="gmail_sendername">Eric W. Biederman</b> <<a href="mailto:ebiederman@lnxi.com">ebiederman@lnxi.com</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
"Lu, Yinghai" <<a href="mailto:yinghai.lu@amd.com">yinghai.lu@amd.com</a>> writes:<br><br>> I tried to make cache_as_ram_auto.c more smaller, the problem is<br>> I can not use jmp __cpu_reset, and I need use goto cpu_reset_x
<br><br>The practical problem was not how much the code had to do<br>(except for failover.c) but instead how little of that code<br>was in common functions, and instead was per-motherboard.<br>Implementing on one motherboard while experimenting is fine but when
<br>you go multi-board too much cut and paste is dangerous.<br><br>The first round of dual core and cache as ram code feels<br>like an old style adventure game:<br><br>   You have now entered a twisty maze of ifdefs.<br><br>
Eric<br><br>--<br>LinuxBIOS mailing list<br><a href="mailto:LinuxBIOS@openbios.org">LinuxBIOS@openbios.org</a><br><a href="http://www.openbios.org/mailman/listinfo/linuxbios">http://www.openbios.org/mailman/listinfo/linuxbios
</a><br></blockquote></div><br>