On 09/06/2017 02:42 PM, Cornelia Huck wrote: >> @@ -488,7 +446,8 @@ static int virtio_ccw_cb(SubchDev *sch, CCW1 ccw) >> } else { >> virtio_bus_get_vdev_config(&dev->bus, vdev->config); >> /* XXX config space endianness */ > Unrelated: That should be fine, I guess? >
Would be pretty bad if it weren't. It's the responsibility of the virtio device to give us the right endianness. We don't know the layout so we could not fix it even if we had to. I guess this comment should be removed. Regards, Halil