qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH V3 00/44] Virtio-refactoring part2.
@ 2013-01-15 14:09 fred.konrad
  2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 01/44] virtio: make virtio device's structures public fred.konrad
                   ` (43 more replies)
  0 siblings, 44 replies; 49+ messages in thread
From: fred.konrad @ 2013-01-15 14:09 UTC (permalink / raw)
  To: qemu-devel, aliguori
  Cc: kwolf, peter.maydell, e.voevodin, mst, mark.burton, agraf,
	amit.shah, aneesh.kumar, stefanha, deepakcs, cornelia.huck,
	pbonzini, afaerber, fred.konrad

From: KONRAD Frederic <fred.konrad@greensocs.com>

This is the part 2/3 of the virtio-refactoring for testing/reviewing.

Don't be afraid by the 44 steps, the structure is the same for all devices:
    * Move the *conf field to conf: (I think trivial and short.)
    * Create the virtio-x device.
    * Switch virtio-x-pci to the new API: (Always the same change.)
    * Switch virtio-x-s390 to the new API: (Always the same change.)
    * Use QOM casts: short.
    * Remove qdev field: very short.
    * Cleanup init/exit functions: short.

Also some devices needed more change.

You can clone that from here:
git://git.greensocs.com/qemu_virtio.git virtio_v3_part2

(No need to pull virtio_v3_part1: it is included in virtio_v3_part2).

Thanks,

Fred.

Changes patch v2 -> v3:
    * virtio-bus: change prefix: virtio_device_ -> virtio_bus_.
    * virtio-bus: drop get/set nvectors functions.
    * virtio-pci: use qbus_create_inplace instead of qbus_create.
    * virtio-pci: abstracted.
    * virtio-pci: remove the setting of PCI identifier in
      virtio_pci_device_plugged.
    * virtio-s390: same as above.

Changes since patch v1:
    * virtio-pci: Avoid code duplication (in switch/case).
    * virtio-x-bus: Add the VirtioXBusClass.
    * cut the series.

Changes since v9:
    * Refactored all the remaining devices with the same way as virtio-blk.
    * Moved the virtio device's structure in the header.
    * Fixed the init with only one allocation.
    * Fixed the indentation and code style errors.
    * s390 device's are refactored the same way as pci one.
    * Moved max_dev from BusState to BusClass.

Changes v8 -> v9:
    * Modified virtio-bus.c function name. (virtio_device_ prefix.)
    * Removed qdev_create(..) in virtio-blk-x, object_new
      + object_property_add_child used instead.

Changes v7 -> v8:
    * Moved virtio-blk-pci PCI IDs to "Add the virtio-blk device.".
    * virtio : Added virtio_device_init which init children and plug the device.
    * virtio : Added virtio_common_cleanup in the same way as
      virtio_common_init.
    * virtio-blk : Moved virtio_plug_device to virtio-device init.
    * virtio-blk : Inline the virtio_blk_exit function in
      virtio_blk_device_exit and use virtio_common_cleanup.
    * virtio-s390-bus : Added virtio-s390-bus.
    * virtio-s390-device : Added a virtio-s390-bus.
    * virtio-blk-s390 : Switched to the new API.
    * virtio : removed function pointer.
    * virtio : removed VirtinBindings.
    * virtio : cleaned up init and exit function.

Changes v6 -> v7:
    * virtio-bus : Added virtio-bus-reset.
    * virtio-pci : Fixed virtio-pci-exit.
    * virtio-pci : Added virtio-pci-rst.
    * virtio-pci : Added VirtioPCIClass filled with an init function.
    * virtio-blk : Added virtio_blk_set_conf.
    * virtio-blk : QOM casts.
    * virtio-blk-pci : Switched to the new API.

Changes v5 -> v6:
    * Renamed virtio_common_init_ to virtio_init, modify virtio_common_init to
      allocate and call virtio_init. Drop the unused structure size parameters.
    * Renamed init/exit callback in VirtioBusClass.
    * Renamed virtio_blk_init virtio_blk_common_init.
    * Modified virtio_blk_init to call virtio_blk_common_init.

Changes v4 -> v5:
    * use ERROR_CLASS_GENERIC_ERROR in place of creating a new error type for
      the maximum device limitation. ( Peter )
    * Removed bus_in_use function. We assume that the virtio-bus is not in use,
      when plugin in. ( Peter )
    * Added virtio_bus_destroy_device().
    * Implemented the exit function of virtio-pci.
    * Implemented the init callback for virtio-pci ( must be modified, it still
      access vdev directly. ).
    * Implemented the exit callback for virtio-pci.
    * Started virtio-device refactoring.
    * Started virtio-blk refactoring. 

Changes v3 -> v4:
    * Added virtio-bus.o in Makefile.objs ( accidentally dropped from v3 ).
    * *const* TypeInfo in virtio-bus.
    * Introduced virtio-pci-bus.
    * Reintroduced virtio-pci.
    * Introduced virtio-device.
    * Started virtio-blk refactoring.
    * Added an error type in qerror.h for the "bus full" error.

Changes v2 -> v3:
    * Added VirtioBusClass.
    * Renamed VirtioBus -> VirtioBusState.
    * Renamed qbus -> parent_obj.
    * Plug the device only in a non-full bus.

Changes v1 -> v2:
    * All the little fix you suggest ( License, Debug printf, naming convention,
      ...)
    * Added get_virtio_device_id(), and remove the pci_id* from the VirtioBus
      structure.
    * Added virtio_bus_reset().
    * Added cast macros VIRTIO_BUS.
    * Added virtio_bus_plug_device.
    * Replaced the old-style "bus->qbus" by BUS() macro.

KONRAD Frederic (44):
  virtio: make virtio device's structures public.
  virtio-blk: don't use pointer for configuration.
  virtio-blk: add the virtio-blk device.
  virtio-blk-pci: switch to new API.
  virtio-blk-s390: switch to the new API.
  virtio-blk: cleanup: use QOM cast.
  virtio-blk: cleanup: remove qdev field.
  virtio-net: add the virtio-net device.
  virtio-net-pci: switch to the new API.
  virtio-net-s390: switch to the new API.
  virtio-net: cleanup: use QOM cast.
  virtio-net: cleanup: init and exit function.
  virtio-net: cleanup: remove qdev field.
  virtio-scsi: don't use pointer for configuration.
  virtio-scsi: allocate cmd_vqs array separately.
  virtio-scsi: moving host_features from properties to transport
    properties.
  virtio-scsi: add the virtio-scsi device.
  virtio-scsi-pci: switch to new API.
  virtio-scsi-s390: switch to the new API.
  virtio-scsi: cleanup: use QOM casts.
  virtio-scsi: cleanup: init and exit functions.
  virtio-scsi: cleanup: remove qdev field.
  virtio-balloon: add the virtio-balloon device.
  virtio-balloon-pci: switch to the new API.
  virtio-balloon: cleanup: init and exit function.
  virtio-balloon: cleanup: QOM casts.
  virtio-balloon: cleanup: remove qdev field.
  virtio-rng: don't use pointer for configuration.
  virtio-rng: add virtio-rng device.
  virtio-rng-s390: switch to the new API.
  virtio-rng-pci: switch to the new API.
  virtio-rng.c: cleanup: init and exit functions.
  virtio-rng.c: cleanup: remove qdev field.
  virtio-rng.c: cleanup: use QOM casts.
  virtio-serial: add the virtio-serial device.
  virtio-serial-pci: switch to the new API.
  virtio-serial-s390: switch to the new API.
  virtio-serial: cleanup: init and exit functions.
  virtio-serial: cleanup: use QOM casts.
  virtio-serial: cleanup: remove qdev field.
  virtio-9p: add the virtio-9p device.
  virtio-9p-pci: switch to the new API.
  virtio-9p: cleanup: init function.
  virtio-9p: cleanup: QOM casts.

 hw/9pfs/virtio-9p-device.c | 151 ++++++----
 hw/9pfs/virtio-9p.c        |   2 +-
 hw/9pfs/virtio-9p.h        |  27 +-
 hw/s390-virtio-bus.c       | 157 ++++++----
 hw/s390-virtio-bus.h       |  64 +++-
 hw/virtio-balloon.c        | 113 ++++---
 hw/virtio-balloon.h        |  14 +
 hw/virtio-blk.c            | 153 +++++-----
 hw/virtio-blk.h            |  42 +++
 hw/virtio-net.c            | 287 +++++++++---------
 hw/virtio-net.h            |  58 ++++
 hw/virtio-pci.c            | 711 ++++++++++++++++++++++-----------------------
 hw/virtio-pci.h            |  94 +++++-
 hw/virtio-rng.c            | 143 +++++----
 hw/virtio-rng.h            |  33 +++
 hw/virtio-scsi.c           | 143 +++++----
 hw/virtio-scsi.h           |  31 +-
 hw/virtio-serial-bus.c     | 204 +++++++------
 hw/virtio-serial.h         |  50 ++++
 19 files changed, 1508 insertions(+), 969 deletions(-)

-- 
1.7.11.7

^ permalink raw reply	[flat|nested] 49+ messages in thread

end of thread, other threads:[~2013-01-18  8:10 UTC | newest]

Thread overview: 49+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-15 14:09 [Qemu-devel] [PATCH V3 00/44] Virtio-refactoring part2 fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 01/44] virtio: make virtio device's structures public fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 02/44] virtio-blk: don't use pointer for configuration fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 03/44] virtio-blk: add the virtio-blk device fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 04/44] virtio-blk-pci: switch to new API fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 05/44] virtio-blk-s390: switch to the " fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 06/44] virtio-blk: cleanup: use QOM cast fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 07/44] virtio-blk: cleanup: remove qdev field fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 08/44] virtio-net: add the virtio-net device fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 09/44] virtio-net-pci: switch to the new API fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 10/44] virtio-net-s390: " fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 11/44] virtio-net: cleanup: use QOM cast fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 12/44] virtio-net: cleanup: init and exit function fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 13/44] virtio-net: cleanup: remove qdev field fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 14/44] virtio-scsi: don't use pointer for configuration fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 15/44] virtio-scsi: allocate cmd_vqs array separately fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 16/44] virtio-scsi: moving host_features from properties to transport properties fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 17/44] virtio-scsi: add the virtio-scsi device fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 18/44] virtio-scsi-pci: switch to new API fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 19/44] virtio-scsi-s390: switch to the " fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 20/44] virtio-scsi: cleanup: use QOM casts fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 21/44] virtio-scsi: cleanup: init and exit functions fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 22/44] virtio-scsi: cleanup: remove qdev field fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 23/44] virtio-balloon: add the virtio-balloon device fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 24/44] virtio-balloon-pci: switch to the new API fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 25/44] virtio-balloon: cleanup: init and exit function fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 26/44] virtio-balloon: cleanup: QOM casts fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 27/44] virtio-balloon: cleanup: remove qdev field fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 28/44] virtio-rng: don't use pointer for configuration fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 29/44] virtio-rng: add virtio-rng device fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 30/44] virtio-rng-s390: switch to the new API fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 31/44] virtio-rng-pci: " fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 32/44] virtio-rng.c: cleanup: init and exit functions fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 33/44] virtio-rng.c: cleanup: remove qdev field fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 34/44] virtio-rng.c: cleanup: use QOM casts fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 35/44] virtio-serial: add the virtio-serial device fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 36/44] virtio-serial-pci: switch to the new API fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 37/44] virtio-serial-s390: " fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 38/44] virtio-serial: cleanup: init and exit functions fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 39/44] virtio-serial: cleanup: use QOM casts fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 40/44] virtio-serial: cleanup: remove qdev field fred.konrad
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 41/44] virtio-9p: add the virtio-9p device fred.konrad
2013-01-18  8:04   ` Deepak C Shetty
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 42/44] virtio-9p-pci: switch to the new API fred.konrad
2013-01-18  8:04   ` Deepak C Shetty
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 43/44] virtio-9p: cleanup: init function fred.konrad
2013-01-18  8:04   ` Deepak C Shetty
2013-01-15 14:09 ` [Qemu-devel] [PATCH V3 44/44] virtio-9p: cleanup: QOM casts fred.konrad
2013-01-18  8:05   ` Deepak C Shetty

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).