From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59083) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aUBoG-0001h2-HR for qemu-devel@nongnu.org; Fri, 12 Feb 2016 06:22:49 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aUBoB-0006OA-Fx for qemu-devel@nongnu.org; Fri, 12 Feb 2016 06:22:48 -0500 References: <20160128140212.GS3869@thinpad.lan.raisama.net> <20160128180031.1d940e2f@nial.brq.redhat.com> <20160203175504.GI26314@thinpad.lan.raisama.net> <20160203205556-mutt-send-email-mst@redhat.com> <56B33676.6000502@redhat.com> <56B338F3.8000500@suse.de> <56B33C2A.8030405@redhat.com> <20160204140322-mutt-send-email-mst@redhat.com> <87r3grd2bq.fsf@blackfin.pond.sub.org> <56B4546B.2030604@redhat.com> <20160211194101.GG6239@thinpad.lan.raisama.net> <56BDA31A.8000908@redhat.com> From: =?UTF-8?Q?Andreas_F=c3=a4rber?= Message-ID: <56BDC081.60402@suse.de> Date: Fri, 12 Feb 2016 12:22:41 +0100 MIME-Version: 1.0 In-Reply-To: <56BDA31A.8000908@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] qdev & hw/core owner? (was Re: [PATCH v19 7/9] machine: add properties to compat_props incrementaly) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Marcel Apfelbaum Cc: Peter Maydell , qemu-devel@nongnu.org, Eduardo Habkost , "Michael S. Tsirkin" , ghammer@redhat.com, agraf@suse.de, Markus Armbruster , borntraeger@de.ibm.com, qemu-ppc@nongnu.org, Gerd Hoffmann , david@gibson.dropbear.id.au, Igor Mammedov , cornelia.huck@de.ibm.com, Paolo Bonzini , lcapitulino@redhat.com, lersek@redhat.com, rth@twiddle.net Am 12.02.2016 um 10:17 schrieb Marcel Apfelbaum: > On 02/11/2016 09:41 PM, Eduardo Habkost wrote: >> On Fri, Feb 05, 2016 at 09:51:07AM +0200, Marcel Apfelbaum wrote: >>> On 02/05/2016 09:49 AM, Markus Armbruster wrote: >>>> "Michael S. Tsirkin" writes: >>>> >>>>> On Thu, Feb 04, 2016 at 12:55:22PM +0100, Paolo Bonzini wrote: >>>>>> >>>>>> >>>>>> On 04/02/2016 12:41, Andreas F=C3=A4rber wrote: >>>>>>> You're talking about machine, right? Some time ago I had proposed >>>>>>> Marcel >>>>>>> who initially worked on it, but I'm fine with anyone taking it. >>>>>> >>>>>> Yes. >>>>>> >>>>>>> For some (but not all) core qdev parts related to the (stalled) Q= OM >>>>>>> migration I've been taking care of via qom-next. Last time this >>>>>>> came up >>>>>>> you didn't want anyone to be M: for qdev, so maybe we can use R: >>>>>>> so that >>>>>>> at least people automatically get CC'ed and we avoid this recurri= ng >>>>>>> discussion? >>>>>> >>>>>> I might have changed my mind on that. You definitely should be M: >>>>>> for qdev. >>>>>> >>>>>> Paolo >>>>> >>>>> If Andreas wants to, that's also fine. Several maintainers are >>>>> better than one. >>>> >>>> *If* the maintainers are all willing and able to work together. >>>> >>> >>> No problem here from my point of view :) >> >> No problem to me, too. :) >> >> I am going to be away from work for 15 days starting on Tuesday >> Feb 16th. So if Marcel wants to start queueing patches already, >> please be my guest. I will be able to help on that after I'm >> back. >> >=20 > Hi, >=20 > If there are only a few patches on the mailing list, they can wait. > If the number will grow I'll send a pull request. >=20 > So the MAINTAINER file should look like this, right? >=20 > Regarding qdev, Andreas, I also think you are the most qualified > to take it, will you? >=20 > diff --git a/MAINTAINERS b/MAINTAINERS > index 2d6ee17..a86491a 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1200,6 +1200,13 @@ F: docs/*qmp-* > F: scripts/qmp/ > T: git git://repo.or.cz/qemu/armbru.git qapi-next >=20 > +Machine > +M: Eduardo Habkost > +M: Marcel Apfelbaum > +S: Supported > +F: hw/core/machine.c > +F: include/hw/boards.h > + Fine with me, ack. For qdev.c itself I prefer not to create a misleading "QDev" section but rather just proposed a first step to split up qdev.c not just into common vs. system-only code but also in better maintainable subareas. That's targeted at having a section like "Core device API" covering a to-be-created device.c with myself plus some backup as maintainer, then Igor/mst/whomever for "Device hotplug interface" or the like. qdev-system.c we could consider to split up so that the block/net/char specific parts can be assigned clear maintainers - haven't investigated that part yet. In the meantime we could simply create multiple sections covering different aspects of qdev* files. Cheers, Andreas --=20 SUSE Linux GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Felix Imend=C3=B6rffer, Jane Smithard, Graham Norton; HRB 21284 (AG N= =C3=BCrnberg)