Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-13 Thread Yuanhan Liu
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: > > > > On Thu, Nov 12, 2015

Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-13 Thread Michael S. Tsirkin
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: > > > On Thu, Nov 12, 2015 at 03:33:41PM +0200, Michael S. Tsirkin wrote: > > > > On Wed, Nov 11, 2015

Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-12 Thread Yuanhan Liu
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: > > > > > > > > Patch 1 rena

Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-12 Thread Michael S. Tsirkin
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: > > > > > > Patch 1 rename RESET_DEVICE back to RESET_OWNER > > > > > > Patch 2 introduced a new func

Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-12 Thread Yuanhan Liu
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 > > when reset happens,

Re: [Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-12 Thread Michael S. Tsirkin
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 > when reset happens, say, driver is unloaded (unbinded). > > Michael suggested to do that o

[Qemu-devel] [PATCH v4 0/5] handle vhost reset/start/stop correctly

2015-11-11 Thread Yuanhan Liu
Patch 1 rename RESET_DEVICE back to RESET_OWNER Patch 2 introduced a new function: vhost_net_reset(), which is invoked when reset happens, say, driver is unloaded (unbinded). Michael suggested to do that only when MQ is not negotiated. However, reset happens no matter MQ