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

[Wanpeng Li]

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