[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 24/30] scsi: cleanup not used anymore SCSIBusInf
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 24/30] scsi: cleanup not used anymore SCSIBusInfo{hotplug, hot_unplug} fields |
Date: |
Wed, 24 Sep 2014 14:12:23 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0 |
Il 24/09/2014 13:48, Igor Mammedov ha scritto:
> SCSI subsytem was converted to hotplug handler API and
> doesn't use SCSIBusInfo{hotplug, hot_unplug} fields and
> related callbacks anymore.
>
> Signed-off-by: Igor Mammedov <address@hidden>
> ---
> hw/scsi/scsi-bus.c | 16 ----------------
> include/hw/scsi/scsi.h | 2 --
> 2 files changed, 18 deletions(-)
>
> diff --git a/hw/scsi/scsi-bus.c b/hw/scsi/scsi-bus.c
> index 33495b6..9bdc63b 100644
> --- a/hw/scsi/scsi-bus.c
> +++ b/hw/scsi/scsi-bus.c
> @@ -202,10 +202,6 @@ static void scsi_qdev_realize(DeviceState *qdev, Error
> **errp)
> }
> dev->vmsentry = qemu_add_vm_change_state_handler(scsi_dma_restart_cb,
> dev);
> -
> - if (bus->info->hotplug) {
> - bus->info->hotplug(bus, dev);
> - }
> }
>
> static void scsi_qdev_unrealize(DeviceState *qdev, Error **errp)
> @@ -1918,17 +1914,6 @@ static int get_scsi_requests(QEMUFile *f, void *pv,
> size_t size)
> return 0;
> }
>
> -static int scsi_qdev_unplug(DeviceState *qdev)
> -{
> - SCSIDevice *dev = SCSI_DEVICE(qdev);
> - SCSIBus *bus = DO_UPCAST(SCSIBus, qbus, dev->qdev.parent_bus);
> -
> - if (bus->info->hot_unplug) {
> - bus->info->hot_unplug(bus, dev);
> - }
> - return qdev_simple_unplug_cb(qdev);
> -}
> -
> static const VMStateInfo vmstate_info_scsi_requests = {
> .name = "scsi-requests",
> .get = get_scsi_requests,
> @@ -1992,7 +1977,6 @@ static void scsi_device_class_init(ObjectClass *klass,
> void *data)
> set_bit(DEVICE_CATEGORY_STORAGE, k->categories);
> k->bus_type = TYPE_SCSI_BUS;
> k->realize = scsi_qdev_realize;
> - k->unplug = scsi_qdev_unplug;
> k->unrealize = scsi_qdev_unrealize;
> k->props = scsi_props;
> }
> diff --git a/include/hw/scsi/scsi.h b/include/hw/scsi/scsi.h
> index 2e3a8f9..8a2581e 100644
> --- a/include/hw/scsi/scsi.h
> +++ b/include/hw/scsi/scsi.h
> @@ -138,8 +138,6 @@ struct SCSIBusInfo {
> void (*transfer_data)(SCSIRequest *req, uint32_t arg);
> void (*complete)(SCSIRequest *req, uint32_t arg, size_t resid);
> void (*cancel)(SCSIRequest *req);
> - void (*hotplug)(SCSIBus *bus, SCSIDevice *dev);
> - void (*hot_unplug)(SCSIBus *bus, SCSIDevice *dev);
> void (*change)(SCSIBus *bus, SCSIDevice *dev, SCSISense sense);
> QEMUSGList *(*get_sg_list)(SCSIRequest *req);
>
>
Reviewed-by: Paolo Bonzini <address@hidden>
- Re: [Qemu-devel] [PATCH 20/30] s390x: convert virtio-ccw to hotplug handler API, (continued)
- [Qemu-devel] [PATCH 21/30] scsi: make scsi_bus_new() assign hotplug controller, Igor Mammedov, 2014/09/24
- [Qemu-devel] [PATCH 22/30] scsi: convert pvscsi HBA to hotplug hander API, Igor Mammedov, 2014/09/24
- [Qemu-devel] [PATCH 23/30] scsi: convert virtio-scsi HBA to hotplug handler API, Igor Mammedov, 2014/09/24
- [Qemu-devel] [PATCH 24/30] scsi: cleanup not used anymore SCSIBusInfo{hotplug, hot_unplug} fields, Igor Mammedov, 2014/09/24
- Re: [Qemu-devel] [PATCH 24/30] scsi: cleanup not used anymore SCSIBusInfo{hotplug, hot_unplug} fields,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Igor Mammedov, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Paolo Bonzini, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Igor Mammedov, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Paolo Bonzini, 2014/09/24
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Gerd Hoffmann, 2014/09/25
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Igor Mammedov, 2014/09/25
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Gerd Hoffmann, 2014/09/25
- Re: [Qemu-devel] [PATCH 25/30] usb-bot: drop not needed "allow_hotplug = 0", Gerd Hoffmann, 2014/09/25
[Qemu-devel] [PATCH 26/30] usb-storage: make its storage SCSI bus hotpluggable explicitly, Igor Mammedov, 2014/09/24