We won't reference var 'i' anymore; there is no need to assign a value
to it.
cc: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Yuanhan Liu yuanhan@linux.intel.com
---
hw/virtio/vhost.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index 2712c6f
On Wed, Oct 21, 2015 at 01:39:11PM +0300, Michael S. Tsirkin wrote:
> On Wed, Oct 21, 2015 at 05:07:18PM +0800, Yuanhan Liu wrote:
> > Send VHOST_USER_SET_VRING_ENABLE at start/stop when multiple queue
> > is negotiated, to inform the backend that we are ready or not.
>
> OK
They are VHOST_USER_XXX instead of VHOST_XXX messages.
Also, add VHOST_USER_GET_QUEUE_NUM to the section that
requries replies.
Cc: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 21 +++--
later.
Patch 2 is a minor fix or request naming in vhost-user spec
Patch 3 adds the vhost-user mq unit test
---
Yuanhan Liu (3):
vhost: rename VHOST_RESET_DEVICE back to VHOST_RESET_OWNER
doc: vhost-user: request naming fix
vhost-user-test: add multiple queue test
docs/specs/vhost-user.txt
Setting VHOST_USER_PROTOCOL_F_MQ protocol feature bit to claim that we
support MQ feature, and simply assume we support 2 queue pairs at most.
Cc: Michael S. Tsirkin <m...@redhat.com>
Cc: Jason Wang <jasow...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
It turned out that it breaks stuff (VHOST_RESET_OWNER is not defined),
so revert it:
http://lists.nongnu.org/archive/html/qemu-devel/2015-10/msg00949.html
CC: "Michael S. Tsirkin" <m...@redhat.com>
Reported-by: Paolo Bonzini <pbonz...@redhat.com>
Signed-off-by
On Wed, Oct 21, 2015 at 05:13:49PM +0300, Michael S. Tsirkin wrote:
> On Wed, Oct 21, 2015 at 09:04:17PM +0800, Yuanhan Liu wrote:
> > On Wed, Oct 21, 2015 at 01:40:59PM +0300, Michael S. Tsirkin wrote:
> > > On Wed, Oct 21, 2015 at 05:07:14PM +0800, Yuanhan Liu wrote:
> >
On Wed, Oct 21, 2015 at 05:11:44PM +0300, Michael S. Tsirkin wrote:
> On Wed, Oct 21, 2015 at 09:43:16PM +0800, Yuanhan Liu wrote:
> > On Wed, Oct 21, 2015 at 01:39:11PM +0300, Michael S. Tsirkin wrote:
> > > On Wed, Oct 21, 2015 at 05:07:18PM +0800, Yuanhan Liu w
On Wed, Oct 21, 2015 at 01:40:59PM +0300, Michael S. Tsirkin wrote:
> On Wed, Oct 21, 2015 at 05:07:14PM +0800, Yuanhan Liu wrote:
> > This reverts commit d1f8b30ec8dde0318fd1b98d24a64926feae9625.
> >
> > It turned out that it breaks stuff, so revert it:
> >
>
This patch basically reverts commit d1f8b30e.
It turned out that it breaks stuff, so revert it:
http://lists.nongnu.org/archive/html/qemu-devel/2015-10/msg00949.html
CC: "Michael S. Tsirkin" <m...@redhat.com>
Reported-by: Paolo Bonzini <pbonz...@redhat.com>
Sign
.
---
Yuanhan Liu (5):
vhost: rename RESET_DEVICE backto RESET_OWNER
vhost: reset vhost net when virtio_net_reset happens
vhost: introduce vhost_set/get_protocol_features callbacks
vhost: send SET_PROTOCOL_FEATURES at start
vhost: send SET_VRING_ENABLE at start/stop
docs/specs/vhost-user.txt
So that the backend can restore the protocol features after a reset.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost.c | 8
1 file changed, 8 insertions(+)
diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index de29968..be48511 100644
--- a/hw/
Send SET_VRING_ENABLE at start/stop, to give the backend
an explicit sign of our state.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c
index be48511..0e956b5
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-user.c| 7 +++
include/hw/virtio/vhost-backend.h | 6 ++
2 files changed, 13 insertions(+)
diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
index 4766f98..2d8bdbd 100644
--- a/hw/
When a virtio net driver is unloaded (unbind), virtio_net_reset happens.
For vhost net, we should also send a message (RESET_OWNER) to the backend
to do some proper reset settings.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/net/vhost_net.c | 20 ++
On Fri, Nov 06, 2015 at 10:01:58AM +, Peter Maydell wrote:
> On 6 November 2015 at 01:34, Yuanhan Liu <yuanhan@linux.intel.com> wrote:
> > On Thu, Nov 05, 2015 at 11:42:15AM +, Peter Maydell wrote:
> >> On 3 October 2015 at 17:33, Michael S. Tsirkin
First of all, RESET_OWNER message is sent incorrectly, as it's sent
before GET_VRING_BASE. And the reset message would let the later call
get nothing correct.
And, sending SET_VRING_ENABLE at stop, which has already been done,
makes more sense than RESET_OWNER.
Signed-off-by: Yuanhan Liu
But not depend on PROTOCOL_F_MQ feature bit. So that we could use
SET_VRING_ENABLE to sign the backend on stop, even if MQ is disabled.
That's reasonable, since we will have one queue pair at least.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-user.c | 2
On Thu, Nov 12, 2015 at 03:33:41PM +0200, Michael S. Tsirkin wrote:
> On Wed, Nov 11, 2015 at 09:24:36PM +0800, Yuanhan Liu wrote:
> >
> > Patch 1 rename RESET_DEVICE back to RESET_OWNER
> >
> > Patch 2 introduced a new function: vhost_net_reset(), which is invoked
>
On Thu, Nov 12, 2015 at 04:44:19PM +0200, Michael S. Tsirkin wrote:
> On Thu, Nov 12, 2015 at 10:08:15PM +0800, Yuanhan Liu wrote:
> > On Thu, Nov 12, 2015 at 03:33:41PM +0200, Michael S. Tsirkin wrote:
> > > On Wed, Nov 11, 2015 at 09:24:36PM +0800, Yuanhan Liu wrote:
> &
On Fri, Nov 13, 2015 at 12:24:52PM +0200, Michael S. Tsirkin wrote:
> On Fri, Nov 13, 2015 at 10:03:29AM +0800, Yuanhan Liu wrote:
> > On Thu, Nov 12, 2015 at 04:44:19PM +0200, Michael S. Tsirkin wrote:
> > > On Thu, Nov 12, 2015 at 10:08:15PM +0800, Yuanhan Liu wrote:
> >
Thanks.
Reviewed-by: Yuanhan Liu <yuanhan@linux.intel.com>
--yliu
On Wed, Nov 11, 2015 at 02:53:29PM +0800, Wen Congyang wrote:
> Signed-off-by: Wen Congyang <we...@cn.fujitsu.com>
> ---
> net/vhost-user.c | 2 +-
> 1 file changed, 1 insertion(+),
On Thu, Nov 05, 2015 at 11:42:15AM +, Peter Maydell wrote:
> On 3 October 2015 at 17:33, Michael S. Tsirkin <m...@redhat.com> wrote:
> > On Fri, Oct 02, 2015 at 06:18:51PM +0200, Paolo Bonzini wrote:
> >>
> >>
> >> On 24/09/2015 15:20, Michael
On Sat, Oct 03, 2015 at 07:33:16PM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 02, 2015 at 06:18:51PM +0200, Paolo Bonzini wrote:
> >
> >
> > On 24/09/2015 15:20, Michael S. Tsirkin wrote:
> > > From: Yuanhan Liu <yuanhan@linux.intel.com
On Fri, Oct 16, 2015 at 10:20:08AM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 16, 2015 at 03:19:01PM +0800, Yuanhan Liu wrote:
> > Setting VHOST_USER_PROTOCOL_F_MQ protocol feature bit to claim that we
> > support MQ feature, and simply assume we support 2 queue pairs at
On Fri, Oct 16, 2015 at 10:21:40AM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 16, 2015 at 03:18:59PM +0800, Yuanhan Liu wrote:
> > This reverts commit d1f8b30ec8dde0318fd1b98d24a64926feae9625.
> >
> > It turned out that it breaks stuff, so revert it:
> >
>
On Fri, Oct 16, 2015 at 10:47:49AM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 16, 2015 at 03:32:56PM +0800, Yuanhan Liu wrote:
> > On Fri, Oct 16, 2015 at 10:21:40AM +0300, Michael S. Tsirkin wrote:
> > > On Fri, Oct 16, 2015 at 03:18:59PM +0800, Yuanhan Liu wrote:
> >
They are VHOST_USER_XXX instead of VHOST_XXX messages.
Also, add VHOST_USER_GET_QUEUE_NUM to the section that
requries replies.
Cc: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 17 +
1
hat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 4 ++--
hw/net/vhost_net.c | 2 +-
hw/virtio/vhost-user.c | 8
linux-headers/linux/vhost.h | 2 +-
tests/vhost-user-test.c | 2 +-
5 files changed, 9 insertions(+), 9 del
Setting VHOST_USER_PROTOCOL_F_MQ protocol feature bit to claim that we
support MQ feature, and simply assume we support 2 queue pairs at most.
Cc: Michael S. Tsirkin <m...@redhat.com>
Cc: Jason Wang <jasow...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
On Fri, Oct 16, 2015 at 11:15:23AM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 16, 2015 at 04:04:09PM +0800, Yuanhan Liu wrote:
> > On Fri, Oct 16, 2015 at 10:47:49AM +0300, Michael S. Tsirkin wrote:
> > > On Fri, Oct 16, 2015 at 03:32:56PM +0800, Yuanhan Liu wrote:
> >
On Wed, Oct 21, 2015 at 12:15:52PM +0300, Michael S. Tsirkin wrote:
> On Wed, Oct 21, 2015 at 04:55:16PM +0800, Yuanhan Liu wrote:
> > On Fri, Oct 16, 2015 at 10:47:49AM +0300, Michael S. Tsirkin wrote:
> > > On Fri, Oct 16, 2015 at 03:32:56PM +0800, Yuanhan Liu wrote:
> >
hat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 4 ++--
hw/net/vhost_net.c | 2 +-
hw/virtio/vhost-user.c | 8
linux-headers/linux/vhost.h | 2 +-
tests/vhost-user-test.c | 2 +-
5 files changed, 9 insertions(+), 9 del
They are VHOST_USER_XXX instead of VHOST_XXX messages.
Also, add VHOST_USER_GET_QUEUE_NUM to the section that
requries replies.
Cc: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 17 +
1
Setting VHOST_USER_PROTOCOL_F_MQ protocol feature bit to claim that we
support MQ feature, and simply assume we support 0xff queue pairs at most.
Cc: Michael S. Tsirkin <m...@redhat.com>
Cc: Jason Wang <jasow...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.co
On Fri, Oct 16, 2015 at 10:47:49AM +0300, Michael S. Tsirkin wrote:
> On Fri, Oct 16, 2015 at 03:32:56PM +0800, Yuanhan Liu wrote:
> > On Fri, Oct 16, 2015 at 10:21:40AM +0300, Michael S. Tsirkin wrote:
> > > On Fri, Oct 16, 2015 at 03:18:59PM +0800, Yuanhan Liu wrote:
> >
kin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/net/vhost_net.c | 7 ---
1 file changed, 7 deletions(-)
diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
index 804f5c9..95da5f8 100644
--- a/hw/net/vhost_net.c
+++ b/hw/net/vhost_net.c
@@ -293,1
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-user.c | 1 -
hw/virtio/vhost.c | 18 ++
2 files changed, 18 insertions(+), 1 deletion(-)
diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
index 12a9104..6532a73 100644
--- a/hw/
On Mon, Aug 31, 2015 at 02:30:06PM +0300, Michael S. Tsirkin wrote:
[snip..]
> > Then how about the other case:
> > When startup the vhost backend, user specify 8 queues, like:
> > Vhost-user ... -queues 8 # Y=8
> > When startup the guest, user specify 4 queues, like:
> > Qemu-kvm ...
On Thu, Aug 13, 2015 at 12:18:38PM +0300, Michael S. Tsirkin wrote:
> On Wed, Aug 12, 2015 at 02:25:41PM +0800, Ouyang Changchun wrote:
> > Based on patch by Nikolay Nikolaev:
> > Vhost-user will implement the multi queue support in a similar way
> > to what vhost already has - a separate thread
On Thu, Aug 13, 2015 at 12:22:50PM +0300, Michael S. Tsirkin wrote:
> On Wed, Aug 12, 2015 at 02:25:42PM +0800, Ouyang Changchun wrote:
> > This patch is based on top of "vhost-user: protocol updates" series
> > proposed earlier by Michael S. Tsirkin.
> >
> > Use new message
On Tue, Sep 01, 2015 at 01:09:15PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 01, 2015 at 05:16:31PM +0800, Yuanhan Liu wrote:
> > On Thu, Aug 13, 2015 at 12:22:50PM +0300, Michael S. Tsirkin wrote:
> > > On Wed, Aug 12, 2015 at 02:25:42PM +0800, Ouyang Changchun wrote:
On Tue, Sep 01, 2015 at 01:07:11PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 01, 2015 at 05:13:50PM +0800, Yuanhan Liu wrote:
> > On Thu, Aug 13, 2015 at 12:18:38PM +0300, Michael S. Tsirkin wrote:
> > > On Wed, Aug 12, 2015 at 02:25:41PM +0800, Ouyang Changchun wrote:
>
On Tue, Sep 01, 2015 at 12:41:49PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 01, 2015 at 05:20:06PM +0800, Yuanhan Liu wrote:
> > On Mon, Aug 31, 2015 at 02:30:06PM +0300, Michael S. Tsirkin wrote:
> > [snip..]
> > > > Then how about the other case:
> > &
Quote from Michael:
We really should rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE.
Suggested-by: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 4 ++--
hw/net/vhost_net.c | 2 +-
hw/
From: "Michael S. Tsirkin" <m...@redhat.com>
Support a separate bitmask for vhost-user protocol features,
and messages to get/set protocol features.
Invoke them at init.
No features are defined yet.
v2: leverage vhost_user_call for request handling -- Yuanhan Liu
Signed-o
igned-off-by: Nikolay Nikolaev <n.nikol...@virtualopensystems.com>
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 13 +
hw/virtio/vhost-user.c| 31 ++-
include/net/net.h
is
done in the following patch.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 11 +++
hw/net/vhost_net.c| 7 +++
hw/virtio/vhost-user.c| 14 +-
include/hw/virtio/vhost.h | 1 +
include/net/vhost_net.h | 1 +
5
ble/enable a specific virt queue.
Michael S. Tsirkin (1):
vhost-user: add protocol feature negotiation
Ouyang Changchun (1):
vhost-user: add multiple queue support
Yuanhan Liu (4):
vhost-user: use VHOST_USER_XXX macro for switch statement
vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVI
So that we could let vhost_user_call to handle extented requests,
such as VHOST_USER_GET/SET_PROTOCOL_FEATURES, instead of invoking
vhost_user_read/write and constructing the msg again by ourself.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-user.
number of virt queue ranging from 1 to max virt
queue pair.
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 10 ++
hw/net/vhost_net.c| 18 ++
h
. Here is a quick set to 0 at net_vhost_user_init() stage, and it
will be set properly soon in the next patch.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/net/vhost_net.c | 16 +++-
net/vhost-user.c | 1 +
2 files changed, 8 insertions(+), 9 deletions(-)
diff
On Tue, Sep 08, 2015 at 03:22:30PM -0600, Eric Blake wrote:
> On 09/08/2015 01:38 AM, Yuanhan Liu wrote:
> > From: Ouyang Changchun <changchun.ouy...@intel.com>
> >
> > This patch is initially based a patch from Nikolay Nikolaev.
> >
> > Here is the latest
On Thu, Sep 10, 2015 at 02:54:02PM +0800, Jason Wang wrote:
>
>
> On 09/10/2015 02:18 PM, Yuanhan Liu wrote:
> > On Thu, Sep 10, 2015 at 01:52:30PM +0800, Jason Wang wrote:
> >>
> >> On 09/10/2015 01:17 PM, Yuanhan Liu wrote:
> >>> On Thu, Sep 1
On Wed, Sep 09, 2015 at 08:05:11AM +, Ouyang, Changchun wrote:
>
>
> > -Original Message-
> > From: Yuanhan Liu [mailto:yuanhan@linux.intel.com]
> > Sent: Tuesday, September 8, 2015 3:39 PM
> > To: qemu-devel@nongnu.org
> > Cc: m...@redhat
On Wed, Sep 09, 2015 at 03:18:53PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 08, 2015 at 03:38:46PM +0800, Yuanhan Liu wrote:
> > From: Ouyang Changchun <changchun.ouy...@intel.com>
> >
> > This patch is initially based a patch from Nikolay Nikolaev.
> >
On Wed, Sep 09, 2015 at 01:45:50PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 08, 2015 at 03:38:47PM +0800, Yuanhan Liu wrote:
> > From: Changchun Ouyang <changchun.ouy...@intel.com>
> >
> > Add a new message, VHOST_USER_SET_VRING_FLAG, to enable and disable
>
On Wed, Sep 16, 2015 at 11:12:08AM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 15, 2015 at 03:10:27PM +0800, Yuanhan Liu wrote:
> > Hi,
> >
> > Here is the updated patch set for enabling vhost-user multiple queue.
>
> Will review, thanks.
Thank you!
> BTW, p
On Wed, Sep 16, 2015 at 11:10:35AM +0300, Michael S. Tsirkin wrote:
> On Wed, Sep 16, 2015 at 10:06:56AM +0800, Yuanhan Liu wrote:
> > If not, I'd like to put them into the commit log, as putting it
> > outside the commit log gives unnecessary extra burden to patch
> > autho
On Wed, Sep 16, 2015 at 02:39:06PM +0300, Marcel Apfelbaum wrote:
> On 09/15/2015 10:10 AM, Yuanhan Liu wrote:
> >Hi,
> >
> >Here is the updated patch set for enabling vhost-user multiple queue.
>
> Hi,
>
> I think this series will not work with the current
On Tue, Sep 15, 2015 at 09:02:07AM -0600, Eric Blake wrote:
> On 09/15/2015 01:10 AM, Yuanhan Liu wrote:
> > From: Changchun Ouyang <changchun.ouy...@intel.com>
> >
> > This patch is initially based a patch from Nikolay Nikolaev.
> >
> > Here is the latest
On Tue, Sep 15, 2015 at 08:56:05AM -0600, Eric Blake wrote:
> On 09/15/2015 01:10 AM, Yuanhan Liu wrote:
> > From: Changchun Ouyang <changchun.ouy...@intel.com>
> >
> > This patch is initially based a patch from Nikolay Nikolaev.
>
> Subject line confusion -
On Wed, Sep 16, 2015 at 10:06:56AM +0800, Yuanhan Liu wrote:
> On Tue, Sep 15, 2015 at 09:02:07AM -0600, Eric Blake wrote:
> > On 09/15/2015 01:10 AM, Yuanhan Liu wrote:
> > > From: Changchun Ouyang <changchun.ouy...@intel.com>
> > >
> > > This pat
On Wed, Sep 16, 2015 at 08:15:59AM -0600, Eric Blake wrote:
> On 09/15/2015 08:06 PM, Yuanhan Liu wrote:
>
> >> Up to here is mostly fine for the commit message. Meanwhile...
> >>
> >>> v9: per suggested by Jason Wang, we could invoke qemu_chr_add_handle
On Mon, Sep 14, 2015 at 06:00:41PM +0800, Jason Wang wrote:
>
>
> On 09/08/2015 03:38 PM, Yuanhan Liu wrote:
> > From: Ouyang Changchun <changchun.ouy...@intel.com>
> >
> > This patch is initially based a patch from Nikolay Nikolaev.
> >
> > Her
is
done in the following patch.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 11 +++
hw/net/vhost_net.c| 7 +++
hw/virtio/vhost-user.c| 15 ++-
include/hw/virtio/vhost.h | 1 +
include/net/vhost_net.h | 1 +
5
_net_init() based on the
value from net->nc->queue_index.
Signed-off-by: Nikolay Nikolaev <n.nikol...@virtualopensystems.com>
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.tx
So that we could let vhost_user_call to handle extented requests,
such as VHOST_USER_GET/SET_PROTOCOL_FEATURES, instead of invoking
vhost_user_read/write and constructing the msg again by ourself.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-user.
--yliu
---
Changchun Ouyang (2):
vhost-user: add multiple queue support
vhost-user: add a new message to disable/enable a specific virt queue.
Michael S. Tsirkin (1):
vhost-user: add protocol feature negotiation
Yuanhan Liu (4):
vhost-user: use VHOST_USER_XXX macro for switch stateme
Quote from Michael:
We really should rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE.
Suggested-by: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 4 ++--
hw/net/vhost_net.c | 2 +-
hw/
air pairs[];
} VhostUserState;
v8: set net->dev.vq_index correctly inside vhost_net_init() based on the
value from net->nc->queue_index.
Signed-off-by: Nikolay Nikolaev <n.nikol...@virtualopensystems.com>
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Li
number of virt queue ranging from 1 to max virt
queue pair.
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 12 +++-
hw/net/vhost_net.c| 18 +++
at.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-backend.c | 10 +-
hw/virtio/vhost-user.c| 12 ++--
hw/virtio/vhost.c | 15 ++-
include/hw/virtio/vhost-backend.h | 2 ++
4 files changed, 27 in
From: "Michael S. Tsirkin" <m...@redhat.com>
Support a separate bitmask for vhost-user protocol features,
and messages to get/set protocol features.
Invoke them at init.
No features are defined yet.
v2: leverage vhost_user_call for request handling -- Yuanhan Liu
Signed-o
On Thu, Sep 17, 2015 at 03:12:55PM +0800, Jason Wang wrote:
>
>
> On 09/15/2015 03:10 PM, Yuanhan Liu wrote:
> > From: "Michael S. Tsirkin" <m...@redhat.com>
> >
> > Support a separate bitmask for vhost-user protocol features,
> > and messages
On Thu, Sep 17, 2015 at 03:23:55PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 15, 2015 at 03:10:27PM +0800, Yuanhan Liu wrote:
> > Hi,
> >
> > Here is the updated patch set for enabling vhost-user multiple queue.
> > This patch set introduces
On Thu, Sep 10, 2015 at 12:46:00PM +0800, Jason Wang wrote:
>
>
> On 09/10/2015 11:57 AM, Yuanhan Liu wrote:
> > On Thu, Sep 10, 2015 at 11:14:27AM +0800, Jason Wang wrote:
> >>
> >> On 09/08/2015 03:38 PM, Yuanhan Liu wrote:
> >>>
On Thu, Sep 10, 2015 at 11:14:27AM +0800, Jason Wang wrote:
>
>
> On 09/08/2015 03:38 PM, Yuanhan Liu wrote:
> > So that we could use the `vq_index' as well in the vhost_net_init
> > stage, which is required when adding vhost-user multiple-queue support,
> >
On Thu, Sep 10, 2015 at 01:52:30PM +0800, Jason Wang wrote:
>
>
> On 09/10/2015 01:17 PM, Yuanhan Liu wrote:
> > On Thu, Sep 10, 2015 at 12:46:00PM +0800, Jason Wang wrote:
> >> >
> >> >
> >> > On 09/10/2015 11:57 AM, Yuanhan Liu wrote:
> &
On Wed, Sep 09, 2015 at 01:45:50PM +0300, Michael S. Tsirkin wrote:
> On Tue, Sep 08, 2015 at 03:38:47PM +0800, Yuanhan Liu wrote:
[snip ]
> > diff --git a/hw/virtio/vhost-user.c b/hw/virtio/vhost-user.c
> > index 11e46b5..ca6f7fa 100644
> > --- a/hw/virtio/vhost-user.
On Thu, Sep 10, 2015 at 02:54:02PM +0800, Jason Wang wrote:
>
>
> On 09/10/2015 02:18 PM, Yuanhan Liu wrote:
> > On Thu, Sep 10, 2015 at 01:52:30PM +0800, Jason Wang wrote:
> >>
> >> On 09/10/2015 01:17 PM, Yuanhan Liu wrote:
> >>> On Thu, Sep 1
Quote from Michael:
We really should rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE.
Suggested-by: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 4 ++--
hw/net/vhost_net.c | 2 +-
hw/
er: add protocol feature negotiation
Yuanhan Liu (3):
vhost-user: use VHOST_USER_XXX macro for switch statement
vhost: rename VHOST_RESET_OWNER to VHOST_RESET_DEVICE
vhost-user: add VHOST_USER_GET_QUEUE_NUM message
docs/specs/vhost-user.txt | 75 +++-
hw
number of virt queue ranging from 1 to max virt
queue pair.
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 12 +++-
hw/net/vhost_net.c| 18 +++
From: "Michael S. Tsirkin" <m...@redhat.com>
Support a separate bitmask for vhost-user protocol features,
and messages to get/set protocol features.
Invoke them at init.
No features are defined yet.
v2: leverage vhost_user_call for request handling -- Yuanhan Liu
Signed-o
So that we could let vhost_user_call to handle extented requests,
such as VHOST_USER_GET/SET_PROTOCOL_FEATURES, instead of invoking
vhost_user_read/write and constructing the msg again by ourself.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
hw/virtio/vhost-user.
air pairs[];
} VhostUserState;
v8: set net->dev.vq_index correctly inside vhost_net_init() based on the
value from net->nc->queue_index.
Signed-off-by: Nikolay Nikolaev <n.nikol...@virtualopensystems.com>
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Li
is
done in the following patch.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
---
docs/specs/vhost-user.txt | 11 +++
hw/net/vhost_net.c| 7 +++
hw/virtio/vhost-user.c| 15 ++-
include/hw/virtio/vhost.h | 1 +
include/net/vhost_net.h | 1 +
5
On Thu, Sep 24, 2015 at 01:34:31PM +0800, Jason Wang wrote:
>
>
> Some nitpicks and comments. If you plan to send another version, please
> consider to fix them.
I will, but I'd like to hold a while before getting more comments from
Michael; I don't want to repost a whole new version too often
On Thu, Sep 24, 2015 at 01:13:24PM +0300, Marcel Apfelbaum wrote:
> On 09/23/2015 07:19 AM, Yuanhan Liu wrote:
> >From: "Michael S. Tsirkin" <m...@redhat.com>
> >
> >Support a separate bitmask for vhost-user protocol features,
> >and messages to ge
On Thu, Sep 24, 2015 at 01:13:24PM +0300, Marcel Apfelbaum wrote:
> >diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c
> >index 1d76b94..9d32d76 100644
> >--- a/hw/net/vhost_net.c
> >+++ b/hw/net/vhost_net.c
> >@@ -152,8 +152,10 @@ struct vhost_net *vhost_net_init(VhostNetOptions
> >*options)
at.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
Reviewed-by: Jason Wang <jasow...@redhat.com>
Tested-by: Marcel Apfelbaum <mar...@redhat.com>
---
hw/virtio/vhost-backend.c | 10 +-
hw/virtio/vhost-user.c| 12 ++--
hw/virtio/v
UE_NUM
For above messages, we simply ignore them when they are not sent the first
time.
Signed-off-by: Nikolay Nikolaev <n.nikol...@virtualopensystems.com>
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
Review
ch is also suggested by Jason Wang.
Thanks.
--yliu
---
Changchun Ouyang (2):
vhost-user: add multiple queue support
vhost-user: add a new message to disable/enable a specific virt queue.
Michael S. Tsirkin (1):
vhost-user: add protocol feature negotiation
Yuanhan Liu (4):
vhost
So that we could let vhost_user_call to handle extented requests,
such as VHOST_USER_GET/SET_PROTOCOL_FEATURES, instead of invoking
vhost_user_read/write and constructing the msg again by ourself.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
Reviewed-by: Marcel Apfelbau
eue
or something else.
Suggested-by: Michael S. Tsirkin <m...@redhat.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
Reviewed-by: Marcel Apfelbaum <mar...@redhat.com>
Tested-by: Marcel Apfelbaum <mar...@redhat.com>
---
docs/specs/vhost-user.txt | 4 ++--
hw/net
number of virt queue ranging from 1 to max virt
queue pair.
Signed-off-by: Changchun Ouyang <changchun.ouy...@intel.com>
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
Reviewed-by: Jason Wang <jasow...@redhat.com>
Tested-by: Marcel Apfelbaum <mar...@redhat.com>
is
done in the following patch.
Signed-off-by: Yuanhan Liu <yuanhan@linux.intel.com>
Reviewed-by: Marcel Apfelbaum <mar...@redhat.com>
Tested-by: Marcel Apfelbaum <mar...@redhat.com>
---
v12: mark VHOST_USER_GET_QUEUE_NUM as a message need reply.
---
docs/specs/v
From: "Michael S. Tsirkin" <m...@redhat.com>
Support a separate bitmask for vhost-user protocol features,
and messages to get/set protocol features.
Invoke them at init.
No features are defined yet.
[ leverage vhost_user_call for request handling -- Yuanhan Liu ]
Signed-o
On Tue, Sep 22, 2015 at 03:52:49PM -0300, Eduardo Habkost wrote:
> On Fri, Sep 18, 2015 at 10:58:43PM +0800, Yuanhan Liu wrote:
> [...]
> > diff --git a/net/vhost-user.c b/net/vhost-user.c
> > index 93dcecd..4fa3d64 100644
> > --- a/net/vhost-user.c
&g
1 - 100 of 170 matches
Mail list logo