Re: [Qemu-devel] [PATCH v2] blockdev: reset werror/rerror on drive_del

[Kevin Wolf]

Am 05.06.2013 um 10:33 hat Stefan Hajnoczi geschrieben:
> Paolo Bonzini <address@hidden> suggested the following test case:
> 
> 1. Launch a guest and wait at the GRUB boot menu:
> 
>   qemu-system-x86_64 -enable-kvm -m 1024 \
>    -drive if=none,cache=none,file=test.img,id=foo,werror=stop,rerror=stop
>    -device virtio-blk-pci,drive=foo,id=virtio0,addr=4
> 
> 2. Hot unplug the device:
> 
>   (qemu) drive_del foo
> 
> 3. Select the first boot menu entry

Can we have a qtest or qemu-iotests version of this manual test case?

> Without this patch the guest pauses due to ENOMEDIUM.  The guest is
> stuck in a continuous pause loop since the I/O request is retried and
> fails immediately again when the guest is resumed.
> 
> With this patch the error is reported to the guest.
> 
> Note that this scenario actually happens sometimes during libvirt disk
> hot unplug, where device_del is followed by drive_del.  I/O may still be
> submitted to the drive after drive_del if the guest does not process the
> PCI hot unplug notification.
> 
> Reported-by: Dafna Ron <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>

Reviewed-by: Kevin Wolf <address@hidden>