Re: [PATCH] powerpc/book3e-64: use a separate TLB handler when linear map is bolted

2011-06-22 Thread Benjamin Herrenschmidt
On Wed, 2011-06-22 at 16:24 -0500, Scott Wood wrote: > > This saved another 1% or so. I don't think TLB_SCRATCH needs to be saved > by crit/debug/mcheck -- they shouldn't be taking TLB misses, at least > in the bolted case. The code they call will (think accessing the UART) > Even non-bolted,

Re: [PATCH] powerpc/book3e-64: use a separate TLB handler when linear map is bolted

2011-06-22 Thread Scott Wood
On Sat, 18 Jun 2011 08:44:29 +1000 Benjamin Herrenschmidt wrote: > Can't you just re-org the PACA instead ? (with a comment) ? Or at least > if you want to keep it that way, put the cache line explanation in a > comment somewhere. OK. > > > > +.macro tlb_prolog_bolted addr > > > > + mtspr

Re: [PATCH] powerpc/book3e-64: use a separate TLB handler when linear map is bolted

2011-06-17 Thread Benjamin Herrenschmidt
On Fri, 2011-06-17 at 11:32 -0500, Scott Wood wrote: > On Fri, 17 Jun 2011 12:00:50 +1000 > Benjamin Herrenschmidt wrote: > > > Does this completely replace your previous series of 7 patches ? (IE. > > Should I ditch them in patchwork ?) Or does it apply on top of them ? > > It replaces them. >

Re: [PATCH] powerpc/book3e-64: use a separate TLB handler when linear map is bolted

2011-06-17 Thread Scott Wood
On Fri, 17 Jun 2011 12:00:50 +1000 Benjamin Herrenschmidt wrote: > Does this completely replace your previous series of 7 patches ? (IE. > Should I ditch them in patchwork ?) Or does it apply on top of them ? It replaces them. > > #define SET_IVOR(vector_number, vector_offset) \ > > diff -

Re: [PATCH] powerpc/book3e-64: use a separate TLB handler when linear map is bolted

2011-06-16 Thread Benjamin Herrenschmidt
On Fri, 2011-06-03 at 17:12 -0500, Scott Wood wrote: > On MMUs such as FSL where we can guarantee the entire linear mapping is > bolted, we don't need to worry about linear TLB misses. If on top of > that we do a full table walk, we get rid of all recursive TLB faults, and > can dispense with some