Re: [PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-13 Thread Paolo Bonzini
Il 12/03/2013 10:55, Michael S. Tsirkin ha scritto: > > Why? Why should qemu prevent from improving the driver and the benefit > > to the other user of the driver? Kvm tool is already using it since last > > Aug. This adds the missing disk hotplug support to kvm tool. > > Basically we shouldn't ad

Re: [PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-12 Thread Asias He
On Tue, Mar 12, 2013 at 11:55:14AM +0200, Michael S. Tsirkin wrote: > On Tue, Mar 12, 2013 at 10:31:09AM +0800, Asias He wrote: > > On Mon, Mar 11, 2013 at 02:03:27PM +0200, Michael S. Tsirkin wrote: > > > On Fri, Mar 08, 2013 at 10:21:41AM +0800, Asias He wrote: > > > > Changes in v2: > > > > - Re

Re: [PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-12 Thread Michael S. Tsirkin
On Tue, Mar 12, 2013 at 10:31:09AM +0800, Asias He wrote: > On Mon, Mar 11, 2013 at 02:03:27PM +0200, Michael S. Tsirkin wrote: > > On Fri, Mar 08, 2013 at 10:21:41AM +0800, Asias He wrote: > > > Changes in v2: > > > - Remove code duplication in tcm_vhost_{hotplug,hotunplug} > > > - Fix racing of v

Re: [PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-11 Thread Asias He
On Mon, Mar 11, 2013 at 02:03:27PM +0200, Michael S. Tsirkin wrote: > On Fri, Mar 08, 2013 at 10:21:41AM +0800, Asias He wrote: > > Changes in v2: > > - Remove code duplication in tcm_vhost_{hotplug,hotunplug} > > - Fix racing of vs_events_nr > > - Add flush fix patch to this series > > Asias He (6

Re: [PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-11 Thread Michael S. Tsirkin
On Fri, Mar 08, 2013 at 10:21:41AM +0800, Asias He wrote: > Changes in v2: > - Remove code duplication in tcm_vhost_{hotplug,hotunplug} > - Fix racing of vs_events_nr > - Add flush fix patch to this series > Asias He (6): > tcm_vhost: Add missed lock in vhost_scsi_clear_endpoint() > tcm_vhost:

Re: [PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-08 Thread Stefan Hajnoczi
On Fri, Mar 08, 2013 at 10:21:41AM +0800, Asias He wrote: > Changes in v2: > - Remove code duplication in tcm_vhost_{hotplug,hotunplug} > - Fix racing of vs_events_nr > - Add flush fix patch to this series > > Asias He (6): > tcm_vhost: Add missed lock in vhost_scsi_clear_endpoint() > tcm_vhos

[PATCH V2 0/6] tcm_vhost hotplug/hotunplug support and locking/flushing fix

2013-03-07 Thread Asias He
Changes in v2: - Remove code duplication in tcm_vhost_{hotplug,hotunplug} - Fix racing of vs_events_nr - Add flush fix patch to this series Asias He (6): tcm_vhost: Add missed lock in vhost_scsi_clear_endpoint() tcm_vhost: Introduce tcm_vhost_check_feature() tcm_vhost: Introduce tcm_vhost_ch