From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=37486 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ou8v0-0003Wv-42 for qemu-devel@nongnu.org; Fri, 10 Sep 2010 15:05:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1Ou8uv-000316-VK for qemu-devel@nongnu.org; Fri, 10 Sep 2010 15:05:49 -0400 Received: from mail-vw0-f45.google.com ([209.85.212.45]:58955) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1Ou8uv-000312-Rp for qemu-devel@nongnu.org; Fri, 10 Sep 2010 15:05:45 -0400 Received: by vws19 with SMTP id 19so3014664vws.4 for ; Fri, 10 Sep 2010 12:05:45 -0700 (PDT) Message-ID: <4C8A8183.7070001@codemonkey.ws> Date: Fri, 10 Sep 2010 14:05:39 -0500 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] Re: [PATCH] Use a Linux-style MAINTAINERS file References: <1284067117-31127-1-git-send-email-aliguori@us.ibm.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Blue Swirl Cc: Alexander Graf , Anthony Liguori , qemu-devel@nongnu.org On 09/10/2010 12:45 PM, Blue Swirl wrote: > On Thu, Sep 9, 2010 at 9:18 PM, Anthony Liguori wrote: > >> I make no claims that this is accurate or exhaustive but I think it's a >> reasonable place to start. >> >> As the file mentions, the purpose of this file is to give contributors >> information about who they can go to with questions about a particular piece of >> code or who they can ask for review. >> >> If you sign up for a piece of code and indicate that it's Maintained or >> Supported, please be prepared to be responsive to questions about that >> subsystem. >> >> Signed-off-by: Anthony Liguori >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index 79dfc7f..3894cd8 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -9,89 +9,388 @@ 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 qemu-devel and not any specific individual privately. >> >> -Project leaders: >> ----------------- >> +Please see the MAINTAINERS file in the Linux kernel for information about how >> +to update this file. >> >> -Paul Brook >> -Anthony Liguori >> +General Project Administration >> +------------------------------ >> +M: Anthony Liguori >> +M: Paul Brook >> >> Guest CPU cores (TCG): >> > Perhaps the CPU names should be in alphabetical order. Yeah, I was thinking that alphabetical order across the board would make sense. > For > completeness, KVM CPUs should have X: target-xxx/kvm.c, though > currently TCG and KVM maintainers match. > You mean F: right? I already have that in my copy. >> +SPARC Machines >> +-------------- >> +Sun4u >> > There is also Sun4v here, though its status could be Odd Fixes. These > are actually machine classes, but currently only few machines are > implemented. > What should the entry look like? >> +M: Blue Swirl >> +S: Maintained >> > Maybe also the technical status should be indicated, something like > Unusable, Boots BIOS, Boots a kernel, Stable? > I'd avoid that here. If something is unusable for long, I think that automatically makes it Orphan'd, no? >> +F: hw/sun4u.c >> + >> +Sun4m >> > There's also Sun4c and Sun4d machine classes. There are 9 different > Sun4m machines, but do we want to list them? The same thing actually > applies to isapc machine. > The granularity is what makes sense as someone who's hacking on the code. So whether you think folks would need to see the difference really is up to you since you're the primary person in this space. For PC, I don't think there's really a need to call out each machine individually. > How about also listing the ROMs under pc-bios? I'd claim openbios-*, > except if Alex wants openbios-ppc, that's fine too. > Yes, that's a good idea. Regards, Anthony Liguori