diff mbox series

MAINTAINERS: Remove myself as EEH maintainer

Message ID 20220806085301.25142-1-ruscur@russell.cc (mailing list archive)
State Superseded
Headers show
Series MAINTAINERS: Remove myself as EEH maintainer | expand

Checks

Context Check Description
snowpatch_ozlabs/github-powerpc_kernel_qemu success Successfully ran 23 jobs.
snowpatch_ozlabs/github-powerpc_sparse success Successfully ran 4 jobs.
snowpatch_ozlabs/github-powerpc_clang success Successfully ran 6 jobs.

Commit Message

Russell Currey Aug. 6, 2022, 8:53 a.m. UTC
I haven't touched EEH in a long time I don't have much knowledge of the
subsystem at this point either, so it's misleading to have me as a
maintainer.

I remain grateful to Oliver for picking up my slack over the years.

Signed-off-by: Russell Currey <ruscur@russell.cc>
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)

Comments

Michael Ellerman Aug. 11, 2022, 6:22 a.m. UTC | #1
Russell Currey <ruscur@russell.cc> writes:
> I haven't touched EEH in a long time I don't have much knowledge of the
> subsystem at this point either, so it's misleading to have me as a
> maintainer.

Thank you for your service.

> I remain grateful to Oliver for picking up my slack over the years.

Ack.

But I wonder if he is still happy being listed as the only maintainer.
Given the status is "Supported" that means "Someone is actually paid to
look after this" - and I suspect Oracle are probably not paying him to
do that?

Should we change the status? Or just drop the entry entirely and fold it
into the top-level powerpc one?

cheers

> diff --git a/MAINTAINERS b/MAINTAINERS
> index a9f77648c107..dfe6081fa0b3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15639,7 +15639,6 @@ F:	drivers/pci/endpoint/
>  F:	tools/pci/
>  
>  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
> -M:	Russell Currey <ruscur@russell.cc>
>  M:	Oliver O'Halloran <oohall@gmail.com>
>  L:	linuxppc-dev@lists.ozlabs.org
>  S:	Supported
> -- 
> 2.37.1
Oliver O'Halloran Aug. 11, 2022, 7:19 a.m. UTC | #2
On Thu, Aug 11, 2022 at 4:22 PM Michael Ellerman <mpe@ellerman.id.au> wrote:
>
> Russell Currey <ruscur@russell.cc> writes:
> > I haven't touched EEH in a long time I don't have much knowledge of the
> > subsystem at this point either, so it's misleading to have me as a
> > maintainer.
>
> Thank you for your service.
>
> > I remain grateful to Oliver for picking up my slack over the years.
>
> Ack.
>
> But I wonder if he is still happy being listed as the only maintainer.
> Given the status is "Supported" that means "Someone is actually paid to
> look after this" - and I suspect Oracle are probably not paying him to
> do that?

I'm still happy to field questions and/or give reviews occasionally if
needed, but yeah I don't have the time, hardware, or inclination to do
any actual maintenance. IIRC Mahesh was supposed to take over
supporting EEH after I left IBM. If he's still around he should
probably be listed as a maintainer.
Mahesh J Salgaonkar Aug. 17, 2022, 6:27 a.m. UTC | #3
On 2022-08-11 17:19:47 Thu, Oliver O'Halloran wrote:
> On Thu, Aug 11, 2022 at 4:22 PM Michael Ellerman <mpe@ellerman.id.au> wrote:
> >
> > Russell Currey <ruscur@russell.cc> writes:
> > > I haven't touched EEH in a long time I don't have much knowledge of the
> > > subsystem at this point either, so it's misleading to have me as a
> > > maintainer.
> >
> > Thank you for your service.
> >
> > > I remain grateful to Oliver for picking up my slack over the years.
> >
> > Ack.
> >
> > But I wonder if he is still happy being listed as the only maintainer.
> > Given the status is "Supported" that means "Someone is actually paid to
> > look after this" - and I suspect Oracle are probably not paying him to
> > do that?
> 
> I'm still happy to field questions and/or give reviews occasionally if
> needed, but yeah I don't have the time, hardware, or inclination to do
> any actual maintenance. IIRC Mahesh was supposed to take over
> supporting EEH after I left IBM. If he's still around he should
> probably be listed as a maintainer.

Yes, I am still around. I am currently looking into the EEH and will be
glad to take over the maintenanership of EEH for powerpc. Please feel
free to add me as maintainer for EEH.

Thanks,
Bjorn Helgaas Aug. 19, 2022, 8:19 p.m. UTC | #4
On Sat, Aug 06, 2022 at 06:53:01PM +1000, Russell Currey wrote:
> I haven't touched EEH in a long time I don't have much knowledge of the
> subsystem at this point either, so it's misleading to have me as a
> maintainer.
> 
> I remain grateful to Oliver for picking up my slack over the years.
> 
> Signed-off-by: Russell Currey <ruscur@russell.cc>
> ---
>  MAINTAINERS | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a9f77648c107..dfe6081fa0b3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15639,7 +15639,6 @@ F:	drivers/pci/endpoint/
>  F:	tools/pci/
>  
>  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
> -M:	Russell Currey <ruscur@russell.cc>
>  M:	Oliver O'Halloran <oohall@gmail.com>
>  L:	linuxppc-dev@lists.ozlabs.org
>  S:	Supported

I was thinking along these lines, but if you want to take this,
Michael, I'll drop it:


commit 92ea781689d1 ("MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer")
Author: Russell Currey <ruscur@russell.cc>
Date:   Sat Aug 6 18:53:01 2022 +1000

    MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer
    
    Update EEH entry:
    
      - Russell: lacks time to maintain EEH.
    
      - Oliver: lacks time & hardware to do actual maintenance, but happy to
        field questions and review things.
    
      - Mahesh: glad to take over EEH maintenance.
    
    [bhelgaas: commit log, add Mahesh, make Oliver reviewer]
    Link: https://lore.kernel.org/r/20220806085301.25142-1-ruscur@russell.cc
    Signed-off-by: Russell Currey <ruscur@russell.cc>
    Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>

diff --git a/MAINTAINERS b/MAINTAINERS
index f60dfac7661c..51def5ac9462 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15696,8 +15696,8 @@ F:	drivers/pci/endpoint/
 F:	tools/pci/
 
 PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
-M:	Russell Currey <ruscur@russell.cc>
-M:	Oliver O'Halloran <oohall@gmail.com>
+M:	Mahesh J Salgaonkar <mahesh@linux.ibm.com>
+R:	Oliver O'Halloran <oohall@gmail.com>
 L:	linuxppc-dev@lists.ozlabs.org
 S:	Supported
 F:	Documentation/PCI/pci-error-recovery.rst
Michael Ellerman Aug. 20, 2022, 12:17 a.m. UTC | #5
Bjorn Helgaas <helgaas@kernel.org> writes:
> On Sat, Aug 06, 2022 at 06:53:01PM +1000, Russell Currey wrote:
>> I haven't touched EEH in a long time I don't have much knowledge of the
>> subsystem at this point either, so it's misleading to have me as a
>> maintainer.
>> 
>> I remain grateful to Oliver for picking up my slack over the years.
>> 
>> Signed-off-by: Russell Currey <ruscur@russell.cc>
>> ---
>>  MAINTAINERS | 1 -
>>  1 file changed, 1 deletion(-)
>> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index a9f77648c107..dfe6081fa0b3 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -15639,7 +15639,6 @@ F:	drivers/pci/endpoint/
>>  F:	tools/pci/
>>  
>>  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
>> -M:	Russell Currey <ruscur@russell.cc>
>>  M:	Oliver O'Halloran <oohall@gmail.com>
>>  L:	linuxppc-dev@lists.ozlabs.org
>>  S:	Supported
>
> I was thinking along these lines, but if you want to take this,
> Michael, I'll drop it:

Hi Bjorn,

I was hoping one of the protagonists would send a patch :), but that
looks perfect.

You may as well merge it given you've gone to the trouble to make a
proper patch out of the discussion.

Acked-by: Michael Ellerman <mpe@ellerman.id.au>

cheers

> commit 92ea781689d1 ("MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer")
> Author: Russell Currey <ruscur@russell.cc>
> Date:   Sat Aug 6 18:53:01 2022 +1000
>
>     MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer
>     
>     Update EEH entry:
>     
>       - Russell: lacks time to maintain EEH.
>     
>       - Oliver: lacks time & hardware to do actual maintenance, but happy to
>         field questions and review things.
>     
>       - Mahesh: glad to take over EEH maintenance.
>     
>     [bhelgaas: commit log, add Mahesh, make Oliver reviewer]
>     Link: https://lore.kernel.org/r/20220806085301.25142-1-ruscur@russell.cc
>     Signed-off-by: Russell Currey <ruscur@russell.cc>
>     Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f60dfac7661c..51def5ac9462 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -15696,8 +15696,8 @@ F:	drivers/pci/endpoint/
>  F:	tools/pci/
>  
>  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
> -M:	Russell Currey <ruscur@russell.cc>
> -M:	Oliver O'Halloran <oohall@gmail.com>
> +M:	Mahesh J Salgaonkar <mahesh@linux.ibm.com>
> +R:	Oliver O'Halloran <oohall@gmail.com>
>  L:	linuxppc-dev@lists.ozlabs.org
>  S:	Supported
>  F:	Documentation/PCI/pci-error-recovery.rst
Bjorn Helgaas Aug. 25, 2022, 8:03 p.m. UTC | #6
On Sat, Aug 20, 2022 at 10:17:41AM +1000, Michael Ellerman wrote:
> Bjorn Helgaas <helgaas@kernel.org> writes:
> > On Sat, Aug 06, 2022 at 06:53:01PM +1000, Russell Currey wrote:
> >> I haven't touched EEH in a long time I don't have much knowledge of the
> >> subsystem at this point either, so it's misleading to have me as a
> >> maintainer.
> >> 
> >> I remain grateful to Oliver for picking up my slack over the years.
> >> 
> >> Signed-off-by: Russell Currey <ruscur@russell.cc>
> >> ---
> >>  MAINTAINERS | 1 -
> >>  1 file changed, 1 deletion(-)
> >> 
> >> diff --git a/MAINTAINERS b/MAINTAINERS
> >> index a9f77648c107..dfe6081fa0b3 100644
> >> --- a/MAINTAINERS
> >> +++ b/MAINTAINERS
> >> @@ -15639,7 +15639,6 @@ F:	drivers/pci/endpoint/
> >>  F:	tools/pci/
> >>  
> >>  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
> >> -M:	Russell Currey <ruscur@russell.cc>
> >>  M:	Oliver O'Halloran <oohall@gmail.com>
> >>  L:	linuxppc-dev@lists.ozlabs.org
> >>  S:	Supported
> >
> > I was thinking along these lines, but if you want to take this,
> > Michael, I'll drop it:
> 
> Hi Bjorn,
> 
> I was hoping one of the protagonists would send a patch :), but that
> looks perfect.

Waiting for that patch would have been the *smart* thing to do, but I
added your ack and put it on for-linus for v6.0.  Thanks!

> Acked-by: Michael Ellerman <mpe@ellerman.id.au>
> 
> cheers
> 
> > commit 92ea781689d1 ("MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer")
> > Author: Russell Currey <ruscur@russell.cc>
> > Date:   Sat Aug 6 18:53:01 2022 +1000
> >
> >     MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer
> >     
> >     Update EEH entry:
> >     
> >       - Russell: lacks time to maintain EEH.
> >     
> >       - Oliver: lacks time & hardware to do actual maintenance, but happy to
> >         field questions and review things.
> >     
> >       - Mahesh: glad to take over EEH maintenance.
> >     
> >     [bhelgaas: commit log, add Mahesh, make Oliver reviewer]
> >     Link: https://lore.kernel.org/r/20220806085301.25142-1-ruscur@russell.cc
> >     Signed-off-by: Russell Currey <ruscur@russell.cc>
> >     Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index f60dfac7661c..51def5ac9462 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -15696,8 +15696,8 @@ F:	drivers/pci/endpoint/
> >  F:	tools/pci/
> >  
> >  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
> > -M:	Russell Currey <ruscur@russell.cc>
> > -M:	Oliver O'Halloran <oohall@gmail.com>
> > +M:	Mahesh J Salgaonkar <mahesh@linux.ibm.com>
> > +R:	Oliver O'Halloran <oohall@gmail.com>
> >  L:	linuxppc-dev@lists.ozlabs.org
> >  S:	Supported
> >  F:	Documentation/PCI/pci-error-recovery.rst
Michael Ellerman Aug. 26, 2022, 6:45 a.m. UTC | #7
Bjorn Helgaas <helgaas@kernel.org> writes:
> On Sat, Aug 20, 2022 at 10:17:41AM +1000, Michael Ellerman wrote:
>> Bjorn Helgaas <helgaas@kernel.org> writes:
>> > On Sat, Aug 06, 2022 at 06:53:01PM +1000, Russell Currey wrote:
>> >> I haven't touched EEH in a long time I don't have much knowledge of the
>> >> subsystem at this point either, so it's misleading to have me as a
>> >> maintainer.
>> >> 
>> >> I remain grateful to Oliver for picking up my slack over the years.
>> >> 
>> >> Signed-off-by: Russell Currey <ruscur@russell.cc>
>> >> ---
>> >>  MAINTAINERS | 1 -
>> >>  1 file changed, 1 deletion(-)
>> >> 
>> >> diff --git a/MAINTAINERS b/MAINTAINERS
>> >> index a9f77648c107..dfe6081fa0b3 100644
>> >> --- a/MAINTAINERS
>> >> +++ b/MAINTAINERS
>> >> @@ -15639,7 +15639,6 @@ F:	drivers/pci/endpoint/
>> >>  F:	tools/pci/
>> >>  
>> >>  PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
>> >> -M:	Russell Currey <ruscur@russell.cc>
>> >>  M:	Oliver O'Halloran <oohall@gmail.com>
>> >>  L:	linuxppc-dev@lists.ozlabs.org
>> >>  S:	Supported
>> >
>> > I was thinking along these lines, but if you want to take this,
>> > Michael, I'll drop it:
>> 
>> Hi Bjorn,
>> 
>> I was hoping one of the protagonists would send a patch :), but that
>> looks perfect.
>
> Waiting for that patch would have been the *smart* thing to do, but I
> added your ack and put it on for-linus for v6.0.  Thanks!

:) Thanks.

cheers
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index a9f77648c107..dfe6081fa0b3 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15639,7 +15639,6 @@  F:	drivers/pci/endpoint/
 F:	tools/pci/
 
 PCI ENHANCED ERROR HANDLING (EEH) FOR POWERPC
-M:	Russell Currey <ruscur@russell.cc>
 M:	Oliver O'Halloran <oohall@gmail.com>
 L:	linuxppc-dev@lists.ozlabs.org
 S:	Supported