[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] vhost-user-scsi: implement handle_output
From: |
Stefan Hajnoczi |
Subject: |
Re: [PATCH] vhost-user-scsi: implement handle_output |
Date: |
Mon, 21 Oct 2019 12:20:56 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Thu, Oct 17, 2019 at 04:39:44PM +0000, Felipe Franciosi wrote:
> Originally, vhost-user-scsi did not implement a handle_output callback
> as that didn't seem necessary. Turns out it is.
>
> Depending on which other devices are presented to a VM, SeaBIOS may
> decide to map vhost-user-scsi devices on the 64-bit range of the address
> space. As a result, SeaBIOS will kick VQs via the config space. Those
> land on Qemu (not the vhost backend) and are missed, causing the VM not
> to boot. This fixes the issue by getting Qemu to post the notification.
>
> Signed-off-by: Felipe Franciosi <address@hidden>
> ---
> hw/scsi/vhost-user-scsi.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/hw/scsi/vhost-user-scsi.c b/hw/scsi/vhost-user-scsi.c
> index 6a6c15dd32..13278ed151 100644
> --- a/hw/scsi/vhost-user-scsi.c
> +++ b/hw/scsi/vhost-user-scsi.c
> @@ -62,8 +62,9 @@ static void vhost_user_scsi_set_status(VirtIODevice *vdev,
> uint8_t status)
> }
> }
>
> -static void vhost_dummy_handle_output(VirtIODevice *vdev, VirtQueue *vq)
> +static void vhost_handle_output(VirtIODevice *vdev, VirtQueue *vq)
> {
> + event_notifier_set(virtio_queue_get_host_notifier(vq));
> }
It would be nice to fix this in hw/virtio/virtio.c:virtio_queue_notify()
so that all devices are automatically covered.
I'll send a patch and CC you.
Stefan
signature.asc
Description: PGP signature
- [PATCH] vhost-user-scsi: implement handle_output, Felipe Franciosi, 2019/10/17
- Re: [PATCH] vhost-user-scsi: implement handle_output, Yongji Xie, 2019/10/17
- Re: [PATCH] vhost-user-scsi: implement handle_output, Felipe Franciosi, 2019/10/18
- Re: [PATCH] vhost-user-scsi: implement handle_output, Yongji Xie, 2019/10/21
- Re: [PATCH] vhost-user-scsi: implement handle_output, Felipe Franciosi, 2019/10/21
- RE: [PATCH] vhost-user-scsi: implement handle_output, Liu, Changpeng, 2019/10/21
- Re: [PATCH] vhost-user-scsi: implement handle_output, Yongji Xie, 2019/10/21
- Re: [PATCH] vhost-user-scsi: implement handle_output, Stefan Hajnoczi, 2019/10/22
- Re: [PATCH] vhost-user-scsi: implement handle_output, Yongji Xie, 2019/10/21
Re: [PATCH] vhost-user-scsi: implement handle_output,
Stefan Hajnoczi <=