[PPC64] Remove LARGE_PAGE_SHIFT constant
David Gibson
david at gibson.dropbear.id.au
Sat Sep 18 09:43:57 EST 2004
On Fri, Sep 17, 2004 at 02:33:34PM -0300, Marcelo Tosatti wrote:
> On Fri, Sep 17, 2004 at 11:40:29AM -0700, Dave Hansen wrote:
> > On Fri, 2004-09-17 at 10:03, Marcelo Tosatti wrote:
> > > On Fri, Sep 17, 2004 at 11:13:20AM +1000, David Gibson wrote:
> > > > Andrew, please apply:
> > > >
> > > > For historical reasons, ppc64 has ended up with two #defines for the
> > > > size of a large (16M) page: LARGE_PAGE_SHIFT and HPAGE_SHIFT. This
> > > > patch removes LARGE_PAGE_SHIFT in favour of the more widely used
> > > > HPAGE_SHIFT.
> > >
> > > Nitpicking, "LARGE_PAGE_xxx" is used by x86/x86_64:
> > >
> > > #define LARGE_PAGE_MASK (~(LARGE_PAGE_SIZE-1))
> > > #define LARGE_PAGE_SIZE (1UL << PMD_SHIFT)
> > >
> > > Wouldnt it be nice to keep consistency between archs?
> >
> > Actually, if everybody makes sure to define PMD_SHIFT, we should be able
> > to use common macros, right?
>
> Yeap.
No. The large page shift is not necessarily the same as the
PMD_SHIFT. It's the same on x86, but it is less on sparc64 and sh,
and greater on ppc64.
--
David Gibson | For every complex problem there is a
david AT gibson.dropbear.id.au | solution which is simple, neat and
| wrong.
http://www.ozlabs.org/people/dgibson
More information about the Linuxppc64-dev
mailing list