From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:41108) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sd6ty-0007JT-W5 for qemu-devel@nongnu.org; Fri, 08 Jun 2012 17:39:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Sd6tw-0008JH-Jn for qemu-devel@nongnu.org; Fri, 08 Jun 2012 17:39:26 -0400 Received: from cantor2.suse.de ([195.135.220.15]:45167 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Sd6tw-0008Iy-A6 for qemu-devel@nongnu.org; Fri, 08 Jun 2012 17:39:24 -0400 Message-ID: <4FD27103.5070708@suse.de> Date: Fri, 08 Jun 2012 23:39:15 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1339169713-31205-1-git-send-email-afaerber@suse.de> In-Reply-To: <1339169713-31205-1-git-send-email-afaerber@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL] qom-next queue, first batch: fixed-width visitors List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Jan Kiszka , Michael Roth , Luiz Capitulino , Anthony Liguori , Paolo Bonzini , Laszlo Ersek Am 08.06.2012 17:35, schrieb Andreas F=C3=A4rber: > Hello Anthony, >=20 > To get moving with the merge of qom-next into qemu.git, now that the Ma= kefile > PULL is in, please pull a first small batch, grouping patches directly = related > to fixed-width visitors. >=20 > Outlook: >=20 > A second batch would include the QBus refactoring - still cherry-pickin= g, > reordering and re-testing. Preview is ready: http://repo.or.cz/w/qemu/afaerber.git/shortlog/refs/heads/qom-next-2 exec.c | 4 +- hw/acpi_piix4.c | 10 +- hw/arm_l2x0.c | 2 +- hw/i2c.c | 30 +++-- hw/ide/internal.h | 3 + hw/ide/qdev.c | 31 +++-- hw/intel-hda.c | 37 +++-- hw/intel-hda.h | 3 + hw/isa-bus.c | 23 +++- hw/isa.h | 3 + hw/lsi53c895a.c | 5 +- hw/m48t59.c | 40 +++--- hw/pc_piix.c | 7 +- hw/pci-hotplug.c | 6 +- hw/pci.c | 51 +++++--- hw/pci_bridge.c | 2 +- hw/pci_internals.h | 3 +- hw/qdev-monitor.c | 101 ++++++++------- hw/qdev-properties.c | 65 +++------ hw/qdev.c | 300 +++++++++++++++++++++++++++-------------- hw/qdev.h | 65 +++++---- hw/s390-virtio-bus.c | 37 +++--- hw/s390-virtio-bus.h | 4 + hw/scsi-bus.c | 58 +++++--- hw/scsi.h | 3 + hw/spapr_pci.c | 7 +- hw/spapr_vio.c | 47 ++++--- hw/spapr_vio.h | 3 + hw/spapr_vty.c | 6 +- hw/ssi.c | 29 +++-- hw/sysbus.c | 45 ++++++- hw/sysbus.h | 3 + hw/usb.h | 3 + hw/usb/bus.c | 45 ++++--- hw/usb/desc.c | 5 +- hw/usb/dev-smartcard-reader.c | 25 +++-- hw/virtio-scsi.c | 6 +- hw/virtio-serial-bus.c | 36 ++++-- include/qemu/object.h | 47 ++++++- qom/object.c | 158 +++++++++++++++------ savevm.c | 12 +- 41 files changed, 868 insertions(+), 502 deletions(-) > I'm preparing a two-patch realize series that could get appended to the= second. > Still open are the static property movements and their prerequisites. > > CPU-related patches are split off into their own pull. >=20 > Cc: Anthony Liguori > Cc: Paolo Bonzini > Cc: Michael Roth > Cc: Laszlo Ersek > Cc: Luiz Capitulino >=20 > Jan, this does not include the big file movement but the largest part o= f the > qdev-properties.c changes and in particular the ones that affect PCI. I= suggest > that you rebase on this one and me/Paolo the remaining parts on yours. >=20 > Cc: Jan Kiszka >=20 > The following changes since commit fa79c914efd35cb60e0bc18512c03690c48b= 13e2: >=20 > Merge remote-tracking branch 'bonzini/nested-makefiles-3' into stagin= g (2012-06-07 17:21:40 +0800) >=20 > are available in the git repository at: >=20 > git://repo.or.cz/qemu/afaerber.git qom-next-1 >=20 > Andreas F=C3=A4rber (1): > target-i386: Use uint32 visitor for [x]level properties >=20 > Michael Roth (6): > qapi: Add Visitor interfaces for uint*_t and int*_t > qapi: Unit tests for visitor-based serialization > qapi: String visitor, use %f representation for floats > qapi: Add String visitor coverage to serialization unit tests > qdev: Use int32_t container for devfn property > qdev: Switch property accessors to fixed-width visitor interfaces >=20 > Paolo Bonzini (1): > qdev: Remove PropertyInfo range checking >=20 > hw/mc146818rtc.c | 7 - > hw/pci.c | 2 +- > hw/pci.h | 2 +- > hw/qdev-addr.c | 4 +- > hw/qdev-properties.c | 205 ++++------ > hw/qdev.h | 4 +- > qapi/qapi-visit-core.c | 139 +++++++ > qapi/qapi-visit-core.h | 16 + > qapi/string-output-visitor.c | 2 +- > target-i386/cpu.c | 42 +-- > tests/Makefile | 4 +- > tests/test-string-output-visitor.c | 2 +- > tests/test-visitor-serialization.c | 784 ++++++++++++++++++++++++++++= ++++++++ > 13 files changed, 1041 insertions(+), 172 deletions(-) > create mode 100644 tests/test-visitor-serialization.c --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg