Message ID | 20190313195555.6198-1-sean.j.christopherson@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | MAINTAINERS: Add KVM selftests to existing KVM entry | expand |
On Wed, Mar 13, 2019 at 12:55:55PM -0700, Sean Christopherson wrote: > It's safe to assume Paolo and Radim are maintaining the KVM selftests > given that the vast majority of commits have their SOBs. Play nice > with get_maintainers and make it official. > > Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 8347269aa386..b95f5b69e5be 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -8295,6 +8295,7 @@ F: include/linux/kvm* > F: include/kvm/iodev.h > F: virt/kvm/* > F: tools/kvm/ > +F: tools/testing/selftests/kvm/ Gah, this also needs: tools/testing/selftests/kvm/*/ > > KERNEL VIRTUAL MACHINE FOR AMD-V (KVM/amd) > M: Joerg Roedel <joro@8bytes.org> > -- > 2.21.0 >
On 13/03/19 21:16, Sean Christopherson wrote: > Gah, this also needs: tools/testing/selftests/kvm/*/ > No, it doesn't. $ scripts/get_maintainer.pl tools/testing/selftests/kvm/x86_64/state_test.c Paolo Bonzini <pbonzini@redhat.com> (supporter:KERNEL VIRTUAL MACHINE (KVM),commit_signer:8/9=89%,authored:3/9=33%) "Radim Krčmář" <rkrcmar@redhat.com> (supporter:KERNEL VIRTUAL MACHINE (KVM),commit_signer:1/9=11%) Queued, thanks. Paolo
On Fri, Mar 15, 2019 at 07:07:45PM +0100, Paolo Bonzini wrote: > On 13/03/19 21:16, Sean Christopherson wrote: > > Gah, this also needs: tools/testing/selftests/kvm/*/ > > > > No, it doesn't. > > $ scripts/get_maintainer.pl tools/testing/selftests/kvm/x86_64/state_test.c > Paolo Bonzini <pbonzini@redhat.com> (supporter:KERNEL VIRTUAL MACHINE (KVM),commit_signer:8/9=89%,authored:3/9=33%) > "Radim Krčmář" <rkrcmar@redhat.com> (supporter:KERNEL VIRTUAL MACHINE (KVM),commit_signer:1/9=11%) Ah, I always run with --pattern-depth=1 when sending KVM patches to avoid picking up the x86 maintainers. I guess now is as good a time as any to dynamically select the depth. That means the similiar "F: arch/x86/kvm/*/" entry I added is also gratutious. Want me to send a revert for c2a7fad5fcd3 ("MAINTAINERS: Add arch/x86/kvm sub-directories to existing KVM/x86 entry"), or just leave it be? > Queued, thanks. > > Paolo
diff --git a/MAINTAINERS b/MAINTAINERS index 8347269aa386..b95f5b69e5be 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -8295,6 +8295,7 @@ F: include/linux/kvm* F: include/kvm/iodev.h F: virt/kvm/* F: tools/kvm/ +F: tools/testing/selftests/kvm/ KERNEL VIRTUAL MACHINE FOR AMD-V (KVM/amd) M: Joerg Roedel <joro@8bytes.org>
It's safe to assume Paolo and Radim are maintaining the KVM selftests given that the vast majority of commits have their SOBs. Play nice with get_maintainers and make it official. Signed-off-by: Sean Christopherson <sean.j.christopherson@intel.com> --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+)