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

2022-08-26 Thread Michael Ellerman
Bjorn Helgaas  writes:
> On Sat, Aug 20, 2022 at 10:17:41AM +1000, Michael Ellerman wrote:
>> Bjorn Helgaas  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 
>> >> ---
>> >>  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 
>> >>  M:   Oliver O'Halloran 
>> >>  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


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

2022-08-25 Thread Bjorn Helgaas
On Sat, Aug 20, 2022 at 10:17:41AM +1000, Michael Ellerman wrote:
> Bjorn Helgaas  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 
> >> ---
> >>  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 
> >>  M:Oliver O'Halloran 
> >>  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 
> 
> cheers
> 
> > commit 92ea781689d1 ("MAINTAINERS: Add Mahesh J Salgaonkar as EEH 
> > maintainer")
> > Author: Russell Currey 
> > 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-rus...@russell.cc
> > Signed-off-by: Russell Currey 
> > Signed-off-by: Bjorn Helgaas 
> >
> > 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 
> > -M: Oliver O'Halloran 
> > +M: Mahesh J Salgaonkar 
> > +R: Oliver O'Halloran 
> >  L: linuxppc-dev@lists.ozlabs.org
> >  S: Supported
> >  F: Documentation/PCI/pci-error-recovery.rst


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

2022-08-19 Thread Michael Ellerman
Bjorn Helgaas  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 
>> ---
>>  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 
>>  M:  Oliver O'Halloran 
>>  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 

cheers

> commit 92ea781689d1 ("MAINTAINERS: Add Mahesh J Salgaonkar as EEH maintainer")
> Author: Russell Currey 
> 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-rus...@russell.cc
> Signed-off-by: Russell Currey 
> Signed-off-by: Bjorn Helgaas 
>
> 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 
> -M:   Oliver O'Halloran 
> +M:   Mahesh J Salgaonkar 
> +R:   Oliver O'Halloran 
>  L:   linuxppc-dev@lists.ozlabs.org
>  S:   Supported
>  F:   Documentation/PCI/pci-error-recovery.rst


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

2022-08-19 Thread 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 
> ---
>  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 
>  M:   Oliver O'Halloran 
>  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 
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-rus...@russell.cc
Signed-off-by: Russell Currey 
Signed-off-by: Bjorn Helgaas 

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 
-M: Oliver O'Halloran 
+M: Mahesh J Salgaonkar 
+R: Oliver O'Halloran 
 L: linuxppc-dev@lists.ozlabs.org
 S: Supported
 F: Documentation/PCI/pci-error-recovery.rst


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

2022-08-17 Thread 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  wrote:
> >
> > Russell Currey  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

2022-08-11 Thread Oliver O'Halloran
On Thu, Aug 11, 2022 at 4:22 PM Michael Ellerman  wrote:
>
> Russell Currey  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

2022-08-11 Thread Michael Ellerman
Russell Currey  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 
>  M:   Oliver O'Halloran 
>  L:   linuxppc-dev@lists.ozlabs.org
>  S:   Supported
> -- 
> 2.37.1


[PATCH] MAINTAINERS: Remove myself as EEH maintainer

2022-08-06 Thread 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 
---
 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 
 M: Oliver O'Halloran 
 L: linuxppc-dev@lists.ozlabs.org
 S: Supported
-- 
2.37.1