Re: [PATCH/RFC] powerpc/mm: Cleanup handling of execute permission

2009-08-19 Thread Benjamin Herrenschmidt
On Wed, 2009-08-19 at 15:59 -0500, Becky Bruce wrote: > On Aug 18, 2009, at 5:33 PM, Benjamin Herrenschmidt wrote: > > > > FYI, I pulled your updated test branch this morning, booted, and did a > full LTP run on 8572. The results are consistent with the baseline I > have, so it looks like th

Re: [PATCH/RFC] powerpc/mm: Cleanup handling of execute permission

2009-08-19 Thread Becky Bruce
On Aug 18, 2009, at 5:33 PM, Benjamin Herrenschmidt wrote: On Tue, 2009-08-18 at 16:56 -0400, Josh Boyer wrote: On Fri, Aug 14, 2009 at 05:39:42PM -0500, Becky Bruce wrote: Ben, This breaks the boot on 8572. I don't know why yet (and I'm probably not going to figure it out before I go hom

Re: [PATCH/RFC] powerpc/mm: Cleanup handling of execute permission

2009-08-18 Thread Benjamin Herrenschmidt
On Tue, 2009-08-18 at 16:56 -0400, Josh Boyer wrote: > On Fri, Aug 14, 2009 at 05:39:42PM -0500, Becky Bruce wrote: > > Ben, > > > > This breaks the boot on 8572. I don't know why yet (and I'm probably > > not going to figure it out before I go home, because, frankly, it's late > > on a Friday

Re: [PATCH/RFC] powerpc/mm: Cleanup handling of execute permission

2009-08-18 Thread Josh Boyer
On Fri, Aug 14, 2009 at 05:39:42PM -0500, Becky Bruce wrote: > Ben, > > This breaks the boot on 8572. I don't know why yet (and I'm probably > not going to figure it out before I go home, because, frankly, it's late > on a Friday afternoon and I need a glass of wine or, perhaps, a beer). > > Ku

Re: [PATCH/RFC] powerpc/mm: Cleanup handling of execute permission

2009-08-14 Thread Becky Bruce
Ben, This breaks the boot on 8572. I don't know why yet (and I'm probably not going to figure it out before I go home, because, frankly, it's late on a Friday afternoon and I need a glass of wine or, perhaps, a beer). Kumar and I will poke into this more and let you know what we find out

[PATCH/RFC] powerpc/mm: Cleanup handling of execute permission

2009-07-28 Thread Benjamin Herrenschmidt
This is an attempt at cleaning up a bit the way we handle execute permission on powerpc (again !). _PAGE_HWEXEC is gone, _PAGE_EXEC is now only defined by CPUs that can do something with it, and the myriad of #ifdef's in the I$/D$ coherency code is reduced to 2 cases that hopefully should cover ev