From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52938) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WwV26-00021j-H5 for qemu-devel@nongnu.org; Mon, 16 Jun 2014 07:25:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WwV20-0001JC-Gw for qemu-devel@nongnu.org; Mon, 16 Jun 2014 07:25:02 -0400 Received: from mx1.redhat.com ([209.132.183.28]:24152) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WwV20-0001J5-83 for qemu-devel@nongnu.org; Mon, 16 Jun 2014 07:24:56 -0400 From: Stefan Hajnoczi Date: Mon, 16 Jun 2014 19:23:29 +0800 Message-Id: <1402917843-6459-6-git-send-email-stefanha@redhat.com> In-Reply-To: <1402917843-6459-1-git-send-email-stefanha@redhat.com> References: <1402917843-6459-1-git-send-email-stefanha@redhat.com> Subject: [Qemu-devel] [PULL 05/39] blockdev: Remove unused DriveInfo reference count List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Peter Maydell , Markus Armbruster , Stefan Hajnoczi From: Markus Armbruster It's always one since commit fa510eb dropped the last drive_get_ref(). Signed-off-by: Markus Armbruster Reviewed-by: Benoit Canet Signed-off-by: Stefan Hajnoczi --- blockdev.c | 18 ++---------------- device-hotplug.c | 2 +- hw/ide/piix.c | 2 +- include/sysemu/blockdev.h | 4 +--- 4 files changed, 5 insertions(+), 21 deletions(-) diff --git a/blockdev.c b/blockdev.c index 0ed03b7..9b0f8ac 100644 --- a/blockdev.c +++ b/blockdev.c @@ -106,7 +106,7 @@ void blockdev_auto_del(BlockDriverState *bs) DriveInfo *dinfo = drive_get_by_blockdev(bs); if (dinfo && dinfo->auto_del) { - drive_put_ref(dinfo); + drive_del(dinfo); } } @@ -213,7 +213,7 @@ static void bdrv_format_print(void *opaque, const char *name) error_printf(" %s", name); } -static void drive_del(DriveInfo *dinfo) +void drive_del(DriveInfo *dinfo) { if (dinfo->opts) { qemu_opts_del(dinfo->opts); @@ -226,19 +226,6 @@ static void drive_del(DriveInfo *dinfo) g_free(dinfo); } -void drive_put_ref(DriveInfo *dinfo) -{ - assert(dinfo->refcount); - if (--dinfo->refcount == 0) { - drive_del(dinfo); - } -} - -void drive_get_ref(DriveInfo *dinfo) -{ - dinfo->refcount++; -} - typedef struct { QEMUBH *bh; BlockDriverState *bs; @@ -497,7 +484,6 @@ static DriveInfo *blockdev_init(const char *file, QDict *bs_opts, dinfo->bdrv->open_flags = snapshot ? BDRV_O_SNAPSHOT : 0; dinfo->bdrv->read_only = ro; dinfo->bdrv->detect_zeroes = detect_zeroes; - dinfo->refcount = 1; QTAILQ_INSERT_TAIL(&drives, dinfo, next); bdrv_set_on_error(dinfo->bdrv, on_read_error, on_write_error); diff --git a/device-hotplug.c b/device-hotplug.c index fc09d10..e6a1ffb 100644 --- a/device-hotplug.c +++ b/device-hotplug.c @@ -76,6 +76,6 @@ void drive_hot_add(Monitor *mon, const QDict *qdict) err: if (dinfo) { - drive_put_ref(dinfo); + drive_del(dinfo); } } diff --git a/hw/ide/piix.c b/hw/ide/piix.c index 40757eb..8651726 100644 --- a/hw/ide/piix.c +++ b/hw/ide/piix.c @@ -184,7 +184,7 @@ int pci_piix3_xen_ide_unplug(DeviceState *dev) } bdrv_close(di->bdrv); pci_ide->bus[di->bus].ifs[di->unit].bs = NULL; - drive_put_ref(di); + drive_del(di); } } qdev_reset_all(DEVICE(dev)); diff --git a/include/sysemu/blockdev.h b/include/sysemu/blockdev.h index 0fdbd68..23a5d10 100644 --- a/include/sysemu/blockdev.h +++ b/include/sysemu/blockdev.h @@ -43,21 +43,19 @@ struct DriveInfo { QemuOpts *opts; char *serial; QTAILQ_ENTRY(DriveInfo) next; - int refcount; }; DriveInfo *drive_get(BlockInterfaceType type, int bus, int unit); DriveInfo *drive_get_by_index(BlockInterfaceType type, int index); int drive_get_max_bus(BlockInterfaceType type); DriveInfo *drive_get_next(BlockInterfaceType type); -void drive_get_ref(DriveInfo *dinfo); -void drive_put_ref(DriveInfo *dinfo); DriveInfo *drive_get_by_blockdev(BlockDriverState *bs); QemuOpts *drive_def(const char *optstr); QemuOpts *drive_add(BlockInterfaceType type, int index, const char *file, const char *optstr); DriveInfo *drive_new(QemuOpts *arg, BlockInterfaceType block_default_type); +void drive_del(DriveInfo *dinfo); /* device-hotplug */ -- 1.9.3