[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 21/21] vhost-scsi: create a vhost-scsi-common abs
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PULL 21/21] vhost-scsi: create a vhost-scsi-common abstraction |
Date: |
Tue, 30 May 2017 16:16:23 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0 |
On 30/05/2017 16:06, Felipe Franciosi wrote:
>> This breaks the mingw build:
>>
>> hw/scsi/vhost-scsi-common.c:19:25: fatal error: linux/vhost.h: No such file
>> or directory
>> #include <linux/vhost.h>
>> ^
>> compilation terminated.
> Thanks for the report.
>
> Paolo: I know you wanted this enabled by default whenever virtio was built,
> but maybe we should stick to a configure switch live vhost-net, vhost-scsi
> and vhost-vsock?
>
> Let me know what you think and I'll send an extra patch for it straight away.
This was my mistake. Your patch broke s390, and when I fixed it I
copied the wrong setting:
default-configs/pci.mak:CONFIG_VHOST_USER_SCSI=$(CONFIG_POSIX)
default-configs/s390x-softmmu.mak:CONFIG_VHOST_USER_SCSI=y
Unfortunately, the mingw docker test doesn't build all targets, so I
missed it.
I would like vhost-user to be available on non-Linux POSIX, but this is
not immediately possible because vhost-backend.c includes the kernel
backend. So we'll have to change CONFIG_POSIX to CONFIG_LINUX too.
Paolo
- [Qemu-devel] [PULL 14/21] Fix the -accel parameter and the documentation for 'hax', (continued)
- [Qemu-devel] [PULL 14/21] Fix the -accel parameter and the documentation for 'hax', Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 15/21] MAINTAINERS: Add "R:" tag for self-appointed reviewers, Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 16/21] get_maintainer: Teach get_maintainer.pl about the new "R:" tag, Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 19/21] get_maintainer: add subsystem to reviewer output, Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 17/21] get_maintainer: it's '--pattern-depth', not '-pattern-depth', Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 20/21] libvhost-user: replace vasprintf() to fix build, Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 18/21] get_maintainer: --r (list reviewer) is on by default, Paolo Bonzini, 2017/05/05
- [Qemu-devel] [PULL 21/21] vhost-scsi: create a vhost-scsi-common abstraction, Paolo Bonzini, 2017/05/05
Re: [Qemu-devel] [PULL 00/21] Misc patches for 2017-05-05, Stefan Hajnoczi, 2017/05/08
Re: [Qemu-devel] [PULL 00/21] Misc patches for 2017-05-05, Stefan Hajnoczi, 2017/05/30