Re: [PATCH/RFC] Shared page tables

From: Hugh Dickins
Date: Tue Jan 24 2006 - 13:17:57 EST


On Tue, 24 Jan 2006, Dave McCracken wrote:
> --On Tuesday, January 24, 2006 17:50:17 +0000 Hugh Dickins
> <hugh@xxxxxxxxxxx> wrote:
> > On Mon, 23 Jan 2006, Dave McCracken wrote:
> >
> >> I needed a function that returns a struct page for pgd and pud, defined
> >> in each architecture. I decided the simplest way was to redefine
> >> pgd_page and pud_page to match pmd_page and pte_page. Both functions
> >> are pretty much used one place per architecture, so the change is
> >> trivial. I could come up with new functions instead if you think it's
> >> an issue. I do have a bit of a fetish about symmetry across levels :)
> >
> > Sounds to me like you made the right decision.
>
> I had a thought... would it be preferable for me to make this change as a
> separate patch across all archictures in the name of consistency? Or

Yes - but don't expect it to be taken if your shared pagetables aren't:
just submit it as the first(ish) patch in your shared pagetables series.

> should I continue to roll it into the shared pagetable patch as we enable
> each architecture?

Hugh
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/