Messages in this thread | | | Date | Mon, 3 Sep 2018 13:52:06 +0100 | From | Will Deacon <> | Subject | Re: [PATCH 00/12] Avoid synchronous TLB invalidation for intermediate page-table entries on arm64 |
| |
On Fri, Aug 31, 2018 at 08:32:34PM +1000, Nicholas Piggin wrote: > On Fri, 31 Aug 2018 12:10:14 +0200 > Peter Zijlstra <peterz@infradead.org> wrote: > > > On Fri, Aug 31, 2018 at 10:54:18AM +0100, Will Deacon wrote: > > > > > Proposal below (omitted Linus because that seems to be the pattern elsewhere > > > in the file and he's not going to shout at himself when things break :) > > > Anybody I've missed? > > > > > > Will > > > > > > --->8 > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > index a5b256b25905..7224b5618883 100644 > > > --- a/MAINTAINERS > > > +++ b/MAINTAINERS > > > @@ -9681,6 +9681,15 @@ S: Maintained > > > F: arch/arm/boot/dts/mmp* > > > F: arch/arm/mach-mmp/ > > > > > > +MMU GATHER AND TLB INVALIDATION > > > +M: Will Deacon <will.deacon@arm.com> > > > +M: Nick Piggin <npiggin@gmail.com> > > Oh gee, I suppose. powerpc hash is kind of interesting because it's > crazy, Aneesh knows that code a lot better than I do. radix modulo > some minor details of exact instructions is fairly like x86 (he > wrote a lot of that code too AFAIK).
Sure, as long as we have Power represented here. Would you rather add Aneesh instead of yourself, or shall we just add you both?
> > > > +M: Peter Zijlstra <peterz@infradead.org> > > > +L: linux-arch@vger.kernel.org > > Maybe put linux-mm as well? Or should there just be one list?
If we do the landgrab on mmu_gather (which I think makes sense), then adding both lists makes sense to me. I'll spin this as a proper patch, along with Peter's code move.
> > > +S: Maintained > > > +F: include/asm-generic/tlb.h > > > +F: arch/*/include/asm/tlb.h > > > + > > > MN88472 MEDIA DRIVER > > > M: Antti Palosaari <crope@iki.fi> > > > L: linux-media@vger.kernel.org > > > > If we're going to do that (and I'm not opposed); it might make sense to > > do something like the below and add: > > > > F: mm/mmu_gather.c > > I think that is a good idea regardless. How do feel about calling it > tlb.c? Easier to type and autocompletes sooner.
No strong opinion on name, but I slightly prefer mmu_gather.c so that it avoids any remote possibility of confusion with tlb.c vs hugetlb.c
Will
| |