Re: [PATCH v2] MAINTAINERS: Reorganize KVM/x86 maintainership

From: Wanpeng Li
Date: Thu Jun 23 2022 - 22:11:51 EST


On Thu, 23 Jun 2022 at 22:32, Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote:
>
> For the last few years I have been the sole maintainer of KVM, albeit
> getting serious help from all the people who have reviewed hundreds of
> patches. The volume of KVM x86 alone has gotten to the point where one
> maintainer is not enough; especially if that maintainer is not doing it
> full time and if they want to keep up with the evolution of ARM64 and
> RISC-V at both the architecture and the hypervisor level.
>
> So, this patch is the first step in restoring double maintainership
> or even transitioning to the submaintainer model of other architectures.
>
> The changes here were mostly proposed by Sean offlist and they are twofold:
>
> - revisiting the set of KVM x86 reviewers. It's important to have an
> an accurate list of people that are actively reviewing patches ("R"),
> as well as people that are able to act on bug reports ("M"). Otherwise,
> voids to be filled are not easily visible. The proposal is to split
> KVM on Hyper-V, which is where Vitaly has been the main contributor
> for quite some time now; likewise for KVM paravirt support, which
> has been the main interest of Wanpeng and to which Vitaly has also
> contributed (e.g., for async page faults). Jim and Joerg have not been
> particularly active (though Joerg has worked on guest support for AMD
> SEV); knowing them a bit, I can't imagine they would object to their
> removal or even be surprised, but please speak up if you do.
>
> - promoting Sean to maintainer for KVM x86 host support. While for
> now this changes little, let's treat it as a harbinger for future
> changes. The plan is that I would keep the final integration testing
> for quite some time, and probably focus more on -rc work. This will
> give me more time to clean up my ad hoc setup and moving towards a
> more public CI, with Sean focusing instead on next-release patches,
> and the testing up to where kvm-unit-tests and selftests pass. In
> order to facilitate collaboration between Sean and myself, we'll
> also formalize a bit more the various branches of kvm.git.
>
> Nothing is going to change with respect to handling pull requests to Linus
> and from other architectures, as well as maintainance of the generic code
> (which I expect and hope to be more important as architectures try to
> share more code) and documentation. However, it's not a coincidence
> that my entry is now the last for x86, ready to be demoted to reviewer
> if/when the right time comes.
>
> Suggested-by: Sean Christopherson <seanjc@xxxxxxxxxx>
> Co-developed-by: Sean Christopherson <seanjc@xxxxxxxxxx>
> Cc: kvm@xxxxxxxxxxxxxxx
> Cc: Sean Christopherson <seanjc@xxxxxxxxxx>
> Cc: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx>
> Cc: Wanpeng Li <wanpengli@xxxxxxxxxxx>
> Cc: Jim Mattson <jmattson@xxxxxxxxxx>
> Cc: Joerg Roedel <joro@xxxxxxxxxx>
> Signed-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> ---
> MAINTAINERS | 40 +++++++++++++++++++++++++++++++---------
> 1 file changed, 31 insertions(+), 9 deletions(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 97014ae3e5ed..968b622bc3ce 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10897,28 +10897,50 @@ F: tools/testing/selftests/kvm/*/s390x/
> F: tools/testing/selftests/kvm/s390x/
>
> KERNEL VIRTUAL MACHINE FOR X86 (KVM/x86)
> +M: Sean Christopherson <seanjc@xxxxxxxxxx>
> M: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> -R: Sean Christopherson <seanjc@xxxxxxxxxx>
> -R: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx>
> -R: Wanpeng Li <wanpengli@xxxxxxxxxxx>
> -R: Jim Mattson <jmattson@xxxxxxxxxx>
> -R: Joerg Roedel <joro@xxxxxxxxxx>
> L: kvm@xxxxxxxxxxxxxxx
> S: Supported
> -W: http://www.linux-kvm.org
> T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
> F: arch/x86/include/asm/kvm*
> -F: arch/x86/include/asm/pvclock-abi.h
> F: arch/x86/include/asm/svm.h
> F: arch/x86/include/asm/vmx*.h
> F: arch/x86/include/uapi/asm/kvm*
> F: arch/x86/include/uapi/asm/svm.h
> F: arch/x86/include/uapi/asm/vmx.h
> -F: arch/x86/kernel/kvm.c
> -F: arch/x86/kernel/kvmclock.c
> F: arch/x86/kvm/
> F: arch/x86/kvm/*/
>
> +KVM PARAVIRT (KVM/paravirt)
> +M: Paolo Bonzini <pbonzini@xxxxxxxxxx>
> +R: Wanpeng Li <wanpengli@xxxxxxxxxxx>

I can probably volunteer as an "M" and act on bugs here since most of
the important features(except Async PF) contributed by me in recent
years . :)
Wanpeng