Patchwork Slot 1 regression! Asus P2B/P3B boards no longer boots with model_6bx CPUs

login
register
about
Submitter Keith Hui
Date 2010-10-12 04:09:38
Message ID <AANLkTiksyUZ+opTk=KZfv5sWMkKEX7_6bSSq8g1rdBXq@mail.gmail.com>
Download mbox | patch
Permalink /patch/2099/
State Accepted
Headers show

Comments

Keith Hui - 2010-10-12 04:09:38
On Mon, Oct 11, 2010 at 9:41 PM, Idwer Vollering <vidwer@gmail.com> wrote:
> 2010/10/12 Keith Hui <buurin@gmail.com>
>>
>> Guys,
>
> Hi Keith,
>>
>> I could no longer boot my P3B-F with my Tualeron and r5938. Dies with
>> "unknown CPU". I believe it will happen with any Slot 1 440BX boards
>> that supports model_6bx CPUs.
>
> Can you confirm that this changeset caused the regression:
> http://tracker.coreboot.org/trac/coreboot/changeset/5909 ?
Not that one directly. This regression was caused by the entire
cpu/intel/model_6bx branch not being pulled in for cpu/intel/slot_1 so
the Tualatin cpuids are missing from the whitelist. So it must be one
of the patches when we tried to clean up the Makefiles and forgot that
model_6bx is a possibility for slot 1 boards and needs to be
included..

>>
>> I need to make the change below to make it work. abuild tested. Boot
>> tested on P2B-LS and P3B-F.
>
>
> In which directory does this changed file reside, src/cpu/intel/slot_1/ or
> src/cpu/intel/model_6bx/ ?
>
src/cpu/intel/slot_1

Thanks for the catch. Updated below.
>>
>> After we split the rest of the P6 family from model_6xx, a similar
>> change will need to be made here.
>>
Signed-off-by: Keith Hui <buurin@gmail.com>
>>
>> 8<-------------------------
Uwe Hermann - 2010-10-12 23:23:58
On Tue, Oct 12, 2010 at 12:09:38AM -0400, Keith Hui wrote:
> >> After we split the rest of the P6 family from model_6xx, a similar
> >> change will need to be made here.
> >>
> Signed-off-by: Keith Hui <buurin@gmail.com>

Thanks, r5945.

Can you post an extra patch for splitting off CPUs to make it easier to
review? The last one was merged into the already huge L2 patch which
made it really hard to read.

After the split, your L2 patch should be relatively small and clean,
please repost that too then.


Thanks, Uwe.

Patch

Index: Makefile.inc
===================================================================
--- src/cpu/intel/slot_1/Makefile.inc        (revision 5938)
+++ src/cpu/intel/slot_1/Makefile.inc        (working copy)
@@ -20,6 +20,7 @@ 
 ramstage-y += slot_1.c
 subdirs-y += ../model_6xx
+subdirs-y += ../model_6bx
 subdirs-y += ../../x86/tsc
 subdirs-y += ../../x86/mtrr
 subdirs-y += ../../x86/lapic