Re: [Qemu-devel] [PULL] qom-next queue, first batch: fixed-width visitors

[Andreas Färber]

Am 08.06.2012 17:35, schrieb Andreas Färber:
> Hello Anthony,
> 
> To get moving with the merge of qom-next into qemu.git, now that the Makefile
> PULL is in, please pull a first small batch, grouping patches directly related
> to fixed-width visitors.
> 
> Outlook:
> 
> A second batch would include the QBus refactoring - still cherry-picking,
> 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.
> 
> Cc: Anthony Liguori <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Michael Roth <address@hidden>
> Cc: Laszlo Ersek <address@hidden>
> Cc: Luiz Capitulino <address@hidden>
> 
> Jan, this does not include the big file movement but the largest part of 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.
> 
> Cc: Jan Kiszka <address@hidden>
> 
> The following changes since commit fa79c914efd35cb60e0bc18512c03690c48b13e2:
> 
>   Merge remote-tracking branch 'bonzini/nested-makefiles-3' into staging 
> (2012-06-07 17:21:40 +0800)
> 
> are available in the git repository at:
> 
>   git://repo.or.cz/qemu/afaerber.git qom-next-1
> 
> Andreas Färber (1):
>       target-i386: Use uint32 visitor for [x]level properties
> 
> 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
> 
> Paolo Bonzini (1):
>       qdev: Remove PropertyInfo range checking
> 
>  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

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg