[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <47E1EAD6.5080905@redhat.com>
Date: Thu, 20 Mar 2008 01:40:54 -0300
From: Glauber Costa <gcosta@...hat.com>
To: Yinghai Lu <yhlu.kernel@...il.com>
CC: Ingo Molnar <mingo@...e.hu>, linux-kernel@...r.kernel.org,
akpm@...ux-foundation.org, tglx@...utronix.de,
Andi Kleen <andi@...stfloor.org>
Subject: Re: [PATCH 0/79] smpboot integration
Yinghai Lu wrote:
> On Wed, Mar 19, 2008 at 8:00 PM, Yinghai Lu <yhlu.kernel@...il.com> wrote:
>> On Wed, Mar 19, 2008 at 7:18 PM, Yinghai Lu <yhlu.kernel@...il.com> wrote:
>> > On Wed, Mar 19, 2008 at 10:35 AM, Ingo Molnar <mingo@...e.hu> wrote:
>> > >
>> > > * Glauber de Oliveira Costa <gcosta@...hat.com> wrote:
>> > >
>> > > > Testing and bisectability:
>> > > >
>> > > > The end result was tested in all my hardware (which includes qemu ;-).
>> > > > It does not mean it will boot _your_ hardware, but I did my best ;-)
>> > > >
>> > > > The tree at least compiles in more than 20 randconfigs (for each of
>> > > > x86_64 and i386) For i386, each of the subarchitectures was compiled
>> > > > at least once. (By compile, I obviously mean, every patch,
>> > > > individually)
>> > >
>> > > very nice work! I'll pick it up - and i'm not too worried about
>> > > breakages because at 80 patches granularity any problem should be
>> > > identifiable in a very finegrained way.
>> > >
>> >
>> > it broke 4 sockets quad core above with 64 bit
>> >
>> > Booting processor 11/15 ip 6000
>> > Initializing CPU#11
>> > masked ExtINT on CPU#11
>> > Calibrating delay using timer specific routine.. 4589.46 BogoMIPS (lpj=9178934)
>> > CPU: L1 I Cache: 64K (64 bytes/line), D cache 64K (64 bytes/line)
>> > CPU: L2 Cache: 512K (64 bytes/line)
>> > CPU 11/f -> Node 2
>> > CPU: Physical Processor ID: 2
>> > CPU: Processor Core ID: 3
>> > CPU11: Quad-Core AMD Opteron(tm) Processor 8356 stepping 03
>> > checking TSC synchronization [CPU#0 -> CPU#11]: passed.
>> > Booting processor 12/16 ip 6000
>> >
>> > looks like local apic id up 4 bit is masked out. so can not start 0x10
>> > above any more.
>>
>> in wakeup_secondary_via_INIT
>> before the patchsets
>> 64 bit code:
>>
>> /*
>> * Send IPI
>> */
>> apic_write(APIC_ICR, APIC_INT_LEVELTRIG | APIC_INT_ASSERT
>> | APIC_DM_INIT);
>>
>>
>> after patchset
>>
>> /* Boot on the stack */
>> /* Kick the second */
>> apic_write_around(APIC_ICR, APIC_DM_NMI | APIC_DEST_LOGICAL);
>>
>> So that is wrong! esp for system has ext apic id that is has 8 bits
>> instead of 4 bits.
>>
>
> it seems there is two wakeup_secondary_cpu. one for NMI and one INIT.
>
> but should have
>
> #define WAKE_SECONDARY_VIA_INIT
>
> for x86_64
>
> but after
>
> #ifdef CONFIG_X86_64
> #undef WAKE_SECONDARY_VIA_NMI
> #define WAKE_SECONDARY_VIA_INIT
> #endif
>
> it still doesn't work.
>
> YH
Thanks for the testing Yinghai. I'll take a deeper look as soon as I
can. The two routines are provided, since i386 numa-q inits the startup
sequence through NMIs. The _VIA_INIT is already defined in x86_64 in the
mach-default/ headers.
What happens exactly? Does it hang indefinitely ? Or just for a while?
Also, can you provide the exact commit in which this problem start?
(just to be sure)
As a debugging aid, can you also define the Dprintks in the code? I've
seen hangs before in which the processor was indeed executing its init
sequence, (although it did not seem to), but was hanging in the
calibrate loop.
Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists