On 29/03/2016 15:42, Michael S. Tsirkin wrote: > + if (s->dataplane) { > + /* Some guests kick before setting VIRTIO_CONFIG_S_DRIVER_OK so start > + * dataplane here instead of waiting for .set_status(). > + */ > + if (!s->dataplane_started) { > + virtio_blk_data_plane_start(s->dataplane); > + } > + return; > + } > + > + virtio_blk_handle_vq(s, vq);
Another small comment, this can be written simply as if (s->dataplane) { virtio_blk_data_plane_start(s->dataplane); } else { virtio_blk_handle_vq(s, vq); } Paolo