[kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Alex Bennée]

This may be overkill for a project as small as the unit tests now but
perhaps it pays to be explicit?

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 MAINTAINERS | 67 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 67 insertions(+)
 create mode 100644 MAINTAINERS

diff --git a/MAINTAINERS b/MAINTAINERS
new file mode 100644
index 0000000..89b0260
--- /dev/null
+++ b/MAINTAINERS
@@ -0,0 +1,67 @@
+KVM Unit Tests Maintainers
+==========================
+
+The intention of this file is not to establish who owns what portions of the
+code base, but to provide a set of names that developers can consult when they
+have a question about a particular subset and also to provide a set of names
+to be CC'd when submitting a patch to obtain appropriate review.
+
+In general, if you have a question about inclusion of a patch, you
+should consult the KVM mailing list <kvm@vger.kernel.org> and not any
+specific individual privately.
+
+Descriptions of section entries:
+
+	M: Mail patches to: FullName <address@domain>
+	L: Mailing list that is relevant to this area
+	W: Web-page with status/info
+	Q: Patchwork web based patch tracking system site
+	T: SCM tree type and location.  Type is one of: git, hg, quilt, stgit.
+	S: Status, one of the following:
+	   Supported:	Someone is actually paid to look after this.
+	   Maintained:	Someone actually looks after it.
+	   Odd Fixes:	It has a maintainer but they don't have time to do
+			much other than throw the odd patch in. See below.
+	   Orphan:	No current maintainer [but maybe you could take the
+			role as you write your new code].
+	   Obsolete:	Old code. Something tagged obsolete generally means
+			it has been replaced by a better system and you
+			should be using that.
+	F: Files and directories with wildcard patterns.
+	   A trailing slash includes all files and subdirectory files.
+	   F:	drivers/net/	all files in and below drivers/net
+	   F:	drivers/net/*	all files in drivers/net, but not below
+	   F:	*/net/*		all files in "any top level directory"/net
+	   One pattern per line.  Multiple F: lines acceptable.
+	X: Files and directories that are NOT maintained, same rules as F:
+	   Files exclusions are tested before file matches.
+	   Can be useful for excluding a specific subdirectory, for instance:
+	   F:	net/
+	   X:	net/ipv6/
+	   matches all files in and below net excluding net/ipv6/
+	K: Keyword perl extended regex pattern to match content in a
+	   patch or file.  For instance:
+	   K: of_get_profile
+	      matches patches or files that contain "of_get_profile"
+	   K: \b(printk|pr_(info|err))\b
+	      matches patches or files that contain one or more of the words
+	      printk, pr_info or pr_err
+	   One regex pattern per line.  Multiple K: lines acceptable.
+
+
+Maintainers
+-----------
+L: kvm@vger.kernel.org
+M: Paolo Bonzini <pbonzini@redhat.com>
+M: Marcelo Tosatti <mtosatti@redhat.com>
+T: git://git.kernel.org/pub/scm/virt/kvm/kvm-unit-tests.git
+
+Architecture Specific Code:
+---------------------------
+
+ARM
+M: Christoffer Dall <christoffer.dall@linaro.org>
+L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
+L: kvmarm@lists.cs.columbia.edu
+F: arm/*
+
-- 
2.4.6

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Paolo Bonzini]

On 28/07/2015 10:56, Alex Bennée wrote:
> +Architecture Specific Code:
> +---------------------------
> +
> +ARM
> +M: Christoffer Dall <christoffer.dall@linaro.org>
> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> +L: kvmarm@lists.cs.columbia.edu
> +F: arm/*
> +

Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?

Paolo

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Alex Bennée]

Paolo Bonzini <pbonzini@redhat.com> writes:

> On 28/07/2015 10:56, Alex Bennée wrote:
>> +Architecture Specific Code:
>> +---------------------------
>> +
>> +ARM
>> +M: Christoffer Dall <christoffer.dall@linaro.org>
>> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>> +L: kvmarm@lists.cs.columbia.edu
>> +F: arm/*
>> +
>
> Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?

I was going by his demur deferral earlier ;-)

    "Also, for arm patches I usually CC Christoffer too...""

But sure the more the merrier!

-- 
Alex Bennée

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Andrew Jones]

On Tue, Jul 28, 2015 at 12:41:38PM +0100, Alex Bennée wrote:
> 
> Paolo Bonzini <pbonzini@redhat.com> writes:
> 
> > On 28/07/2015 10:56, Alex Bennée wrote:
> >> +Architecture Specific Code:
> >> +---------------------------
> >> +
> >> +ARM
> >> +M: Christoffer Dall <christoffer.dall@linaro.org>
> >> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
> >> +L: kvmarm@lists.cs.columbia.edu
> >> +F: arm/*
> >> +
> >
> > Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?
> 
> I was going by his demur deferral earlier ;-)
> 
>     "Also, for arm patches I usually CC Christoffer too...""
> 
> But sure the more the merrier!

Yeah, I wasn't running away from the role. I was just stating that it's
nice to get Christoffer's input on low-level arm bits. And, particularly
as we start adding real kvm tests, it'd be good to get his and/or Marc's
input to make sure the tests are complete. Anyway, feel free to add me
as a maintainer for kvm-unit-tests/arm. I'm happy to do it.

(Now, I'm not sure we need a full-fledged maintainer file. I was thinking
 a couple lines in the README would suffice. I'll leave that up to Paolo
 though.)

Thanks,
drew

> 
> -- 
> Alex Bennée
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Marc Zyngier]

On 28/07/15 13:44, Andrew Jones wrote:
> On Tue, Jul 28, 2015 at 12:41:38PM +0100, Alex Bennée wrote:
>>
>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>
>>> On 28/07/2015 10:56, Alex Bennée wrote:
>>>> +Architecture Specific Code:
>>>> +---------------------------
>>>> +
>>>> +ARM
>>>> +M: Christoffer Dall <christoffer.dall@linaro.org>
>>>> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>>> +L: kvmarm@lists.cs.columbia.edu
>>>> +F: arm/*
>>>> +
>>>
>>> Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?
>>
>> I was going by his demur deferral earlier ;-)
>>
>>     "Also, for arm patches I usually CC Christoffer too...""
>>
>> But sure the more the merrier!
> 
> Yeah, I wasn't running away from the role. I was just stating that it's
> nice to get Christoffer's input on low-level arm bits. And, particularly
> as we start adding real kvm tests, it'd be good to get his and/or Marc's
> input to make sure the tests are complete. Anyway, feel free to add me
> as a maintainer for kvm-unit-tests/arm. I'm happy to do it.

I'm always happy to be CCed on architecture related questions. Just
don't advertise me as a maintainer for the unit tests! ;-)

	M.
-- 
Jazz is not dead. It just smells funny...

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Christoffer Dall]

On Tue, Jul 28, 2015 at 2:51 PM, Marc Zyngier <marc.zyngier@arm.com> wrote:
> On 28/07/15 13:44, Andrew Jones wrote:
>> On Tue, Jul 28, 2015 at 12:41:38PM +0100, Alex Bennée wrote:
>>>
>>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>>
>>>> On 28/07/2015 10:56, Alex Bennée wrote:
>>>>> +Architecture Specific Code:
>>>>> +---------------------------
>>>>> +
>>>>> +ARM
>>>>> +M: Christoffer Dall <christoffer.dall@linaro.org>
>>>>> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>>>> +L: kvmarm@lists.cs.columbia.edu
>>>>> +F: arm/*
>>>>> +
>>>>
>>>> Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?
>>>
>>> I was going by his demur deferral earlier ;-)
>>>
>>>     "Also, for arm patches I usually CC Christoffer too...""
>>>
>>> But sure the more the merrier!
>>
>> Yeah, I wasn't running away from the role. I was just stating that it's
>> nice to get Christoffer's input on low-level arm bits. And, particularly
>> as we start adding real kvm tests, it'd be good to get his and/or Marc's
>> input to make sure the tests are complete. Anyway, feel free to add me
>> as a maintainer for kvm-unit-tests/arm. I'm happy to do it.
>
> I'm always happy to be CCed on architecture related questions. Just
> don't advertise me as a maintainer for the unit tests! ;-)
>
I'm with Marc here, don't put me in the maintainers file, I don't have
the bandwidth for this at the moment and I'm quite happy seeing Drew
taking care of this.

-Christoffer

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Alex Bennée]

Christoffer Dall <christoffer.dall@linaro.org> writes:

> On Tue, Jul 28, 2015 at 2:51 PM, Marc Zyngier <marc.zyngier@arm.com> wrote:
>> On 28/07/15 13:44, Andrew Jones wrote:
>>> On Tue, Jul 28, 2015 at 12:41:38PM +0100, Alex Bennée wrote:
>>>>
>>>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>>>
>>>>> On 28/07/2015 10:56, Alex Bennée wrote:
>>>>>> +Architecture Specific Code:
>>>>>> +---------------------------
>>>>>> +
>>>>>> +ARM
>>>>>> +M: Christoffer Dall <christoffer.dall@linaro.org>
>>>>>> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>>>>> +L: kvmarm@lists.cs.columbia.edu
>>>>>> +F: arm/*
>>>>>> +
>>>>>
>>>>> Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?
>>>>
>>>> I was going by his demur deferral earlier ;-)
>>>>
>>>>     "Also, for arm patches I usually CC Christoffer too...""
>>>>
>>>> But sure the more the merrier!
>>>
>>> Yeah, I wasn't running away from the role. I was just stating that it's
>>> nice to get Christoffer's input on low-level arm bits. And, particularly
>>> as we start adding real kvm tests, it'd be good to get his and/or Marc's
>>> input to make sure the tests are complete. Anyway, feel free to add me
>>> as a maintainer for kvm-unit-tests/arm. I'm happy to do it.
>>
>> I'm always happy to be CCed on architecture related questions. Just
>> don't advertise me as a maintainer for the unit tests! ;-)
>>
> I'm with Marc here, don't put me in the maintainers file, I don't have
> the bandwidth for this at the moment and I'm quite happy seeing Drew
> taking care of this.

I shall add suggested CC's then and leave Drew as maintainer?

>
> -Christoffer

-- 
Alex Bennée

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Marc Zyngier]

On 29/07/15 10:44, Alex Bennée wrote:
> 
> Christoffer Dall <christoffer.dall@linaro.org> writes:
> 
>> On Tue, Jul 28, 2015 at 2:51 PM, Marc Zyngier <marc.zyngier@arm.com> wrote:
>>> On 28/07/15 13:44, Andrew Jones wrote:
>>>> On Tue, Jul 28, 2015 at 12:41:38PM +0100, Alex Bennée wrote:
>>>>>
>>>>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>>>>
>>>>>> On 28/07/2015 10:56, Alex Bennée wrote:
>>>>>>> +Architecture Specific Code:
>>>>>>> +---------------------------
>>>>>>> +
>>>>>>> +ARM
>>>>>>> +M: Christoffer Dall <christoffer.dall@linaro.org>
>>>>>>> +L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>>>>>> +L: kvmarm@lists.cs.columbia.edu
>>>>>>> +F: arm/*
>>>>>>> +
>>>>>>
>>>>>> Hmm, shouldn't the maintainer be Drew here (or Drew + Christoffer)?
>>>>>
>>>>> I was going by his demur deferral earlier ;-)
>>>>>
>>>>>     "Also, for arm patches I usually CC Christoffer too...""
>>>>>
>>>>> But sure the more the merrier!
>>>>
>>>> Yeah, I wasn't running away from the role. I was just stating that it's
>>>> nice to get Christoffer's input on low-level arm bits. And, particularly
>>>> as we start adding real kvm tests, it'd be good to get his and/or Marc's
>>>> input to make sure the tests are complete. Anyway, feel free to add me
>>>> as a maintainer for kvm-unit-tests/arm. I'm happy to do it.
>>>
>>> I'm always happy to be CCed on architecture related questions. Just
>>> don't advertise me as a maintainer for the unit tests! ;-)
>>>
>> I'm with Marc here, don't put me in the maintainers file, I don't have
>> the bandwidth for this at the moment and I'm quite happy seeing Drew
>> taking care of this.
> 
> I shall add suggested CC's then and leave Drew as maintainer?

That'd be my preferred option. Drew is doing a great job, and he should
be recognized for it.

Thanks,

	M.
-- 
Jazz is not dead. It just smells funny...

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Paolo Bonzini]

Applied as follows:

> +Maintainers
> +-----------
> +M: Paolo Bonzini <pbonzini@redhat.com>
> +M: Marcelo Tosatti <mtosatti@redhat.com>
> +L: kvm@vger.kernel.org
> +T: git://git.kernel.org/pub/scm/virt/kvm/kvm-unit-tests.git
> +
> +Architecture Specific Code:
> +---------------------------
> +
> +ARM
> +M: Drew Jones <drjones@redhat.com>
> +L: kvm@vger.kernel.org
> +L: kvmarm@lists.cs.columbia.edu
> +F: arm/*
> +F: lib/arm/*
> +F: lib/arm64/*
> +
> +x86
> +M: Paolo Bonzini <pbonzini@redhat.com>
> +M: Marcelo Tosatti <mtosatti@redhat.com>
> +L: kvm@vger.kernel.org
> +F: x86/*
> +F: lib/x86/*

Paolo

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Alex Bennée]

Paolo Bonzini <pbonzini@redhat.com> writes:

> Applied as follows:
>
>> +Maintainers
>> +-----------
>> +M: Paolo Bonzini <pbonzini@redhat.com>
>> +M: Marcelo Tosatti <mtosatti@redhat.com>
>> +L: kvm@vger.kernel.org
>> +T: git://git.kernel.org/pub/scm/virt/kvm/kvm-unit-tests.git
>> +
>> +Architecture Specific Code:
>> +---------------------------
>> +
>> +ARM
>> +M: Drew Jones <drjones@redhat.com>
>> +L: kvm@vger.kernel.org
>> +L: kvmarm@lists.cs.columbia.edu
>> +F: arm/*
>> +F: lib/arm/*
>> +F: lib/arm64/*
>> +
>> +x86
>> +M: Paolo Bonzini <pbonzini@redhat.com>
>> +M: Marcelo Tosatti <mtosatti@redhat.com>
>> +L: kvm@vger.kernel.org
>> +F: x86/*
>> +F: lib/x86/*


I haven't seen this upstream yet. Has this been pushed yet?

-- 
Alex Bennée

Re: [kvm-unit-tests RFC PATCH] Add MAINTAINERS file

[Paolo Bonzini]

On 31/07/2015 17:06, Alex Bennée wrote:
> 
> I haven't seen this upstream yet. Has this been pushed yet?

Nope...

Paolo