[PATCH] MAINTAINERS: Remove myself as EEH maintainer

[PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Russell Currey]

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
-- 
2.37.1

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Michael Ellerman]

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

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Oliver O'Halloran]

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.

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Mahesh J Salgaonkar]

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,

-- 
Mahesh J Salgaonkar

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Bjorn Helgaas]

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

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Michael Ellerman]

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

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Bjorn Helgaas]

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

Re: [PATCH] MAINTAINERS: Remove myself as EEH maintainer

[Michael Ellerman]

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