[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/10] usb-storage: migration support
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 10/10] usb-storage: migration support |
Date: |
Fri, 25 May 2012 12:11:10 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120430 Thunderbird/12.0.1 |
Il 25/05/2012 11:44, Gerd Hoffmann ha scritto:
> With all scsi migration support bits in place the
> final step is pretty simple ;)
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> hw/usb/dev-storage.c | 23 +++++++++++++++++++++--
> 1 files changed, 21 insertions(+), 2 deletions(-)
>
> diff --git a/hw/usb/dev-storage.c b/hw/usb/dev-storage.c
> index 1975d26..7646a77 100644
> --- a/hw/usb/dev-storage.c
> +++ b/hw/usb/dev-storage.c
> @@ -506,6 +506,17 @@ static void usb_msd_password_cb(void *opaque, int err)
> qdev_unplug(&s->dev.qdev, NULL);
> }
>
> +static void *usb_msd_load_request(QEMUFile *f, SCSIRequest *req)
> +{
> + MSDState *s = DO_UPCAST(MSDState, dev.qdev, req->bus->qbus.parent);
> +
> + /* nothing to load, just store req in our state struct */
> + assert(s->req == NULL);
> + scsi_req_ref(req);
> + s->req = req;
> + return NULL;
> +}
> +
> static const struct SCSIBusInfo usb_msd_scsi_info = {
> .tcq = false,
> .max_target = 0,
> @@ -513,7 +524,8 @@ static const struct SCSIBusInfo usb_msd_scsi_info = {
>
> .transfer_data = usb_msd_transfer_data,
> .complete = usb_msd_command_complete,
> - .cancel = usb_msd_request_cancelled
> + .cancel = usb_msd_request_cancelled,
> + .load_request = usb_msd_load_request,
> };
>
> static int usb_msd_initfn(USBDevice *dev)
> @@ -633,11 +645,18 @@ static USBDevice *usb_msd_init(USBBus *bus, const char
> *filename)
>
> static const VMStateDescription vmstate_usb_msd = {
> .name = "usb-storage",
> - .unmigratable = 1, /* FIXME: handle transactions which are in flight */
> .version_id = 1,
> .minimum_version_id = 1,
> .fields = (VMStateField []) {
> VMSTATE_USB_DEVICE(dev, MSDState),
> + VMSTATE_UINT32(mode, MSDState),
> + VMSTATE_UINT32(scsi_len, MSDState),
> + VMSTATE_UINT32(scsi_off, MSDState),
> + VMSTATE_UINT32(data_len, MSDState),
> + VMSTATE_UINT32(csw.sig, MSDState),
> + VMSTATE_UINT32(csw.tag, MSDState),
> + VMSTATE_UINT32(csw.residue, MSDState),
> + VMSTATE_UINT8(csw.status, MSDState),
> VMSTATE_END_OF_LIST()
> }
> };
Can usb-storage have multiple requests in flight (in principle, i.e.
according to the USB protocol)? If so, it may be better if you save
scsi_len/scsi_off in save_request/load_request. The migration format
will be forward-compatible.
Otherwise looks good!
Paolo
- [Qemu-devel] [PATCH 00/10] usb: uhci & usb-storage updates, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 03/10] uhci: make bandwidth tunable, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 04/10] uhci: fix trace format strings, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 09/10] usb-storage: add scsi_off, remove scsi_buf, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 05/10] uhci: zap uhci_pre_save, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 02/10] uhci: use bottom half, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 10/10] usb-storage: migration support, Gerd Hoffmann, 2012/05/25
- Re: [Qemu-devel] [PATCH 10/10] usb-storage: migration support,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 08/10] usb-storage: add usb_msd_packet_complete(), Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 01/10] uhci: fix bandwidth management, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 07/10] usb-storage: remove MSDState->residue, Gerd Hoffmann, 2012/05/25
- [Qemu-devel] [PATCH 06/10] scsi: prepare migration code for usb-storage support, Gerd Hoffmann, 2012/05/25