virtio_pci: drop useless del_vqs call
authorSasha Levin <sasha.levin@oracle.com>
Fri, 2 Jan 2015 19:47:39 +0000 (14:47 -0500)
committerRusty Russell <rusty@rustcorp.com.au>
Wed, 21 Jan 2015 05:58:50 +0000 (16:28 +1030)
Device VQs were getting freed twice: once in every device's removal
functions, and then again in virtio_pci_legacy_remove().  The ones in
devices are called first, so drop the useless second call.

Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
drivers/virtio/virtio_pci_legacy.c

index a5486e65e04bd55d5c64a33d3dbeeadb27dd4857..19f93096739a004d6eaa21797f6356b2b627604e 100644 (file)
@@ -309,7 +309,6 @@ void virtio_pci_legacy_remove(struct pci_dev *pci_dev)
 
        unregister_virtio_device(&vp_dev->vdev);
 
-       vp_del_vqs(&vp_dev->vdev);
        pci_iounmap(pci_dev, vp_dev->ioaddr);
        pci_release_regions(pci_dev);
        pci_disable_device(pci_dev);