mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Michael Ellerman <mpe@ellerman.id.au>
To: Joe Perches <joe@perches.com>,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Randy Dunlap <rdunlap@infradead.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH RESEND v4] MAINTAINERS: fix lots of alphabetic ordering
Date: Thu, 20 Jul 2017 20:54:18 +1000	[thread overview]
Message-ID: <87tw27gyzp.fsf@concordia.ellerman.id.au> (raw)
In-Reply-To: <1500525809.14415.3.camel@perches.com>

Joe Perches <joe@perches.com> writes:

> On Wed, 2017-07-19 at 21:24 -0700, Linus Torvalds wrote:
>> On Wed, Jul 19, 2017 at 6:05 PM, Joe Perches <joe@perches.com> wrote:
>> > 
>> > Just for ease of manipulation and not breaking the script much,
>> > I'd suggest just having a MAINTAINERS directory and stuffing
>> > each of the sections into separate files.
>> > 
>> > The script would only need to add $ cat MAINTAINERS/* as input.
>> 
>> So I don't mind the idea of just making MAINTAINERS a directory, but I
>> don't think we want to so far as to make one file per entry. That's
>> what, 1500+ files tiny files or so? Seems a bit excessive.
>> 
>> Maybe we can just do the prefix thing and just do 26 files A-Z
>> instead? Or maybe go by first word (so all the ARM things would go in
>> one place?)
>> 
>> A couple of hundred files sounds fine. A couple of thousand files
>> sound a bit excessive..
>
> $ ls MAINTAINERS.tmp/ | wc -l
> 1735
>
> <shrug>
>
> A couple thousand individual maintainers is also excessive.
>
> Most maintainers in MAINTAINERS aren't really much involved.
>
> A-Z is arbitrary and still difficult to find because it's
> not descriptive as to whatever is actually maintained.
>
> As a concept I think individual files would be better.
> But maybe grouping by subsystem instead of by letter.
>
> Maybe by mirroring the directory layouts.
>
> <by arch>
> drivers/net
> drivers/scsi

Or what about putting separate MAINTAINERS files in the hierarchy at
whichever level makes the most sense, ie. not too big and not too small.

eg.

arch/x86/MAINTAINERS
arch/arm64/MAINTAINERS
arch/powerpc/MAINTAINERS
...
net/MAINTAINERS
mm/MAINTAINERS
drivers/net/MAINTAINERS
drivers/media/MAINTAINERS
drivers/scsi/MAINTAINERS
drivers/gpu/drm/MAINTAINERS
drivers/MAINTAINERS

That way we might end up with 20-50 (?) files, but most of the conflicts
would be handled by a sub system maintainer before Linus sees them.

We might still need a top level file for "other", for things that really
don't fit anywhere.

Files would be allowed to have patterns that match elsewhere in the
tree, so eg. arch files could match against arch specific code in
drivers.

And you can still get the full list just with:

 $ find . -name MAINTAINERS | xargs ca

cheers

  parent reply	other threads:[~2017-07-20 10:54 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-19 21:40 Randy Dunlap
2017-07-19 21:44 ` Andrew Morton
2017-07-20  0:08   ` Linus Torvalds
2017-07-20  0:18     ` Randy Dunlap
2017-07-20  0:18     ` Linus Torvalds
2017-07-20  0:19     ` Randy Dunlap
2017-07-20  0:26       ` Linus Torvalds
2017-07-20  0:35         ` Linus Torvalds
2017-07-20  0:53           ` Linus Torvalds
2017-07-20  1:04             ` Randy Dunlap
2017-07-20  1:05     ` Joe Perches
2017-07-20  3:12       ` Joe Perches
2017-07-20  4:24       ` Linus Torvalds
2017-07-20  4:36         ` Theodore Ts'o
2017-07-20  4:53           ` Linus Torvalds
2017-07-20  4:43         ` Joe Perches
2017-07-20  4:57           ` Linus Torvalds
2017-07-20 10:54           ` Michael Ellerman [this message]
2017-07-20 16:48             ` Joe Perches
2017-07-20 18:03               ` Linus Torvalds
2017-07-20 22:50                 ` Joe Perches
2017-07-19 21:49 ` Joe Perches

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tw27gyzp.fsf@concordia.ellerman.id.au \
    --to=mpe@ellerman.id.au \
    --cc=akpm@linux-foundation.org \
    --cc=joe@perches.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®