<div class="gmail_quote">On Tue, Apr 14, 2009 at 6:16 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="im">On 14.04.2009 17:41, <a href="mailto:svn@coreboot.org">svn@coreboot.org</a> wrote:<br>
> Author: cozzie<br>
> Date: 2009-04-14 17:41:33 +0200 (Tue, 14 Apr 2009)<br>
> New Revision: 1164<br>
><br>
> Modified:<br>
>    coreboot-v3/arch/x86/pirq_routing.c<br>
>    coreboot-v3/arch/x86/via/stage0.S<br>
>    coreboot-v3/mainboard/jetway/Kconfig<br>
> Log:<br>
> Enable caching for Via C7 CPUs, and also improve readability. Tested on hardware<br>
> and seems to be working.<br>
><br>
> Signed-off-by: Corey Osgood <<a href="mailto:corey.osgood@gmail.com">corey.osgood@gmail.com</a>><br>
> Acked-by: Myles Watson <<a href="mailto:mylesgw@gmail.com">mylesgw@gmail.com</a>><br>
><br>
<br>
</div>Sorry I didn't see the patch earlier.<br>
<br>
If (CONFIG_COREBOOT_ROMSIZE_KB + CONFIG_CARSIZE) exceeds L2 cache size,<br>
CAR will explode. I bet your test conditions didn't trigger that condition.<br>
You can calculate allowed ROM cache size, though. CPUID can tell you L2<br>
size and from that you subtract CARSIZE, then round down to the next<br>
power of two.</blockquote><div><br>Alright, I'll fix it tonight. The bootblock and stage0/1 code should always be at the start of the ROM, right? Is there a chance of explosion if stage0+1 is larger then the cached size?<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>
<br>
It seems some unrelated PIRQ change snuck in this changeset.<br>
</blockquote></div><br>Sorry about that, I forgot my svn password and I guess in the process I forgot to single out that file when I figured it out. The Kconfig change is needed to build the Jetway target, so I think it should stay, and the pirq_routing.c is simply clarification, the file does actually reside in include/arch/x86/. If anyone wants the latter revert the latter, feel free.<br>
<br>Thanks,<br>Corey<br>