From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56354) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1em3FF-00030m-0D for qemu-devel@nongnu.org; Wed, 14 Feb 2018 15:01:37 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1em3FE-0000ye-07 for qemu-devel@nongnu.org; Wed, 14 Feb 2018 15:01:33 -0500 References: <1518542328-25741-1-git-send-email-mihajlov@linux.vnet.ibm.com> <20180214115725.34ba0008.cohuck@redhat.com> From: Eric Blake Message-ID: Date: Wed, 14 Feb 2018 14:01:21 -0600 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCHv2 0/3] add query-cpu-fast and related s390 changes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cornelia Huck , Viktor Mihajlovski Cc: agraf@suse.de, ehabkost@redhat.com, david@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, dgilbert@redhat.com, borntraeger@de.ibm.com, qemu-s390x@nongnu.org, pbonzini@redhat.com, rth@twiddle.net On 02/14/2018 09:16 AM, Eric Blake wrote: >> How shall we proceed with this series? Patch 3 depends upon patch 1, so >> I think it makes sense to merge this in one go. >> >> I can give my R-b on patch 1 and Someone(tm) can merge this, or I can >> take the whole series through the s390 tree (with some further >> reviews/acks on patches 2/3). > > I'd still like to give a thorough QMP review; it's on my list to get to > today. Okay, I've added more review comments; I think there's still enough worth polishing that resending a v3 of the series is worthwhile, and do agree that we want all three patches through a single tree (s390 sound fine to me). But in general, I'm okay with the QMP changes (what looks like duplication is caused by modernizing the naming, which means we DON'T want to play games with trying to inherit one type from the other, and the argument about deprecating the old naming seems reasonable). -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org