diff options
author | Sasha Levin <sasha.levin@oracle.com> | 2015-01-02 14:47:39 -0500 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2015-01-21 00:58:50 -0500 |
commit | 2bd56afd44123cea3741c7a46ddd96a46c92b8d9 (patch) | |
tree | ea894fee9d106874bdc3b0dacbf8bd6baad14467 /drivers/virtio | |
parent | 8cfc99b58366ea9f391fe0da7d16940ca6a1d9c0 (diff) |
virtio_pci: drop useless del_vqs call
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>
Diffstat (limited to 'drivers/virtio')
-rw-r--r-- | drivers/virtio/virtio_pci_legacy.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/virtio/virtio_pci_legacy.c b/drivers/virtio/virtio_pci_legacy.c index a5486e65e04b..19f93096739a 100644 --- a/drivers/virtio/virtio_pci_legacy.c +++ b/drivers/virtio/virtio_pci_legacy.c | |||
@@ -309,7 +309,6 @@ void virtio_pci_legacy_remove(struct pci_dev *pci_dev) | |||
309 | 309 | ||
310 | unregister_virtio_device(&vp_dev->vdev); | 310 | unregister_virtio_device(&vp_dev->vdev); |
311 | 311 | ||
312 | vp_del_vqs(&vp_dev->vdev); | ||
313 | pci_iounmap(pci_dev, vp_dev->ioaddr); | 312 | pci_iounmap(pci_dev, vp_dev->ioaddr); |
314 | pci_release_regions(pci_dev); | 313 | pci_release_regions(pci_dev); |
315 | pci_disable_device(pci_dev); | 314 | pci_disable_device(pci_dev); |