Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-07 Thread David Gibson
On Thu, Feb 03, 2011 at 05:15:57PM -0600, Dave Kleikamp wrote: > On Thu, 2011-02-03 at 16:03 +1100, David Gibson wrote: > > On Wed, Feb 02, 2011 at 05:53:59PM -0600, Dave Kleikamp wrote: > > > On Thu, 2011-02-03 at 10:08 +1100, David Gibson wrote: > > > > On Tue, Feb 01, 2011 at 12:48:44PM -0600, D

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-04 Thread Dave Kleikamp
On Fri, 2011-02-04 at 08:56 -0500, Josh Boyer wrote: > On Tue, Feb 01, 2011 at 12:48:44PM -0600, Dave Kleikamp wrote: > >diff --git a/arch/powerpc/mm/tlb_nohash.c b/arch/powerpc/mm/tlb_nohash.c > >index 2a030d8..b33c5e6 100644 > >--- a/arch/powerpc/mm/tlb_nohash.c > >+++ b/arch/powerpc/mm/tlb_nohas

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-04 Thread Josh Boyer
On Tue, Feb 01, 2011 at 12:48:44PM -0600, Dave Kleikamp wrote: >diff --git a/arch/powerpc/mm/tlb_nohash.c b/arch/powerpc/mm/tlb_nohash.c >index 2a030d8..b33c5e6 100644 >--- a/arch/powerpc/mm/tlb_nohash.c >+++ b/arch/powerpc/mm/tlb_nohash.c >@@ -35,6 +35,7 @@ > #include > #include > #include >+#i

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-03 Thread Timur Tabi
On Thu, Feb 3, 2011 at 5:15 PM, Dave Kleikamp wrote: >> Bit too short, I think.  I'd suggest either spelling out >> 'asymmetric-multiprocessor' or 'cooperative-partition' (a more >> accurate term, IMO). > > I could be wrong, but I thought the A stands for Asynchronous, not > Asymmetric.  I though

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-03 Thread Dave Kleikamp
On Thu, 2011-02-03 at 16:03 +1100, David Gibson wrote: > On Wed, Feb 02, 2011 at 05:53:59PM -0600, Dave Kleikamp wrote: > > On Thu, 2011-02-03 at 10:08 +1100, David Gibson wrote: > > > On Tue, Feb 01, 2011 at 12:48:44PM -0600, Dave Kleikamp wrote: > > > > Since other OS's may be running on the othe

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-02 Thread David Gibson
On Wed, Feb 02, 2011 at 05:53:59PM -0600, Dave Kleikamp wrote: > On Thu, 2011-02-03 at 10:08 +1100, David Gibson wrote: > > On Tue, Feb 01, 2011 at 12:48:44PM -0600, Dave Kleikamp wrote: > > > Since other OS's may be running on the other cores don't use tlbivax > > > > [snip] > > > +#ifdef CONFIG_

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-02 Thread Dave Kleikamp
On Thu, 2011-02-03 at 10:08 +1100, David Gibson wrote: > On Tue, Feb 01, 2011 at 12:48:44PM -0600, Dave Kleikamp wrote: > > Since other OS's may be running on the other cores don't use tlbivax > > [snip] > > +#ifdef CONFIG_44x > > +void __init early_init_mmu_44x(void) > > +{ > > + unsigned long

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-02 Thread David Gibson
On Tue, Feb 01, 2011 at 12:48:44PM -0600, Dave Kleikamp wrote: > Since other OS's may be running on the other cores don't use tlbivax [snip] > +#ifdef CONFIG_44x > +void __init early_init_mmu_44x(void) > +{ > + unsigned long root = of_get_flat_dt_root(); > + if (of_flat_dt_is_compatible(ro

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-02 Thread Dave Kleikamp
On Wed, 2011-02-02 at 01:48 -0600, Kumar Gala wrote: > On Feb 1, 2011, at 12:48 PM, Dave Kleikamp wrote: > > > Since other OS's may be running on the other cores don't use tlbivax > > Are you guys building SMP kernel for use with AMP? Just wondering why you'd > be using tlbivax at all. Yes, fo

Re: [PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-01 Thread Kumar Gala
On Feb 1, 2011, at 12:48 PM, Dave Kleikamp wrote: > Since other OS's may be running on the other cores don't use tlbivax Are you guys building SMP kernel for use with AMP? Just wondering why you'd be using tlbivax at all. - k ___ Linuxppc-dev maili

[PATCH V2 4/6] powerpc/44x: don't use tlbivax on AMP systems

2011-02-01 Thread Dave Kleikamp
Since other OS's may be running on the other cores don't use tlbivax Signed-off-by: Dave Kleikamp Cc: Benjamin Herrenschmidt Cc: Josh Boyer Cc: linuxppc-dev@lists.ozlabs.org --- arch/powerpc/include/asm/mmu.h |2 +- arch/powerpc/kernel/setup_32.c |2 ++ arch/powerpc/mm/tlb_nohash.c |