Robin Holt wrote on Friday, April 28, 2006 8:34 AM
> On Thu, Apr 27, 2006 at 07:09:17PM -0700, Chen, Kenneth W wrote:
> > --- ./arch/ia64/kernel/ivt.S.orig 2006-04-27 19:21:55.000000000 -0700
> > +++ ./arch/ia64/kernel/ivt.S 2006-04-27 19:46:44.000000000 -0700
> > @@ -142,16 +142,16 @@ ENTRY(vhpt_miss)
> > (p7) dep r17=r18,r17,3,(PAGE_SHIFT-6) // r17=pgd_offset for region[0-4]
> > cmp.eq p7,p6=0,r21 // unused address bits all zeroes?
> > #ifdef CONFIG_PGTABLE_4
> > - shr.u r28=r22,PUD_SHIFT // shift pud index into position
> > + extr.u r28=r22,PUD_SHIFT,11 // get pud index
>
> How well does this work with different page sizes?
Thanks! I should've use PTRS_PER_PTD_SHIFT for that constant "11". I forgot
to put that in.
> > +
> > +#ifdef CONFIG_PGTABLE_4
> > +int pgtbl3;
> > +static __init int setup_pgtbl3(char *s)
> > +{
> > + pgtbl3 = 1;
> > + return 0;
> > +}
> > +early_param("pgtbl3", setup_pgtbl3);
> > +#endif
>
> What about taking out CONFIG_PGTABLE_*, having the default be 3, and have the
> boot option enable 4 levels? I haven't given this much though yet, but my
> first reaction was that would be the correct way to go.
OK, if there isn't any objections from other people, I will go ahead and do
it that way.
-
: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|