Re: [PATCH V1 1/2] MAINTAINERS: add include/linux/pgalloc.h to MM CORE section

[Harry Yoo <harry.yoo@oracle.com>]

On Mon, Oct 20, 2025 at 03:05:52PM +0100, Lorenzo Stoakes wrote:
> On Mon, Oct 20, 2025 at 04:43:33PM +0900, Harry Yoo wrote:
> > On Sun, Oct 19, 2025 at 03:07:36PM -0700, SeongJae Park wrote:
> > > On Sun, 19 Oct 2025 16:25:08 +0900 Harry Yoo <harry.yoo@oracle.com> wrote:
> > >
> > > > Since include/linux/pgtable.h is already listed in the MM CORE section,
> > > > add it to the section as well to keep it maintained by the appropriate
> > > > maintainers.
> > > >
> > > > Signed-off-by: Harry Yoo <harry.yoo@oracle.com>
> 
> Thanks for adding, but please put before pgtable.h to fix the checkpatch
> warning.
> 
> With that done:
> 
> Acked-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> 
> > >
> > > Acked-by: SeongJae Park <sj@kernel.org>
> >
> > Thanks!
> >
> > > > ---
> > > >  MAINTAINERS | 1 +
> > > >  1 file changed, 1 insertion(+)
> > > >
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index 46126ce2f968..dedb9525c2bc 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -16249,6 +16249,7 @@ F:	include/linux/mmdebug.h
> > > >  F:	include/linux/mmu_notifier.h
> > > >  F:	include/linux/pagewalk.h
> > > >  F:	include/linux/pgtable.h
> > > > +F:	include/linux/pgalloc.h
> > >
> > > Nit.  Should we alphabetically sort?
> >
> > TBH I have no idea.
> >
> > I thought only subsystem names should be alphabetically sorted?
> 
> No it's files too, checkpatch will tell you:
> 
>    ● checkpatch.pl: 262: WARNING: Misordered MAINTAINERS entry - list file patterns in alphabetic order

*blaming myself for not running checkpatch*

Thanks, will fix it!

-- 
Cheers,
Harry / Hyeonggon