linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: add documentation to PSI section
@ 2023-07-28 12:09 Ramazan Safiullin
  2023-07-31 23:10 ` Suren Baghdasaryan
  0 siblings, 1 reply; 9+ messages in thread
From: Ramazan Safiullin @ 2023-07-28 12:09 UTC (permalink / raw)
  To: Lukas Bulwahn, Johannes Weiner, Suren Baghdasaryan, linux-kernel
  Cc: Ramazan Safiullin

While creating a patch for the psi documentation I noticed that
MAINTENERS was missing an entry in psi.

Add entry for psi documentation.

Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index a5c16bb92fe2..2da6643b9488 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17103,6 +17103,7 @@ PRESSURE STALL INFORMATION (PSI)
 M:	Johannes Weiner <hannes@cmpxchg.org>
 M:	Suren Baghdasaryan <surenb@google.com>
 S:	Maintained
+F:	Documentation/accounting/psi.rst
 F:	include/linux/psi*
 F:	kernel/sched/psi.c
 
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-07-28 12:09 [PATCH] MAINTAINERS: add documentation to PSI section Ramazan Safiullin
@ 2023-07-31 23:10 ` Suren Baghdasaryan
  2023-07-31 23:12   ` Suren Baghdasaryan
  0 siblings, 1 reply; 9+ messages in thread
From: Suren Baghdasaryan @ 2023-07-31 23:10 UTC (permalink / raw)
  To: Ramazan Safiullin; +Cc: Lukas Bulwahn, Johannes Weiner, linux-kernel

On Fri, Jul 28, 2023 at 5:09 AM Ramazan Safiullin
<ram.safiullin2001@gmail.com> wrote:
>
> While creating a patch for the psi documentation I noticed that
> MAINTENERS was missing an entry in psi.
>
> Add entry for psi documentation.
>
> Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>

Acked-by: Suren Baghdasaryan <surenb@google.com>

> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a5c16bb92fe2..2da6643b9488 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17103,6 +17103,7 @@ PRESSURE STALL INFORMATION (PSI)
>  M:     Johannes Weiner <hannes@cmpxchg.org>
>  M:     Suren Baghdasaryan <surenb@google.com>
>  S:     Maintained
> +F:     Documentation/accounting/psi.rst
>  F:     include/linux/psi*
>  F:     kernel/sched/psi.c
>
> --
> 2.25.1
>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-07-31 23:10 ` Suren Baghdasaryan
@ 2023-07-31 23:12   ` Suren Baghdasaryan
  2023-08-01 10:39     ` Peter Zijlstra
  0 siblings, 1 reply; 9+ messages in thread
From: Suren Baghdasaryan @ 2023-07-31 23:12 UTC (permalink / raw)
  To: Ramazan Safiullin, Peter Zijlstra
  Cc: Lukas Bulwahn, Johannes Weiner, linux-kernel

On Mon, Jul 31, 2023 at 4:10 PM Suren Baghdasaryan <surenb@google.com> wrote:
>
> On Fri, Jul 28, 2023 at 5:09 AM Ramazan Safiullin
> <ram.safiullin2001@gmail.com> wrote:
> >
> > While creating a patch for the psi documentation I noticed that
> > MAINTENERS was missing an entry in psi.
> >
> > Add entry for psi documentation.
> >
> > Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>
>
> Acked-by: Suren Baghdasaryan <surenb@google.com>

CC'ing Peter to pick up into his tree.

>
> > ---
> >  MAINTAINERS | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index a5c16bb92fe2..2da6643b9488 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -17103,6 +17103,7 @@ PRESSURE STALL INFORMATION (PSI)
> >  M:     Johannes Weiner <hannes@cmpxchg.org>
> >  M:     Suren Baghdasaryan <surenb@google.com>
> >  S:     Maintained
> > +F:     Documentation/accounting/psi.rst
> >  F:     include/linux/psi*
> >  F:     kernel/sched/psi.c
> >
> > --
> > 2.25.1
> >

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-07-31 23:12   ` Suren Baghdasaryan
@ 2023-08-01 10:39     ` Peter Zijlstra
  2023-08-01 11:07       ` Lukas Bulwahn
  0 siblings, 1 reply; 9+ messages in thread
From: Peter Zijlstra @ 2023-08-01 10:39 UTC (permalink / raw)
  To: Suren Baghdasaryan
  Cc: Ramazan Safiullin, Lukas Bulwahn, Johannes Weiner, linux-kernel

On Mon, Jul 31, 2023 at 04:12:00PM -0700, Suren Baghdasaryan wrote:
> On Mon, Jul 31, 2023 at 4:10 PM Suren Baghdasaryan <surenb@google.com> wrote:
> >
> > On Fri, Jul 28, 2023 at 5:09 AM Ramazan Safiullin
> > <ram.safiullin2001@gmail.com> wrote:
> > >
> > > While creating a patch for the psi documentation I noticed that
> > > MAINTENERS was missing an entry in psi.
> > >
> > > Add entry for psi documentation.
> > >
> > > Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>
> >
> > Acked-by: Suren Baghdasaryan <surenb@google.com>
> 
> CC'ing Peter to pick up into his tree.

Well, I'd need the actual patch email, otherwise I can't apply. Either
bounce the thread or have it be resent.

> >
> > > ---
> > >  MAINTAINERS | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index a5c16bb92fe2..2da6643b9488 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -17103,6 +17103,7 @@ PRESSURE STALL INFORMATION (PSI)
> > >  M:     Johannes Weiner <hannes@cmpxchg.org>
> > >  M:     Suren Baghdasaryan <surenb@google.com>
> > >  S:     Maintained
> > > +F:     Documentation/accounting/psi.rst
> > >  F:     include/linux/psi*
> > >  F:     kernel/sched/psi.c
> > >
> > > --
> > > 2.25.1
> > >

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-08-01 10:39     ` Peter Zijlstra
@ 2023-08-01 11:07       ` Lukas Bulwahn
  2023-08-01 13:03         ` Peter Zijlstra
  0 siblings, 1 reply; 9+ messages in thread
From: Lukas Bulwahn @ 2023-08-01 11:07 UTC (permalink / raw)
  To: Peter Zijlstra
  Cc: Suren Baghdasaryan, Ramazan Safiullin, Johannes Weiner, linux-kernel

On Tue, Aug 1, 2023 at 12:39 PM Peter Zijlstra <peterz@infradead.org> wrote:
>
> On Mon, Jul 31, 2023 at 04:12:00PM -0700, Suren Baghdasaryan wrote:
> > On Mon, Jul 31, 2023 at 4:10 PM Suren Baghdasaryan <surenb@google.com> wrote:
> > >
> > > On Fri, Jul 28, 2023 at 5:09 AM Ramazan Safiullin
> > > <ram.safiullin2001@gmail.com> wrote:
> > > >
> > > > While creating a patch for the psi documentation I noticed that
> > > > MAINTENERS was missing an entry in psi.
> > > >
> > > > Add entry for psi documentation.
> > > >
> > > > Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>
> > >
> > > Acked-by: Suren Baghdasaryan <surenb@google.com>
> >
> > CC'ing Peter to pick up into his tree.
>
> Well, I'd need the actual patch email, otherwise I can't apply. Either
> bounce the thread or have it be resent.
>

Speaking on behalf of Ramazan, who I am working with:

Suren, Peter, should we actually better just add you, Peter, as a
reviewer to PSI in MAINTAINERS, so that you get all the patches to psi
stuff in your mailbox and you just need to wait for an Ack from Suren
or Johannes?

Then, we could fix this up in the PSI section and just send out a v2
with the current Ack from Suren and Peter in CC.

And we will fix the typo in the commit message above:
s/MAINTENERS/MAINTAINERS/

Lukas

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-08-01 11:07       ` Lukas Bulwahn
@ 2023-08-01 13:03         ` Peter Zijlstra
  2023-08-01 13:32           ` Johannes Weiner
  0 siblings, 1 reply; 9+ messages in thread
From: Peter Zijlstra @ 2023-08-01 13:03 UTC (permalink / raw)
  To: Lukas Bulwahn
  Cc: Suren Baghdasaryan, Ramazan Safiullin, Johannes Weiner, linux-kernel

On Tue, Aug 01, 2023 at 01:07:09PM +0200, Lukas Bulwahn wrote:
> On Tue, Aug 1, 2023 at 12:39 PM Peter Zijlstra <peterz@infradead.org> wrote:
> >
> > On Mon, Jul 31, 2023 at 04:12:00PM -0700, Suren Baghdasaryan wrote:
> > > On Mon, Jul 31, 2023 at 4:10 PM Suren Baghdasaryan <surenb@google.com> wrote:
> > > >
> > > > On Fri, Jul 28, 2023 at 5:09 AM Ramazan Safiullin
> > > > <ram.safiullin2001@gmail.com> wrote:
> > > > >
> > > > > While creating a patch for the psi documentation I noticed that
> > > > > MAINTENERS was missing an entry in psi.
> > > > >
> > > > > Add entry for psi documentation.
> > > > >
> > > > > Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>
> > > >
> > > > Acked-by: Suren Baghdasaryan <surenb@google.com>
> > >
> > > CC'ing Peter to pick up into his tree.
> >
> > Well, I'd need the actual patch email, otherwise I can't apply. Either
> > bounce the thread or have it be resent.
> >
> 
> Speaking on behalf of Ramazan, who I am working with:
> 
> Suren, Peter, should we actually better just add you, Peter, as a
> reviewer to PSI in MAINTAINERS, so that you get all the patches to psi
> stuff in your mailbox and you just need to wait for an Ack from Suren
> or Johannes?

Well, mostly psi patches touch kernel/sched/psi.c and then I should get
it anyway (throught the kernel/sched/ glob). But sure, one more copy of
my email address in that file isn't going to hurt..

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-08-01 13:03         ` Peter Zijlstra
@ 2023-08-01 13:32           ` Johannes Weiner
  2023-08-01 15:17             ` Peter Zijlstra
  2023-08-09 19:34             ` [tip: sched/core] MAINTAINERS: Add Peter explicitly to the psi section tip-bot2 for Johannes Weiner
  0 siblings, 2 replies; 9+ messages in thread
From: Johannes Weiner @ 2023-08-01 13:32 UTC (permalink / raw)
  To: Peter Zijlstra
  Cc: Lukas Bulwahn, Suren Baghdasaryan, Ramazan Safiullin, linux-kernel

On Tue, Aug 01, 2023 at 03:03:01PM +0200, Peter Zijlstra wrote:
> On Tue, Aug 01, 2023 at 01:07:09PM +0200, Lukas Bulwahn wrote:
> > On Tue, Aug 1, 2023 at 12:39 PM Peter Zijlstra <peterz@infradead.org> wrote:
> > >
> > > On Mon, Jul 31, 2023 at 04:12:00PM -0700, Suren Baghdasaryan wrote:
> > > > On Mon, Jul 31, 2023 at 4:10 PM Suren Baghdasaryan <surenb@google.com> wrote:
> > > > >
> > > > > On Fri, Jul 28, 2023 at 5:09 AM Ramazan Safiullin
> > > > > <ram.safiullin2001@gmail.com> wrote:
> > > > > >
> > > > > > While creating a patch for the psi documentation I noticed that
> > > > > > MAINTENERS was missing an entry in psi.
> > > > > >
> > > > > > Add entry for psi documentation.
> > > > > >
> > > > > > Signed-off-by: Ramazan Safiullin <ram.safiullin2001@gmail.com>
> > > > >
> > > > > Acked-by: Suren Baghdasaryan <surenb@google.com>
> > > >
> > > > CC'ing Peter to pick up into his tree.
> > >
> > > Well, I'd need the actual patch email, otherwise I can't apply. Either
> > > bounce the thread or have it be resent.
> > >
> > 
> > Speaking on behalf of Ramazan, who I am working with:
> > 
> > Suren, Peter, should we actually better just add you, Peter, as a
> > reviewer to PSI in MAINTAINERS, so that you get all the patches to psi
> > stuff in your mailbox and you just need to wait for an Ack from Suren
> > or Johannes?
> 
> Well, mostly psi patches touch kernel/sched/psi.c and then I should get
> it anyway (throught the kernel/sched/ glob). But sure, one more copy of
> my email address in that file isn't going to hurt..

From f00cd5986718e7ff435444523305d480b54e34e8 Mon Sep 17 00:00:00 2001
From: Johannes Weiner <hannes@cmpxchg.org>
Date: Tue, 1 Aug 2023 09:18:21 -0400
Subject: [PATCH] MAINTAINERS: add Peter explicitly to the psi section

Peter is kind enough to route the low-volume psi patches through the
scheduler tree, but he is frequently not CC'd on them.

While he is matched through the SCHEDULER maintainers and reviewers on
kern/sched/*, that list is long, and mostly not applicable to psi
code. Thus, patch submitters often just CC the explicit PSI entries.

Add him to that section, to make sure he gets those patches.

Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 3be1bdfe8ecc..b68f3ab368c1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17037,6 +17037,7 @@ F:	drivers/net/ppp/pptp.c
 PRESSURE STALL INFORMATION (PSI)
 M:	Johannes Weiner <hannes@cmpxchg.org>
 M:	Suren Baghdasaryan <surenb@google.com>
+R:	Peter Ziljstra <peterz@infradead.org>
 S:	Maintained
 F:	include/linux/psi*
 F:	kernel/sched/psi.c
-- 
2.41.0

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: [PATCH] MAINTAINERS: add documentation to PSI section
  2023-08-01 13:32           ` Johannes Weiner
@ 2023-08-01 15:17             ` Peter Zijlstra
  2023-08-09 19:34             ` [tip: sched/core] MAINTAINERS: Add Peter explicitly to the psi section tip-bot2 for Johannes Weiner
  1 sibling, 0 replies; 9+ messages in thread
From: Peter Zijlstra @ 2023-08-01 15:17 UTC (permalink / raw)
  To: Johannes Weiner
  Cc: Lukas Bulwahn, Suren Baghdasaryan, Ramazan Safiullin, linux-kernel

On Tue, Aug 01, 2023 at 09:32:35AM -0400, Johannes Weiner wrote:
> From f00cd5986718e7ff435444523305d480b54e34e8 Mon Sep 17 00:00:00 2001
> From: Johannes Weiner <hannes@cmpxchg.org>
> Date: Tue, 1 Aug 2023 09:18:21 -0400
> Subject: [PATCH] MAINTAINERS: add Peter explicitly to the psi section
> 
> Peter is kind enough to route the low-volume psi patches through the
> scheduler tree, but he is frequently not CC'd on them.
> 
> While he is matched through the SCHEDULER maintainers and reviewers on
> kern/sched/*, that list is long, and mostly not applicable to psi
> code. Thus, patch submitters often just CC the explicit PSI entries.
> 
> Add him to that section, to make sure he gets those patches.
> 
> Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>

Thanks!

> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3be1bdfe8ecc..b68f3ab368c1 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -17037,6 +17037,7 @@ F:	drivers/net/ppp/pptp.c
>  PRESSURE STALL INFORMATION (PSI)
>  M:	Johannes Weiner <hannes@cmpxchg.org>
>  M:	Suren Baghdasaryan <surenb@google.com>
> +R:	Peter Ziljstra <peterz@infradead.org>
>  S:	Maintained
>  F:	include/linux/psi*
>  F:	kernel/sched/psi.c
> -- 
> 2.41.0

^ permalink raw reply	[flat|nested] 9+ messages in thread

* [tip: sched/core] MAINTAINERS: Add Peter explicitly to the psi section
  2023-08-01 13:32           ` Johannes Weiner
  2023-08-01 15:17             ` Peter Zijlstra
@ 2023-08-09 19:34             ` tip-bot2 for Johannes Weiner
  1 sibling, 0 replies; 9+ messages in thread
From: tip-bot2 for Johannes Weiner @ 2023-08-09 19:34 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: Johannes Weiner, Peter Zijlstra (Intel), x86, linux-kernel

The following commit has been merged into the sched/core branch of tip:

Commit-ID:     113d0a6b3954b57907d1a6e3209f4174f504e0ae
Gitweb:        https://git.kernel.org/tip/113d0a6b3954b57907d1a6e3209f4174f504e0ae
Author:        Johannes Weiner <hannes@cmpxchg.org>
AuthorDate:    Tue, 01 Aug 2023 09:18:21 -04:00
Committer:     Peter Zijlstra <peterz@infradead.org>
CommitterDate: Wed, 02 Aug 2023 16:19:25 +02:00

MAINTAINERS: Add Peter explicitly to the psi section

Peter is kind enough to route the low-volume psi patches through the
scheduler tree, but he is frequently not CC'd on them.

While he is matched through the SCHEDULER maintainers and reviewers on
kern/sched/*, that list is long, and mostly not applicable to psi
code. Thus, patch submitters often just CC the explicit PSI entries.

Add him to that section, to make sure he gets those patches.

Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20230801133235.GA1766885@cmpxchg.org
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index aee3406..f017dc6 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17047,6 +17047,7 @@ F:	drivers/net/ppp/pptp.c
 PRESSURE STALL INFORMATION (PSI)
 M:	Johannes Weiner <hannes@cmpxchg.org>
 M:	Suren Baghdasaryan <surenb@google.com>
+R:	Peter Ziljstra <peterz@infradead.org>
 S:	Maintained
 F:	include/linux/psi*
 F:	kernel/sched/psi.c

^ permalink raw reply related	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2023-08-09 19:35 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-28 12:09 [PATCH] MAINTAINERS: add documentation to PSI section Ramazan Safiullin
2023-07-31 23:10 ` Suren Baghdasaryan
2023-07-31 23:12   ` Suren Baghdasaryan
2023-08-01 10:39     ` Peter Zijlstra
2023-08-01 11:07       ` Lukas Bulwahn
2023-08-01 13:03         ` Peter Zijlstra
2023-08-01 13:32           ` Johannes Weiner
2023-08-01 15:17             ` Peter Zijlstra
2023-08-09 19:34             ` [tip: sched/core] MAINTAINERS: Add Peter explicitly to the psi section tip-bot2 for Johannes Weiner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).