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

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

On Thu, 21 Mar 2013 15:15:10 +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-v5
> 
> Note that it is nearly the same series as virtio-blk refactoring. Though the 2nd
> and the 3rd steps are a virtio-scsi specific.

Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com>

> 
> I made basic tests (with linux guests) on:
>  * qemu-system-i386

On s390 (both s390-virtio and virtio-ccw):
Tested-by: Cornelia Huck <cornelia.huck@de.ibm.com>

> 
> Changes v4 -> v5:
>     * Added macro for features properties.
>     * Fixed the bad comment in virtio-scsi.c spotted by Cornelia.
> 
> 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 |  26 +++++++---
>  hw/s390x/s390-virtio-bus.h |  11 +++-
>  hw/s390x/virtio-ccw.c      |  28 ++++++-----
>  hw/s390x/virtio-ccw.h      |  12 ++++-
>  hw/virtio-pci.c            | 115 +++++++++++++++++++++---------------------
>  hw/virtio-pci.h            |  15 +++++-
>  hw/virtio-scsi.c           | 122 ++++++++++++++++++++++++++++++---------------
>  hw/virtio-scsi.h           |  30 +++++++----
>  8 files changed, 226 insertions(+), 133 deletions(-)
>