2022-06-23 14:33:22

by Paolo Bonzini

[permalink] [raw]
Subject: [PATCH v2] MAINTAINERS: Reorganize KVM/x86 maintainership

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 <[email protected]>
Co-developed-by: Sean Christopherson <[email protected]>
Cc: [email protected]
Cc: Sean Christopherson <[email protected]>
Cc: Vitaly Kuznetsov <[email protected]>
Cc: Wanpeng Li <[email protected]>
Cc: Jim Mattson <[email protected]>
Cc: Joerg Roedel <[email protected]>
Signed-off-by: Paolo Bonzini <[email protected]>
---
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 <[email protected]>
M: Paolo Bonzini <[email protected]>
-R: Sean Christopherson <[email protected]>
-R: Vitaly Kuznetsov <[email protected]>
-R: Wanpeng Li <[email protected]>
-R: Jim Mattson <[email protected]>
-R: Joerg Roedel <[email protected]>
L: [email protected]
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 <[email protected]>
+R: Wanpeng Li <[email protected]>
+R: Vitaly Kuznetsov <[email protected]>
+L: [email protected]
+S: Supported
+T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
+F: arch/x86/kernel/kvm.c
+F: arch/x86/kernel/kvmclock.c
+F: arch/x86/include/asm/pvclock-abi.h
+F: include/linux/kvm_para.h
+F: include/uapi/linux/kvm_para.h
+F: include/uapi/asm-generic/kvm_para.h
+F: include/asm-generic/kvm_para.h
+F: arch/um/include/asm/kvm_para.h
+F: arch/x86/include/asm/kvm_para.h
+F: arch/x86/include/uapi/asm/kvm_para.h
+
+KVM X86 HYPER-V (KVM/hyper-v)
+M: Vitaly Kuznetsov <[email protected]>
+M: Sean Christopherson <[email protected]>
+M: Paolo Bonzini <[email protected]>
+L: [email protected]
+S: Supported
+T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
+F: arch/x86/kvm/hyperv.*
+F: arch/x86/kvm/kvm_onhyperv.*
+F: arch/x86/kvm/svm/svm_onhyperv.*
+F: arch/x86/kvm/vmx/evmcs.*
+
KERNFS
M: Greg Kroah-Hartman <[email protected]>
M: Tejun Heo <[email protected]>
--
2.31.1


2022-06-23 15:15:38

by Vitaly Kuznetsov

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

Paolo Bonzini <[email protected]> writes:

> 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 <[email protected]>
> Co-developed-by: Sean Christopherson <[email protected]>
> Cc: [email protected]
> Cc: Sean Christopherson <[email protected]>
> Cc: Vitaly Kuznetsov <[email protected]>
> Cc: Wanpeng Li <[email protected]>
> Cc: Jim Mattson <[email protected]>
> Cc: Joerg Roedel <[email protected]>
> Signed-off-by: Paolo Bonzini <[email protected]>
> ---
> 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 <[email protected]>
> M: Paolo Bonzini <[email protected]>
> -R: Sean Christopherson <[email protected]>
> -R: Vitaly Kuznetsov <[email protected]>
> -R: Wanpeng Li <[email protected]>
> -R: Jim Mattson <[email protected]>
> -R: Joerg Roedel <[email protected]>
> L: [email protected]
> 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 <[email protected]>
> +R: Wanpeng Li <[email protected]>
> +R: Vitaly Kuznetsov <[email protected]>
> +L: [email protected]
> +S: Supported
> +T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
> +F: arch/x86/kernel/kvm.c
> +F: arch/x86/kernel/kvmclock.c
> +F: arch/x86/include/asm/pvclock-abi.h
> +F: include/linux/kvm_para.h
> +F: include/uapi/linux/kvm_para.h
> +F: include/uapi/asm-generic/kvm_para.h
> +F: include/asm-generic/kvm_para.h
> +F: arch/um/include/asm/kvm_para.h
> +F: arch/x86/include/asm/kvm_para.h
> +F: arch/x86/include/uapi/asm/kvm_para.h
> +
> +KVM X86 HYPER-V (KVM/hyper-v)
> +M: Vitaly Kuznetsov <[email protected]>
> +M: Sean Christopherson <[email protected]>
> +M: Paolo Bonzini <[email protected]>
> +L: [email protected]
> +S: Supported
> +T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
> +F: arch/x86/kvm/hyperv.*
> +F: arch/x86/kvm/kvm_onhyperv.*
> +F: arch/x86/kvm/svm/svm_onhyperv.*

"arch/x86/kvm/svm/hyperv.*" is still missing) LGTM otherwise, and as it
has my name:

Acked-by: Vitaly Kuznetsov <[email protected]>

> +F: arch/x86/kvm/vmx/evmcs.*
> +
> KERNFS
> M: Greg Kroah-Hartman <[email protected]>
> M: Tejun Heo <[email protected]>

--
Vitaly

2022-06-24 01:55:00

by Wanpeng Li

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

On Thu, 23 Jun 2022 at 22:32, Paolo Bonzini <[email protected]> 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 <[email protected]>
> Co-developed-by: Sean Christopherson <[email protected]>
> Cc: [email protected]
> Cc: Sean Christopherson <[email protected]>
> Cc: Vitaly Kuznetsov <[email protected]>
> Cc: Wanpeng Li <[email protected]>
> Cc: Jim Mattson <[email protected]>
> Cc: Joerg Roedel <[email protected]>
> Signed-off-by: Paolo Bonzini <[email protected]>
> ---
> 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 <[email protected]>
> M: Paolo Bonzini <[email protected]>
> -R: Sean Christopherson <[email protected]>
> -R: Vitaly Kuznetsov <[email protected]>
> -R: Wanpeng Li <[email protected]>
> -R: Jim Mattson <[email protected]>
> -R: Joerg Roedel <[email protected]>
> L: [email protected]
> 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 <[email protected]>
> +R: Wanpeng Li <[email protected]>
> +R: Vitaly Kuznetsov <[email protected]>
> +L: [email protected]
> +S: Supported
> +T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
> +F: arch/x86/kernel/kvm.c
> +F: arch/x86/kernel/kvmclock.c
> +F: arch/x86/include/asm/pvclock-abi.h
> +F: include/linux/kvm_para.h
> +F: include/uapi/linux/kvm_para.h
> +F: include/uapi/asm-generic/kvm_para.h
> +F: include/asm-generic/kvm_para.h
> +F: arch/um/include/asm/kvm_para.h
> +F: arch/x86/include/asm/kvm_para.h
> +F: arch/x86/include/uapi/asm/kvm_para.h
> +
> +KVM X86 HYPER-V (KVM/hyper-v)
> +M: Vitaly Kuznetsov <[email protected]>
> +M: Sean Christopherson <[email protected]>
> +M: Paolo Bonzini <[email protected]>
> +L: [email protected]
> +S: Supported
> +T: git git://git.kernel.org/pub/scm/virt/kvm/kvm.git
> +F: arch/x86/kvm/hyperv.*
> +F: arch/x86/kvm/kvm_onhyperv.*
> +F: arch/x86/kvm/svm/svm_onhyperv.*
> +F: arch/x86/kvm/vmx/evmcs.*
> +
> KERNFS
> M: Greg Kroah-Hartman <[email protected]>
> M: Tejun Heo <[email protected]>
> --

Looks good to me.

Wanpeng

2022-06-24 02:16:24

by Wanpeng Li

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

On Thu, 23 Jun 2022 at 22:32, Paolo Bonzini <[email protected]> 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 <[email protected]>
> Co-developed-by: Sean Christopherson <[email protected]>
> Cc: [email protected]
> Cc: Sean Christopherson <[email protected]>
> Cc: Vitaly Kuznetsov <[email protected]>
> Cc: Wanpeng Li <[email protected]>
> Cc: Jim Mattson <[email protected]>
> Cc: Joerg Roedel <[email protected]>
> Signed-off-by: Paolo Bonzini <[email protected]>
> ---
> 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 <[email protected]>
> M: Paolo Bonzini <[email protected]>
> -R: Sean Christopherson <[email protected]>
> -R: Vitaly Kuznetsov <[email protected]>
> -R: Wanpeng Li <[email protected]>
> -R: Jim Mattson <[email protected]>
> -R: Joerg Roedel <[email protected]>
> L: [email protected]
> 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 <[email protected]>
> +R: Wanpeng Li <[email protected]>

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