2021-05-11 13:19:12

by Nikolay Borisov

Subject: [PATCH] MAINTAINERS: Add lib/percpu* as part of percpu entry

Without this patch get_maintainers.pl on a patch which modified
lib/percpu_refcount.c produces:

Jens Axboe <[email protected]> (commit_signer:2/5=40%)
Ming Lei <[email protected]> (commit_signer:2/5=40%,authored:2/5=40%,added_lines:99/114=87%,removed_lines:34/43=79%)
"Paul E. McKenney" <[email protected]> (commit_signer:1/5=20%,authored:1/5=20%,added_lines:9/114=8%,removed_lines:3/43=7%)
Tejun Heo <[email protected]> (commit_signer:1/5=20%)
Andrew Morton <[email protected]> (commit_signer:1/5=20%)
Nikolay Borisov <[email protected]> (authored:1/5=20%,removed_lines:3/43=7%)
Joe Perches <[email protected]> (authored:1/5=20%,removed_lines:3/43=7%)
[email protected] (open list)

Whereas with the patch applied it now (properly) prints:

Dennis Zhou <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
Tejun Heo <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
Christoph Lameter <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
[email protected] (open list)

Signed-off-by: Nikolay Borisov <[email protected]>
---
MAINTAINERS | 1 +
1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index d92f85ca831d..b18fed606ddd 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -14004,6 +14004,7 @@ S: Maintained
T: git git://git.kernel.org/pub/scm/linux/kernel/git/dennis/percpu.git
F: arch/*/include/asm/percpu.h
F: include/linux/percpu*.h
+F: lib/percpu*.c
F: mm/percpu*.c

PER-TASK DELAY ACCOUNTING
--
2.25.1

2021-05-11 13:52:34

by Dennis Zhou

Subject: Re: [PATCH] MAINTAINERS: Add lib/percpu* as part of percpu entry

Hello,

On Tue, May 11, 2021 at 04:17:37PM +0300, Nikolay Borisov wrote:
> Without this patch get_maintainers.pl on a patch which modified
> lib/percpu_refcount.c produces:
>
> Jens Axboe <[email protected]> (commit_signer:2/5=40%)
> Ming Lei <[email protected]> (commit_signer:2/5=40%,authored:2/5=40%,added_lines:99/114=87%,removed_lines:34/43=79%)
> "Paul E. McKenney" <[email protected]> (commit_signer:1/5=20%,authored:1/5=20%,added_lines:9/114=8%,removed_lines:3/43=7%)
> Tejun Heo <[email protected]> (commit_signer:1/5=20%)
> Andrew Morton <[email protected]> (commit_signer:1/5=20%)
> Nikolay Borisov <[email protected]> (authored:1/5=20%,removed_lines:3/43=7%)
> Joe Perches <[email protected]> (authored:1/5=20%,removed_lines:3/43=7%)
> [email protected] (open list)
>
> Whereas with the patch applied it now (properly) prints:
>
> Dennis Zhou <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
> Tejun Heo <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
> Christoph Lameter <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
> [email protected] (open list)
>
> Signed-off-by: Nikolay Borisov <[email protected]>
> ---
> MAINTAINERS | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index d92f85ca831d..b18fed606ddd 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14004,6 +14004,7 @@ S: Maintained
> T: git git://git.kernel.org/pub/scm/linux/kernel/git/dennis/percpu.git
> F: arch/*/include/asm/percpu.h
> F: include/linux/percpu*.h
> +F: lib/percpu*.c
> F: mm/percpu*.c
>
> PER-TASK DELAY ACCOUNTING
> --
> 2.25.1
>

Yeah, in the past I've taken percpu_ref stuff, so I think this makes
sense. If no one has any objections I'll pick this up and your other
patch.

Thanks,
Dennis

2021-05-13 04:50:05

by Dennis Zhou

Subject: Re: [PATCH] MAINTAINERS: Add lib/percpu* as part of percpu entry

On Tue, May 11, 2021 at 01:48:36PM +0000, Dennis Zhou wrote:
> Hello,
>
> On Tue, May 11, 2021 at 04:17:37PM +0300, Nikolay Borisov wrote:
> > Without this patch get_maintainers.pl on a patch which modified
> > lib/percpu_refcount.c produces:
> >
> > Jens Axboe <[email protected]> (commit_signer:2/5=40%)
> > Ming Lei <[email protected]> (commit_signer:2/5=40%,authored:2/5=40%,added_lines:99/114=87%,removed_lines:34/43=79%)
> > "Paul E. McKenney" <[email protected]> (commit_signer:1/5=20%,authored:1/5=20%,added_lines:9/114=8%,removed_lines:3/43=7%)
> > Tejun Heo <[email protected]> (commit_signer:1/5=20%)
> > Andrew Morton <[email protected]> (commit_signer:1/5=20%)
> > Nikolay Borisov <[email protected]> (authored:1/5=20%,removed_lines:3/43=7%)
> > Joe Perches <[email protected]> (authored:1/5=20%,removed_lines:3/43=7%)
> > [email protected] (open list)
> >
> > Whereas with the patch applied it now (properly) prints:
> >
> > Dennis Zhou <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
> > Tejun Heo <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
> > Christoph Lameter <[email protected]> (maintainer:PER-CPU MEMORY ALLOCATOR)
> > [email protected] (open list)
> >
> > Signed-off-by: Nikolay Borisov <[email protected]>
> > ---
> > MAINTAINERS | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index d92f85ca831d..b18fed606ddd 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -14004,6 +14004,7 @@ S: Maintained
> > T: git git://git.kernel.org/pub/scm/linux/kernel/git/dennis/percpu.git
> > F: arch/*/include/asm/percpu.h
> > F: include/linux/percpu*.h
> > +F: lib/percpu*.c
> > F: mm/percpu*.c
> >
> > PER-TASK DELAY ACCOUNTING
> > --
> > 2.25.1
> >
>
> Yeah, in the past I've taken percpu_ref stuff, so I think this makes
> sense. If no one has any objections I'll pick this up and your other
> patch.
>
> Thanks,
> Dennis

I've applied this to for-5.13-fixes. I've also updated the mailing list
to linux-mm.

Thanks,
Dennis