Message ID | 1438073815-13534-1-git-send-email-alex.bennee@linaro.org (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
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 -- 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
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!
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 -- 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
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.
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 -- 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
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
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.
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 -- 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
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?
On 31/07/2015 17:06, Alex Bennée wrote: > > I haven't seen this upstream yet. Has this been pushed yet? Nope... Paolo -- 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
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/* +
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