On 2019-01-15 19:19, Paolo Bonzini wrote: > From: Emanuele Giuseppe Esposito <[email protected]> > > This function is intended to group all the qvirtio_* functions that > start the qvirtio devices. > Applied in all tests using this combination of functions. > > Signed-off-by: Emanuele Giuseppe Esposito <[email protected]> > Signed-off-by: Paolo Bonzini <[email protected]> > --- [..] > diff --git a/tests/virtio-blk-test.c b/tests/virtio-blk-test.c > index 04c6087..f97c158 100644 > --- a/tests/virtio-blk-test.c > +++ b/tests/virtio-blk-test.c > @@ -116,10 +116,7 @@ static QVirtioPCIDevice *virtio_blk_pci_init(QPCIBus > *bus, int slot) > g_assert_cmphex(dev->pdev->devfn, ==, ((slot << 3) | PCI_FN)); > > qvirtio_pci_device_enable(dev); > - qvirtio_reset(&dev->vdev); > - qvirtio_set_acknowledge(&dev->vdev); > - qvirtio_set_driver(&dev->vdev); > - > + qvirtio_start_device(&dev->vdev); > return dev; > } > > @@ -333,6 +330,7 @@ static void pci_indirect(void) > qvirtio_set_features(&dev->vdev, features); > > vqpci = (QVirtQueuePCI *)qvirtqueue_setup(&dev->vdev, qs->alloc, 0); > + > qvirtio_set_driver_ok(&dev->vdev);
Still unnecessary white-space change. Apart from that nit, still: Reviewed-by: Thomas Huth <[email protected]>
