Am 28.03.2014 18:13, schrieb Greg Kurz: > On Fri, 28 Mar 2014 11:57:56 +0100 > Greg Kurz <gk...@linux.vnet.ibm.com> wrote: >> @@ -519,8 +521,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(event); >> + evt->reason = virtio_tswap32(reason); > > My bad again, it should be: > > + evt->event = virtio_tswap32(event, vdev); > + evt->reason = virtio_tswap32(reason, vdev);
Pure bikeshedding, but wouldn't it make sense to have the vdev as first argument? Regards, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg