Re: [Qemu-devel] [PATCH v2] vhost-user: start/stop all rings

2015-11-20 Thread Michael S. Tsirkin
On Fri, Nov 20, 2015 at 02:56:43PM +0200, Victor Kaplansky wrote: > On Mon, Nov 16, 2015 at 06:47:03PM +0200, Michael S. Tsirkin wrote: > > We are currently only sending VRING_ENABLE message for the first ring, > > that's wrong: we must start/stop them all. > > > > Reported-by: Victor Kaplansky

Re: [Qemu-devel] [PATCH v2] vhost-user: start/stop all rings

2015-11-20 Thread Victor Kaplansky
On Mon, Nov 16, 2015 at 06:47:03PM +0200, Michael S. Tsirkin wrote: > We are currently only sending VRING_ENABLE message for the first ring, > that's wrong: we must start/stop them all. > > Reported-by: Victor Kaplansky > Signed-off-by: Michael S. Tsirkin > --- > > Changes from v1: > typo

Re: [Qemu-devel] [PATCH v2] vhost-user: start/stop all rings

2015-11-16 Thread Michael S. Tsirkin
On Mon, Nov 16, 2015 at 06:47:03PM +0200, Michael S. Tsirkin wrote: > We are currently only sending VRING_ENABLE message for the first ring, > that's wrong: we must start/stop them all. > > Reported-by: Victor Kaplansky > Signed-off-by: Michael S. Tsirkin Note: this is on top of my tree. > --

[Qemu-devel] [PATCH v2] vhost-user: start/stop all rings

2015-11-16 Thread Michael S. Tsirkin
We are currently only sending VRING_ENABLE message for the first ring, that's wrong: we must start/stop them all. Reported-by: Victor Kaplansky Signed-off-by: Michael S. Tsirkin --- Changes from v1: typo fix hw/virtio/vhost-user.c | 17 +++-- 1 file changed, 11 insertions(+),