blockdev: Clean up automatic drive deletion
authorMarkus Armbruster <armbru@redhat.com>
Fri, 25 Jun 2010 06:09:10 +0000 (08:09 +0200)
committerKevin Wolf <kwolf@redhat.com>
Fri, 2 Jul 2010 11:18:01 +0000 (13:18 +0200)
commit116d2e361122f68ab976c4715e120bcf626a52c7
treeceb120945adedcbcdaf0bd18bb5558289fe92be6
parent9b6312e526bd736f346a3558948e8ae5ad44929e
blockdev: Clean up automatic drive deletion

We automatically delete blockdev host parts on unplug of the guest
device.  Too much magic, but we can't change that now.

The delete happens early in the guest device teardown, before the
connection to the host part is severed.  Thus, the guest part's
pointer to the host part dangles for a brief time.  No actual harm
comes from this, but we'll catch such dangling pointers a few commits
down the road.  Clean up the dangling pointers by delaying the
automatic deletion until the guest part's pointer is gone.

Device usb-storage deliberately makes two qdev properties refer to the
same drive, because it automatically creates a second device.  Again,
too much magic we can't change now.  Multiple references worked okay
before, but now free_drive() dies for the second one.  Zap the extra
reference.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
blockdev.c
blockdev.h
hw/qdev-properties.c
hw/scsi-disk.c
hw/scsi-generic.c
hw/usb-msd.c
hw/virtio-pci.c