From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45318) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aOpw5-0003hR-W0 for qemu-devel@nongnu.org; Thu, 28 Jan 2016 12:00:47 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aOpw2-000552-47 for qemu-devel@nongnu.org; Thu, 28 Jan 2016 12:00:45 -0500 Date: Thu, 28 Jan 2016 18:00:31 +0100 From: Igor Mammedov Message-ID: <20160128180031.1d940e2f@nial.brq.redhat.com> In-Reply-To: <20160128140212.GS3869@thinpad.lan.raisama.net> References: <1453978470-222624-1-git-send-email-imammedo@redhat.com> <1453978688-222752-1-git-send-email-imammedo@redhat.com> <20160128140212.GS3869@thinpad.lan.raisama.net> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v19 7/9] machine: add properties to compat_props incrementaly List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost Cc: agraf@suse.de, mst@redhat.com, ghammer@redhat.com, lersek@redhat.com, qemu-devel@nongnu.org, lcapitulino@redhat.com, borntraeger@de.ibm.com, qemu-ppc@nongnu.org, cornelia.huck@de.ibm.com, pbonzini@redhat.com, rth@twiddle.net, david@gibson.dropbear.id.au On Thu, 28 Jan 2016 12:02:12 -0200 Eduardo Habkost wrote: > On Thu, Jan 28, 2016 at 11:58:08AM +0100, Igor Mammedov wrote: > > Switch to adding compat properties incrementaly instead of > > completly overwriting compat_props per machine type. > > That removes data duplication which we have due to nested > > [PC|SPAPR]_COMPAT_* macros. > > > > It also allows to set default device properties from > > default foo_machine_options() hook, which will be used > > in following patch for putting VMGENID device as > > a function if ISA bridge on pc/q35 machines. > > > > Suggested-by: Eduardo Habkost > > Signed-off-by: Igor Mammedov > > > > Very nice. The only suggestion I have is to use the simpler GList > type, instead of GArray. It's fine with me to use GList here as well, fill free to pick this patch in case you'd like to do it. it should be trivial to swap from one type to another. 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 > > > compat_props GArray > > I assume this line was left here by mistake? yep, it's leftover from squashing fixup. > > > --- > > hw/core/machine.c | 10 ++++++++++ > > hw/ppc/spapr.c | 3 --- > > hw/s390x/s390-virtio-ccw.c | 12 ++---------- > > include/hw/boards.h | 11 +++++++++-- > > include/hw/i386/pc.h | 9 --------- > > vl.c | 6 +++++- > > 6 files changed, 26 insertions(+), 25 deletions(-) > [...] >