2020-08-21 15:50:22

by Steve Wahl

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers


Signed-off-by: Steve Wahl <[email protected]>
---
MAINTAINERS | 9 +++++++++
1 file changed, 9 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index deaafb617361..4c2143d8ae45 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -18874,6 +18874,15 @@ S: Maintained
T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/core
F: arch/x86/platform

+X86 PLATFORM UV HPE SUPERDOME FLEX
+M: Steve Wahl <[email protected]>
+R: Dimitri Sivanich <[email protected]>
+R: Russ Anderson <[email protected]>
+S: Supported
+F: arch/x86/include/asm/uv/
+F: arch/x86/kernel/apic/x2apic_uv_x.c
+F: arch/x86/platform/uv/
+
X86 VDSO
M: Andy Lutomirski <[email protected]>
L: [email protected]
--
2.12.3


2020-08-21 16:49:26

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers

+Joe Perches

On Fri, Aug 21, 2020 at 9:48 AM Steve Wahl <[email protected]> wrote:
>
>
> Signed-off-by: Steve Wahl <[email protected]>

get_maintainers.pl doesn't work on MAINTAINERS. You need to send this
to the maintainers of the files listed in the entry below. Looks like
that would be the x86 maintainers.


What did Mauro, David and I do to become MAINTAINERS maintainers?

Mauro Carvalho Chehab <[email protected]>
(commit_signer:127/806=16%,authored:80/806=10%)
Rob Herring <[email protected]> (commit_signer:103/806=13%)
"David S. Miller" <[email protected]> (commit_signer:99/806=12%)
[email protected] (open list)


Can we make --no-git-fallback the default? It's useful for
informational purposes, but never for who to email patches to. Having
no output would be better, then submitters have to think about where
to send patches.

What ever happened to splitting up MAINTAINERS to subdirectories? That
would help routing MAINTAINERS changes to the right maintainers.

Rob


> ---
> MAINTAINERS | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index deaafb617361..4c2143d8ae45 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -18874,6 +18874,15 @@ S: Maintained
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/core
> F: arch/x86/platform
>
> +X86 PLATFORM UV HPE SUPERDOME FLEX
> +M: Steve Wahl <[email protected]>
> +R: Dimitri Sivanich <[email protected]>
> +R: Russ Anderson <[email protected]>
> +S: Supported
> +F: arch/x86/include/asm/uv/
> +F: arch/x86/kernel/apic/x2apic_uv_x.c
> +F: arch/x86/platform/uv/
> +
> X86 VDSO
> M: Andy Lutomirski <[email protected]>
> L: [email protected]
> --
> 2.12.3
>

2020-08-21 17:07:17

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers

On Fri, 2020-08-21 at 10:45 -0600, Rob Herring wrote:
> +Joe Perches
>
> On Fri, Aug 21, 2020 at 9:48 AM Steve Wahl <[email protected]> wrote:
> >
> > Signed-off-by: Steve Wahl <[email protected]>
>
> get_maintainers.pl doesn't work on MAINTAINERS. You need to send this
> to the maintainers of the files listed in the entry below. Looks like
> that would be the x86 maintainers.
>
>
> What did Mauro, David and I do to become MAINTAINERS maintainers?
>
> Mauro Carvalho Chehab <[email protected]>
> (commit_signer:127/806=16%,authored:80/806=10%)
> Rob Herring <[email protected]> (commit_signer:103/806=13%)
> "David S. Miller" <[email protected]> (commit_signer:99/806=12%)
> [email protected] (open list)
>
>
> Can we make --no-git-fallback the default? It's useful for
> informational purposes, but never for who to email patches to. Having
> no output would be better, then submitters have to think about where
> to send patches.

Doubtful that improves things. At least the --git-fallback option
shows who modified or got patches accepted to files that are
nominally unmaintained. It also shows the upstream path for those
files via Signed-off-by: lines so I think --git-fallback is generally
a good mechanism and control flag for directly unmaintained files.

> What ever happened to splitting up MAINTAINERS to subdirectories? That
> would help routing MAINTAINERS changes to the right maintainers.

Splitting MAINTAINERS into subdirectories would do nothing
to route patches. It would just be convenience to reduce
the total number of changes to a single file.

Those large number of changes to the single MAINTAINERS file
very rarely have any conflicts either, so it wouldn't really
change the overall number of changes to MAINTAINERS entries
spread around the tree.

You are be welcome to try to split the file and get Linus to
accept it. I gave it a go. Try yourself.

https://lore.kernel.org/patchwork/patch/817857/


2020-08-21 17:41:19

by Joe Perches

[permalink] [raw]
Subject: [PATCH] get_maintainer: Exclude MAINTAINERS file(s) from --git-fallback and --git-blame

MAINTAINERS files generally have no specific maintainer but are updated
by individuals for subsystems all over the source tree.

Exclude MAINTAINERS file(s) from --git-fallback and --git-blame searches
so the unlucky individuals that update the files the most are not shown.

Signed-off-by: Joe Perches <[email protected]>
---
scripts/get_maintainer.pl | 1 +
1 file changed, 1 insertion(+)

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 484d2fbf5921..4d5196a11329 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -953,6 +953,7 @@ sub get_maintainers {
}

foreach my $file (@files) {
+ next if ($file =~ /MAINTAINERS$/);
if ($email &&
($email_git || ($email_git_fallback &&
!$exact_pattern_match_hash{$file}))) {



2020-08-21 18:08:41

by Joe Perches

[permalink] [raw]
Subject: [PATCH V2] get_maintainer: Exclude MAINTAINERS file(s) from --git-fallback

MAINTAINERS files generally have no specific maintainer but are updated
by individuals for subsystems all over the source tree.

Exclude MAINTAINERS file(s) from --git-fallback searches so the unlucky
individuals that update the files the most are not shown by default.

Signed-off-by: Joe Perches <[email protected]>
---

V2:

Remove --git and --git-blame from the exclusion.
Someone who _really_ wants that info should get it.

scripts/get_maintainer.pl | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 484d2fbf5921..7ea461dc208e 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -954,8 +954,10 @@ sub get_maintainers {

foreach my $file (@files) {
if ($email &&
- ($email_git || ($email_git_fallback &&
- !$exact_pattern_match_hash{$file}))) {
+ ($email_git ||
+ ($email_git_fallback &&
+ $file !~ /MAINTAINERS$/ &&
+ !$exact_pattern_match_hash{$file}))) {
vcs_file_signoffs($file);
}
if ($email && $email_git_blame) {


2020-08-21 18:21:35

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers

On Fri, Aug 21, 2020 at 11:05 AM Joe Perches <[email protected]> wrote:
>
> On Fri, 2020-08-21 at 10:45 -0600, Rob Herring wrote:
> > +Joe Perches
> >
> > On Fri, Aug 21, 2020 at 9:48 AM Steve Wahl <[email protected]> wrote:
> > >
> > > Signed-off-by: Steve Wahl <[email protected]>
> >
> > get_maintainers.pl doesn't work on MAINTAINERS. You need to send this
> > to the maintainers of the files listed in the entry below. Looks like
> > that would be the x86 maintainers.
> >
> >
> > What did Mauro, David and I do to become MAINTAINERS maintainers?
> >
> > Mauro Carvalho Chehab <[email protected]>
> > (commit_signer:127/806=16%,authored:80/806=10%)
> > Rob Herring <[email protected]> (commit_signer:103/806=13%)
> > "David S. Miller" <[email protected]> (commit_signer:99/806=12%)
> > [email protected] (open list)
> >
> >
> > Can we make --no-git-fallback the default? It's useful for
> > informational purposes, but never for who to email patches to. Having
> > no output would be better, then submitters have to think about where
> > to send patches.
>
> Doubtful that improves things. At least the --git-fallback option
> shows who modified or got patches accepted to files that are
> nominally unmaintained. It also shows the upstream path for those
> files via Signed-off-by: lines so I think --git-fallback is generally
> a good mechanism and control flag for directly unmaintained files.
>
> > What ever happened to splitting up MAINTAINERS to subdirectories? That
> > would help routing MAINTAINERS changes to the right maintainers.
>
> Splitting MAINTAINERS into subdirectories would do nothing
> to route patches. It would just be convenience to reduce
> the total number of changes to a single file.

In general no, but for MAINTAINERS changes it would. Let's say I add
an entry for Documentation/devicetree/foo-bar.txt. With a per
subsystem/path MAINTAINERS file in
Documentation/devicetree/MAINTAINERS, you'd add an entry there and run
get_maintainer.pl:

$ touch Documentation/devicetree/MAINTAINERS
$ scripts/get_maintainer.pl -f Documentation/devicetree/MAINTAINERS
Rob Herring <[email protected]> (maintainer:OPEN FIRMWARE AND
FLATTENED DEVICE TREE BINDINGS)
[email protected] (open list:OPEN FIRMWARE AND FLATTENED
DEVICE TREE BINDINGS)
[email protected] (open list)

> Those large number of changes to the single MAINTAINERS file
> very rarely have any conflicts either, so it wouldn't really
> change the overall number of changes to MAINTAINERS entries
> spread around the tree.
>
> You are be welcome to try to split the file and get Linus to
> accept it. I gave it a go. Try yourself.
>
> https://lore.kernel.org/patchwork/patch/817857/

Yes, I remember that. He didn't seem totally opposed to it which is why I asked.

I have another reason for wanting the split. I want to generate a
MAINTAINERS file from the DT schema files. We have the data there and
it's checked automatically. I don't care to either continually tell
folks to add a MAINTAINERS entry or tell them to run checkpatch.pl to
tell them that. But if the infrastructure got merged, would that
already work?

Rob

2020-08-21 18:36:53

by Joe Perches

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers

On Fri, 2020-08-21 at 12:17 -0600, Rob Herring wrote:
> I have another reason for wanting the split. I want to generate a
> MAINTAINERS file from the DT schema files. We have the data there and
> it's checked automatically. I don't care to either continually tell
> folks to add a MAINTAINERS entry or tell them to run checkpatch.pl to
> tell them that. But if the infrastructure got merged, would that
> already work?

It already gets addresses from yaml files.
The same could be done for dts/dtsi files.

Last month I proposed:

https://lore.kernel.org/linux-arm-kernel/[email protected]/


2020-08-24 14:46:59

by Steve Wahl

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers

Added Darren Hart, Andy Shevchenko, Thomas Gleixner, Ingo Molnar,
Borislav Petkov, and x86.kernel.org, after being told that
get_maintainers.pl doesn't work on MAINTAINERS.

Thanks,

Steve Wahl, HPE

On Fri, Aug 21, 2020 at 10:48:49AM -0500, Steve Wahl wrote:
>
> Signed-off-by: Steve Wahl <[email protected]>
> ---
> MAINTAINERS | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index deaafb617361..4c2143d8ae45 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -18874,6 +18874,15 @@ S: Maintained
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/core
> F: arch/x86/platform
>
> +X86 PLATFORM UV HPE SUPERDOME FLEX
> +M: Steve Wahl <[email protected]>
> +R: Dimitri Sivanich <[email protected]>
> +R: Russ Anderson <[email protected]>
> +S: Supported
> +F: arch/x86/include/asm/uv/
> +F: arch/x86/kernel/apic/x2apic_uv_x.c
> +F: arch/x86/platform/uv/
> +
> X86 VDSO
> M: Andy Lutomirski <[email protected]>
> L: [email protected]
> --
> 2.12.3
>

2020-08-24 15:39:26

by Borislav Petkov

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Add entry for HPE Superdome Flex (UV) maintainers

On Mon, Aug 24, 2020 at 09:45:23AM -0500, Steve Wahl wrote:
> Added Darren Hart, Andy Shevchenko, Thomas Gleixner, Ingo Molnar,
> Borislav Petkov, and x86.kernel.org, after being told that
> get_maintainers.pl doesn't work on MAINTAINERS.
>
> Thanks,
>
> Steve Wahl, HPE
>
> On Fri, Aug 21, 2020 at 10:48:49AM -0500, Steve Wahl wrote:

I could use a sentence here as a commit message, even if it repeats the
subject line.

> > Signed-off-by: Steve Wahl <[email protected]>
> > ---
> > MAINTAINERS | 9 +++++++++
> > 1 file changed, 9 insertions(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index deaafb617361..4c2143d8ae45 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -18874,6 +18874,15 @@ S: Maintained
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/core
> > F: arch/x86/platform
> >
> > +X86 PLATFORM UV HPE SUPERDOME FLEX
> > +M: Steve Wahl <[email protected]>
> > +R: Dimitri Sivanich <[email protected]>
> > +R: Russ Anderson <[email protected]>
> > +S: Supported
> > +F: arch/x86/include/asm/uv/
> > +F: arch/x86/kernel/apic/x2apic_uv_x.c
> > +F: arch/x86/platform/uv/
> > +
> > X86 VDSO
> > M: Andy Lutomirski <[email protected]>
> > L: [email protected]
> > --

Looks ok to me, otherwise.

--
Regards/Gruss,
Boris.

https://people.kernel.org/tglx/notes-about-netiquette