YH,<br>
<br>
I've applied your patch, and started porting some of the bits over from
the s2895 code, however it looks like I'm missing
"cpu/amd/model_fxx/init_cpus.c" or wherever 'init_cpus()' is defined.<br>
<br>
The cleanups look pretty damned good so far though... still trying to get building to work...<br>
<br>
Kudos anyawys ;)<br>
<br>
-san<br>
<br><br><div><span class="gmail_quote">On 10/26/05, <b class="gmail_sendername">Lu, Yinghai</b> <<a href="mailto:yinghai.lu@amd.com">yinghai.lu@amd.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;">












<div>

<p><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial; color: navy;">San,</span></font></p>

<p><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial; color: navy;"> </span></font></p>

<p><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial; color: navy;">Please check the patch for CAR</span></font></p>

<ol style="margin-top: 0in;" start="1" type="1">
 <li style="color: navy;"><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial;">add
     cpu_init_detectedx, it is passed from cache_as_ram.inc and to amd64_main…
     </span></font></li>
 <li style="color: navy;"><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial;">add
     init_cpus for CAR ---more readable</span></font></li>
 <li style="color: navy;"><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial;">post_cache_as_ram  
     -</span></font><font face="Wingdings" size="2"><span style="font-size: 10pt; font-family: Wingdings;">à</span></font><font face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial;"> make cache_as_ram_auto.c
     shorter….</span></font></li>
 <li style="color: navy;"><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial;">renable
     get_apicid_base, in case some one still want BSP apid id =0 but other cpu
     is lifted.</span></font></li>
</ol>

<p><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial; color: navy;"> </span></font></p>

<p><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial; color: navy;">YH</span></font></p>

<p><font color="navy" face="Arial" size="2"><span style="font-size: 10pt; font-family: Arial; color: navy;"> </span></font></p>

<div>

<div style="text-align: center;" align="center"><font face="Times New Roman" size="3"><span style="font-size: 12pt;">

<hr align="center" size="2" width="100%">

</span></font></div>

<p><b><font face="Tahoma" size="2"><span style="font-size: 10pt; font-family: Tahoma; font-weight: bold;">From:</span></font></b><font face="Tahoma" size="2"><span style="font-size: 10pt; font-family: Tahoma;">
<a href="mailto:linuxbios-bounces@openbios.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">linuxbios-bounces@openbios.org</a> [mailto:<a href="mailto:linuxbios-bounces@openbios.org" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">
linuxbios-bounces@openbios.org</a>] <b><span style="font-weight: bold;">On Behalf Of </span></b>San Mehat<br>
<b><span style="font-weight: bold;">Sent:</span></b> Wednesday, October 26, 2005
9:55 AM<br>
<b><span style="font-weight: bold;">To:</span></b> Eric W. Biederman<br>
<b><span style="font-weight: bold;">Cc:</span></b> LinuxBIOS<br>
<b><span style="font-weight: bold;">Subject:</span></b> Re: [LinuxBIOS] Latest
commit - clarification</span></font></p>

</div>

<p><font face="Times New Roman" size="3"><span style="font-size: 12pt;"> </span></font></p>

<p style="margin-bottom: 12pt;"><font face="Times New Roman" size="3"><span style="font-size: 12pt;"> </span></font></p>

<div><span class="q">

<p><span><font face="Times New Roman" size="3"><span style="font-size: 12pt;">On 10/25/05, <b><span style="font-weight: bold;">Eric W.
Biederman</span></b> <<a href="mailto:ebiederman@lnxi.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">ebiederman@lnxi.com</a>>
wrote:</span></font></span></p>

<p><font face="Times New Roman" size="3"><span style="font-size: 12pt;">killing get_apicid_base looks like a bug.<br>
But as far as I can tell everything that is gone is now unnecessary. </span></font></p>

</span><div>

<p style="margin-bottom: 12pt;"><font face="Times New Roman" size="3"><span style="font-size: 12pt;"><br>
Okay... so what replace start_other_cores()?.. In CAR mode, do I make a call to
'do_k8_init_and_stop_secondaries()'?<span class="q"><br>
That code itself wont build due to the following in
'k8_init_and_stop_secondaries()':<br>
  if (init_detected) {<br>
               
asm volatile ("jmp __cpu_reset");<br>
        }<br>
<br>
because in CAR configs, arch/i386/lib/cpu_reset.inc is not included. On this
particular issue, I'm happy to wait for YH's fixes.<br>
<br>
I just think if we're going to deprecate a core call like that, there should be
some documentation on *why* it was removed, as well as some indication as to
where the behavior got 'moved' to.<br>
<br></span><span class="q">
It just makes things a lot easier for thoes of us that are either ramping up,
or trying to maintain things..<br>
<br>
Thanks Eric ;)<br>
<br>
-san<br>
<br>
PS: I'm still going to get you drunk and shave your beard.</span></span></font></p>

</div>

<p><font face="Times New Roman" size="3"><span style="font-size: 12pt;"> </span></font></p>

</div>

<p><font face="Times New Roman" size="3"><span style="font-size: 12pt;"> </span></font></p>

</div>





<br clear="all"></blockquote></div><br>