mbox

[PULL] qom-next queue, second batch: QBus, API additions and cleanups

Message ID 1340027954-19045-1-git-send-email-afaerber@suse.de
State New
Headers show

Pull-request

git://repo.or.cz/qemu/afaerber.git qom-next-2

Message

Andreas Färber June 18, 2012, 1:58 p.m. UTC
Hello Anthony,

Seeing no update from Jan nor PULL from mst, here's - hopefully - the final
batch of qom-next patches, as requested.

As usual, I had remarked in my reply when I've taken the liberty of applying,
e.g., style fixes myself. For the preview link that I posted as reply to the
1st batch, I additionally reordered my object_propert_is_child() patch before
object_child_foreach(), so that it isn't touched again later in the queue, and
I swapped two of Paolo's patches to spare us an [AF: ...] modification comment.

Note that due to dependency issues with the revamped Makefile system this
series (like any other central structural change ATM) requires a clean rebuild.

Cc: Anthony Liguori <anthony@codemonkey.ws>
Cc: Paolo Bonzini <pbonzini@redhat.com>

Cc: Jan Kiszka <jan.kiszka@siemens.com>
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>
Cc: Peter Crosthwaite <peter.crosthwaite@petalogix.com>

The following changes since commit eb2aeacf983a2a88a2b31e8fee067c38bd10abd3:

  audio/winwave: Fix typo (2012-06-15 20:58:54 +0400)

are available in the git repository at:
  git://repo.or.cz/qemu/afaerber.git qom-next-2

Andreas Färber (2):
      qom: Introduce object_property_is_{child,link}()
      arm_l2x0: Rename "type" property to "cache-type"

Anthony Liguori (5):
      qdev: Use wrapper for qdev_get_path
      qdev: Convert busses to QEMU Object Model
      qdev: Connect busses with their parent devices
      qbus: Make child devices links
      qbus: Initialize in standard way

Paolo Bonzini (15):
      qom: Add object_class_get_parent()
      qom: Add object_child_foreach()
      qom: Add class_base_init
      qom: Make Object a type
      qom: Drop type_register_static_alias() macro
      qom: Assert that public types have a non-NULL parent field
      m48t59: Rename "type" property to "model"
      qdev: Push "type" property up to Object
      qdev: Move bus properties to a separate global
      qdev: Move bus properties to abstract superclasses
      qdev: Clean up global properties
      qdev: Remove qdev_prop_set_defaults
      qdev: Move SysBus initialization to sysbus.c
      qdev: Remove qdev_prop_exists()
      qom: Push error reporting to object_property_find()

 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(-)

Comments

Anthony Liguori June 20, 2012, 1:09 p.m. UTC | #1
On 06/18/2012 08:58 AM, Andreas Färber wrote:
> Hello Anthony,
>
> Seeing no update from Jan nor PULL from mst, here's - hopefully - the final
> batch of qom-next patches, as requested.
>
> As usual, I had remarked in my reply when I've taken the liberty of applying,
> e.g., style fixes myself. For the preview link that I posted as reply to the
> 1st batch, I additionally reordered my object_propert_is_child() patch before
> object_child_foreach(), so that it isn't touched again later in the queue, and
> I swapped two of Paolo's patches to spare us an [AF: ...] modification comment.
>
> Note that due to dependency issues with the revamped Makefile system this
> series (like any other central structural change ATM) requires a clean rebuild.
>
> Cc: Anthony Liguori<anthony@codemonkey.ws>
> Cc: Paolo Bonzini<pbonzini@redhat.com>
>
> Cc: Jan Kiszka<jan.kiszka@siemens.com>
> Cc: Michael S. Tsirkin<mst@redhat.com>
> Cc: Igor Mammedov<imammedo@redhat.com>
> Cc: Peter Crosthwaite<peter.crosthwaite@petalogix.com>

Pulled.  Thanks.

Regards,

Anthony Liguori

>
> The following changes since commit eb2aeacf983a2a88a2b31e8fee067c38bd10abd3:
>
>    audio/winwave: Fix typo (2012-06-15 20:58:54 +0400)
>
> are available in the git repository at:
>    git://repo.or.cz/qemu/afaerber.git qom-next-2
>
> Andreas Färber (2):
>        qom: Introduce object_property_is_{child,link}()
>        arm_l2x0: Rename "type" property to "cache-type"
>
> Anthony Liguori (5):
>        qdev: Use wrapper for qdev_get_path
>        qdev: Convert busses to QEMU Object Model
>        qdev: Connect busses with their parent devices
>        qbus: Make child devices links
>        qbus: Initialize in standard way
>
> Paolo Bonzini (15):
>        qom: Add object_class_get_parent()
>        qom: Add object_child_foreach()
>        qom: Add class_base_init
>        qom: Make Object a type
>        qom: Drop type_register_static_alias() macro
>        qom: Assert that public types have a non-NULL parent field
>        m48t59: Rename "type" property to "model"
>        qdev: Push "type" property up to Object
>        qdev: Move bus properties to a separate global
>        qdev: Move bus properties to abstract superclasses
>        qdev: Clean up global properties
>        qdev: Remove qdev_prop_set_defaults
>        qdev: Move SysBus initialization to sysbus.c
>        qdev: Remove qdev_prop_exists()
>        qom: Push error reporting to object_property_find()
>
>   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(-)
>
>