[PATCH] powerpc: Fix invalid page flags in create TLB CAM pathfor PTE_64BIT
Scott Wood
scottwood at freescale.com
Sat Sep 25 01:02:11 EST 2010
On Fri, 24 Sep 2010 07:04:28 +0200
"Chen, Tiejun" <Tiejun.Chen at windriver.com> wrote:
> > -----Original Message-----
> > From:
> > linuxppc-dev-bounces+tiejun.chen=windriver.com at lists.ozlabs.or
> > g
> > [mailto:linuxppc-dev-bounces+tiejun.chen=windriver.com at lists.o
> > zlabs.org] On Behalf Of Benjamin Herrenschmidt
> > Sent: Friday, September 24, 2010 5:59 AM
> > To: Scott Wood
> > Cc: Gortmaker, Paul; linuxppc-dev at lists.ozlabs.org
> > Subject: Re: [PATCH] powerpc: Fix invalid page flags in
> > create TLB CAM pathfor PTE_64BIT
> >
> > On Thu, 2010-09-23 at 15:33 -0500, Scott Wood wrote:
> > > I don't see a generic accessor that can test PTE flags for
> > user access
> > > -- in the absence of one, I guess we need an ifdef here.
> > Or at least
> > > put in a comment so anyone who adds a userspace use knows
> > they need to
> > > fix it.
> >
> > We could make up one in powerpc arch at least
> >
> > #define pte_user(val) ((val & _PAGE_USER) == _PAGE_USER)
> >
>
> Looks good.
>
> Ben and Scott,
>
> But for the patched issue we're discussing we have to do #ifdef that as
> my original modification. Right? Or do you have other suggestion? Then I
> can improve that as v2.
Ben's version should work without any ifdef, since it makes sure
all bits of _PAGE_USER are set.
-Scott
More information about the Linuxppc-dev
mailing list