summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorCédric Le Goater <clg@fr.ibm.com>2014-06-30 17:17:17 +0200
committerPaolo Bonzini <pbonzini@redhat.com>2014-07-01 09:40:38 +0200
commit424baff549a9c8a7b5e814ce2bcb857d25163468 (patch)
tree3b9743827253720f47ef79eb9c8be5e5f2d33f9e
parentdfecbb95e38d94d0e68f6e648a3c6489219a885d (diff)
downloadfocaccia-qemu-424baff549a9c8a7b5e814ce2bcb857d25163468.tar.gz
focaccia-qemu-424baff549a9c8a7b5e814ce2bcb857d25163468.zip
virtio-scsi: scsi events must be converted to target endianness
Virtio SCSI Events need to be byteswapped before being pushed
when host and guest have a different endianness. Not doing so
breaks hotplug of virtio scsi disks, with the following error
message being printed in the guest console:

virtio_scsi: Unsupport virtio scsi event 1000000

This issue got uncovered while testing disk hotplug with a PowerKVM
ppc64le guest. I have checked that this issue also affects a x86_64
guest run on a ppc64 host.

Signed-off-by: Cédric Le Goater <clg@fr.ibm.com>
[ Ported from PowerKVM,
  Greg Kurz <gkurz@linux.vnet.ibm.com> ]
Signed-off-by: Greg Kurz <gkurz@linux.vnet.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
-rw-r--r--hw/scsi/virtio-scsi.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/scsi/virtio-scsi.c b/hw/scsi/virtio-scsi.c
index 3fecdcaa2e..0eb069ae9b 100644
--- a/hw/scsi/virtio-scsi.c
+++ b/hw/scsi/virtio-scsi.c
@@ -587,8 +587,8 @@ static void virtio_scsi_push_event(VirtIOSCSI *s, SCSIDevice *dev,
 
     evt = &req->resp.event;
     memset(evt, 0, sizeof(VirtIOSCSIEvent));
-    evt->event = event;
-    evt->reason = reason;
+    evt->event = virtio_tswap32(vdev, event);
+    evt->reason = virtio_tswap32(vdev, reason);
     if (!dev) {
         assert(event == VIRTIO_SCSI_T_EVENTS_MISSED);
     } else {