diff mbox series

KVM: PPC: Update MAINTAINERS

Message ID 20230608024504.58189-1-npiggin@gmail.com (mailing list archive)
State Accepted
Commit 7cc99ed87e4aeb3738e6ea7dc4d3ae28ad943601
Headers show
Series KVM: PPC: Update MAINTAINERS | expand

Checks

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

Commit Message

Nicholas Piggin June 8, 2023, 2:45 a.m. UTC
Michael is merging KVM PPC patches via the powerpc tree and KVM topic
branches. He doesn't necessarily have time to be across all of KVM so
is reluctant to call himself maintainer, but for the mechanics of how
patches flow upstream, it is maintained and does make sense to have
some contact people in MAINTAINERS.

So add Michael Ellerman as KVM PPC maintainer and myself as reviewer.
Split out the subarchs that don't get so much attention.

Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
---
 MAINTAINERS | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

Sean Christopherson June 28, 2023, 5:56 p.m. UTC | #1
On Thu, Jun 08, 2023, Nicholas Piggin wrote:
> Michael is merging KVM PPC patches via the powerpc tree and KVM topic
> branches. He doesn't necessarily have time to be across all of KVM so
> is reluctant to call himself maintainer, but for the mechanics of how
> patches flow upstream, it is maintained and does make sense to have
> some contact people in MAINTAINERS.
> 
> So add Michael Ellerman as KVM PPC maintainer and myself as reviewer.
> Split out the subarchs that don't get so much attention.
> 
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
> ---

Thanks for documenting the reality of things, much appreciated!

Acked-by: Sean Christopherson <seanjc@google.com>

>  MAINTAINERS | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 0dab9737ec16..44417acd2936 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -11379,7 +11379,13 @@ F:	arch/mips/include/uapi/asm/kvm*
>  F:	arch/mips/kvm/
>  
>  KERNEL VIRTUAL MACHINE FOR POWERPC (KVM/powerpc)
> +M:	Michael Ellerman <mpe@ellerman.id.au>
> +R:	Nicholas Piggin <npiggin@gmail.com>
>  L:	linuxppc-dev@lists.ozlabs.org
> +L:	kvm@vger.kernel.org
> +S:	Maintained (Book3S 64-bit HV)
> +S:	Odd fixes (Book3S 64-bit PR)
> +S:	Orphan (Book3E and 32-bit)

Do you think there's any chance of dropping support for everything except Book3S
64-bit HV at some point soonish?  There haven't been many generic KVM changes that
touch PPC, but in my experience when such series do come along, the many flavors
and layers of PPC incur quite a bit of development and testing cost, and have a
high chance of being broken compared to other architectures.
Michael Ellerman July 3, 2023, 5:26 a.m. UTC | #2
On Thu, 08 Jun 2023 12:45:04 +1000, Nicholas Piggin wrote:
> Michael is merging KVM PPC patches via the powerpc tree and KVM topic
> branches. He doesn't necessarily have time to be across all of KVM so
> is reluctant to call himself maintainer, but for the mechanics of how
> patches flow upstream, it is maintained and does make sense to have
> some contact people in MAINTAINERS.
> 
> So add Michael Ellerman as KVM PPC maintainer and myself as reviewer.
> Split out the subarchs that don't get so much attention.
> 
> [...]

Applied to powerpc/next.

[1/1] KVM: PPC: Update MAINTAINERS
      https://git.kernel.org/powerpc/c/7cc99ed87e4aeb3738e6ea7dc4d3ae28ad943601

cheers
Michael Ellerman July 5, 2023, 10:24 a.m. UTC | #3
Sean Christopherson <seanjc@google.com> writes:
> On Thu, Jun 08, 2023, Nicholas Piggin wrote:
>> Michael is merging KVM PPC patches via the powerpc tree and KVM topic
>> branches. He doesn't necessarily have time to be across all of KVM so
>> is reluctant to call himself maintainer, but for the mechanics of how
>> patches flow upstream, it is maintained and does make sense to have
>> some contact people in MAINTAINERS.
>> 
>> So add Michael Ellerman as KVM PPC maintainer and myself as reviewer.
>> Split out the subarchs that don't get so much attention.
>> 
>> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
>> ---
>
> Thanks for documenting the reality of things, much appreciated!
>
> Acked-by: Sean Christopherson <seanjc@google.com>
>
>>  MAINTAINERS | 6 ++++++
>>  1 file changed, 6 insertions(+)
>> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 0dab9737ec16..44417acd2936 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -11379,7 +11379,13 @@ F:	arch/mips/include/uapi/asm/kvm*
>>  F:	arch/mips/kvm/
>>  
>>  KERNEL VIRTUAL MACHINE FOR POWERPC (KVM/powerpc)
>> +M:	Michael Ellerman <mpe@ellerman.id.au>
>> +R:	Nicholas Piggin <npiggin@gmail.com>
>>  L:	linuxppc-dev@lists.ozlabs.org
>> +L:	kvm@vger.kernel.org
>> +S:	Maintained (Book3S 64-bit HV)
>> +S:	Odd fixes (Book3S 64-bit PR)
>> +S:	Orphan (Book3E and 32-bit)
>
> Do you think there's any chance of dropping support for everything except Book3S
> 64-bit HV at some point soonish?

Nick proposed disabling BookE KVM, which prompted some users to report
they are still actively using it:

  https://lore.kernel.org/all/20221128043623.1745708-1-npiggin@gmail.com/

There are also still some KVM PR users.

In total I'd guess it's only some small 100s of users, but we don't
really know.

> There haven't been many generic KVM changes that touch PPC, but in my
> experience when such series do come along, the many flavors and layers
> of PPC incur quite a bit of development and testing cost, and have a
> high chance of being broken compared to other architectures.

Ack.

cheers
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 0dab9737ec16..44417acd2936 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -11379,7 +11379,13 @@  F:	arch/mips/include/uapi/asm/kvm*
 F:	arch/mips/kvm/
 
 KERNEL VIRTUAL MACHINE FOR POWERPC (KVM/powerpc)
+M:	Michael Ellerman <mpe@ellerman.id.au>
+R:	Nicholas Piggin <npiggin@gmail.com>
 L:	linuxppc-dev@lists.ozlabs.org
+L:	kvm@vger.kernel.org
+S:	Maintained (Book3S 64-bit HV)
+S:	Odd fixes (Book3S 64-bit PR)
+S:	Orphan (Book3E and 32-bit)
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git topic/ppc-kvm
 F:	arch/powerpc/include/asm/kvm*
 F:	arch/powerpc/include/uapi/asm/kvm*