Re: [Qemu-devel] [PATCH v4 00/10] virtio-scsi refactoring.

[Cornelia Huck <cornelia.huck@de.ibm.com>]

On Wed, 20 Mar 2013 15:07:33 +0100
fred.konrad@greensocs.com wrote:

> From: KONRAD Frederic <fred.konrad@greensocs.com>
> 
> This is the next part of virtio-refactoring.
> 
> Basically it creates virtio-scsi device which extends virtio-device.
> Then a virtio-scsi can be connected on a virtio-bus.
> virtio-scsi-pci, virtio-scsi-s390x, virtio-scsi-ccw are created too, they extend
> respectively virtio-pci, virtio-s390-device, virtio-ccw-device and have a
> virtio-scsi.
> 
> You can checkout my branch here:
> 
> git://project.greensocs.com/qemu-virtio.git virtio-scsi-v4

I have not looked at the code yet, but with this tree I can create
virtio-scsi-ccw and virtio-scsi-s390 devices that look sane at first
glance. On virtio-ccw, the device also detaches without creating a
hiccup in qemu.

> 
> Note that it is nearly the same series as virtio-blk refactoring. Though the 2nd
> and the 3rd steps are a virtio-scsi specific.
> 
> I made basic tests (with linux guests) on:
>  * qemu-system-i386
> 
> Changes v3 -> v4:
>     * Added CCW device.
>     * Fixed the configuration issue.
> 
> Thanks,
> 
> Fred
> 
> KONRAD Frederic (10):
>   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-ccw: switch to new API
>   virtio-scsi: cleanup: use QOM casts.
>   virtio-scsi: cleanup: init and exit functions.
>   virtio-scsi: cleanup: remove qdev field.
> 
>  hw/s390x/s390-virtio-bus.c |  30 ++++++++---
>  hw/s390x/s390-virtio-bus.h |  11 +++-
>  hw/s390x/virtio-ccw.c      |  32 +++++++-----
>  hw/s390x/virtio-ccw.h      |  12 ++++-
>  hw/virtio-pci.c            | 119 ++++++++++++++++++++++---------------------
>  hw/virtio-pci.h            |  15 +++++-
>  hw/virtio-scsi.c           | 122 ++++++++++++++++++++++++++++++---------------
>  hw/virtio-scsi.h           |  21 ++++----
>  8 files changed, 230 insertions(+), 132 deletions(-)
>