[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] qdev & hw/core owner? (was Re: [PATCH v19 7/9] machine: a

From: Michael S. Tsirkin
Subject: Re: [Qemu-ppc] qdev & hw/core owner? (was Re: [PATCH v19 7/9] machine: add properties to compat_props incrementaly)
Date: Thu, 4 Feb 2016 14:03:12 +0200

On Thu, Feb 04, 2016 at 12:41:39PM +0100, Andreas Färber wrote:
> Am 04.02.2016 um 12:31 schrieb Paolo Bonzini:
> > On 03/02/2016 20:06, Michael S. Tsirkin wrote:
> >> On Wed, Feb 03, 2016 at 03:55:04PM -0200, Eduardo Habkost wrote:
> >>> On Thu, Jan 28, 2016 at 06:00:31PM +0100, Igor Mammedov wrote:
> >>> [...]
> >>>> It looks like this series might go nowhere but this patch
> >>>> is not tied to it and useful to us in general
> >>>> so perhaps you could pick it up after ACKs from
> >>>> S390/SPAPR maintainers.
> >>>>
> >>>>>
> >>>>> Reviewed-by: Eduardo Habkost <address@hidden>
> >>>
> >>> We don't have a maintainer for hw/core/machine.c, hw/core/qdev*,
> >>> and related files.
> >>>
> >>> Assuming we don't have a volunteer to maintain them officially,
> >>> can we agree on a default destination for those patches so they
> >>> don't linger on the list? Michael? Andreas?
> >>
> >> Not me please. Have too much on my plate.
> >> Would you like to maintain it yourself?
> > 
> > That's my suggestion too.  I guess Igor and I could help with reviews,
> > but testing and sending the pull requests would add too much work.
> > Since you're the main one touching it, it makes sense for you to handle it.
> 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.

Sure, Marcel can do it too.

> For some (but not all) core qdev parts related to the (stalled) QOM
> 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 recurring
> discussion?
> Regards,
> Andreas

I think that was because core was not changing much, so merging through
other trees was more appropriate, but that changed.

So it appears we already can have
M: Eduardo Habkost <address@hidden>
M: Marcel Apfelbaum <address@hidden>

This should spread the review load nicely.

> -- 
> SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)

reply via email to

[Prev in Thread] Current Thread [Next in Thread]