[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/5] vhost: enable for all targets
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH 0/5] vhost: enable for all targets |
Date: |
Mon, 26 Nov 2018 17:43:28 -0500 |
On Mon, Nov 26, 2018 at 02:20:38PM +0100, Paolo Bonzini wrote:
> vhost does not have to be supported only if KVM is present, in fact
> vhost-user does not even need any kind of kernel support. This series
> changes this. The rationale is that, when vhost-user-test.c will be
> converted to qgraph, it will be able to test vhost-user support
> for virtio-mmio backend even on x86.
The reason for limiting it to KVM was very simple:
it has the same set of problems with ordering
as mttcg.
So I guess it's fine but I think we must then limit it
to when tcg emits fence instructions.
Otherwise we'll get subtle race conditions.
> Paolo Bonzini (5):
> vhost-net: move stubs to a separate file
> vhost-net-user: add stubs for when no virtio-net device is present
> vhost: restrict Linux dependency to kernel vhost
> vhost-net: compile it on all targets that have virtio-net.
> vhost-net: revamp configure logic
>
> backends/Makefile.objs | 5 +--
> configure | 101
> ++++++++++++++++++++++++++++-----------------
> default-configs/virtio.mak | 4 +-
> hw/net/Makefile.objs | 4 +-
> hw/net/vhost_net-stub.c | 95 ++++++++++++++++++++++++++++++++++++++++++
> hw/net/vhost_net.c | 78 ++--------------------------------
> hw/virtio/Makefile.objs | 5 ++-
> hw/virtio/vhost-backend.c | 11 ++++-
> include/exec/poison.h | 1 -
> net/Makefile.objs | 4 +-
> net/net.c | 2 +-
> net/vhost-user-stub.c | 23 +++++++++++
> tests/Makefile.include | 5 +--
> 13 files changed, 209 insertions(+), 129 deletions(-)
> create mode 100644 hw/net/vhost_net-stub.c
> create mode 100644 net/vhost-user-stub.c
>
> --
> 1.8.3.1