[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4] virtio-blk: set correct config size for the
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v4] virtio-blk: set correct config size for the host driver |
Date: |
Wed, 13 Feb 2019 16:00:17 +0800 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Tue, Feb 12, 2019 at 09:05:11PM -0500, Michael S. Tsirkin wrote:
> On Wed, Feb 13, 2019 at 09:48:57AM +0800, Changpeng Liu wrote:
> > Commit caa1ee43 "vhost-user-blk: add discard/write zeroes features
> > support" added fields to struct virtio_blk_config. This changes
> > the size of the config space and breaks migration from QEMU 3.1
> > and older:
> >
> > qemu-system-ppc64: get_pci_config_device: Bad config data: i=0x10 read: 41
> > device: 1 cmask: ff wmask: 80 w1cmask:0
> > qemu-system-ppc64: Failed to load PCIDevice:config
> > qemu-system-ppc64: Failed to load virtio-blk:virtio
> > qemu-system-ppc64: error while loading state for instance 0x0 of device
> > 'address@hidden:01.0/virtio-blk'
> > qemu-system-ppc64: load of migration failed: Invalid argument
> >
> > Since virtio-blk doesn't support the "discard" and "write zeroes"
> > features, it shouldn't even expose the associated fields in the
> > config space actually. Just include all fields up to num_queues to
> > match QEMU 3.1 and older.
> >
> > Signed-off-by: Changpeng Liu <address@hidden>
>
>
> Reviewed-by: Michael S. Tsirkin <address@hidden>
>
> Stefan, are you merging this?
Yes.
Stefan
signature.asc
Description: PGP signature
Re: [Qemu-devel] [PATCH v4] virtio-blk: set correct config size for the host driver, Greg Kurz, 2019/02/13