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 add stuff

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 vs_events_nr

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: - Remove code

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-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-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_vhost:

[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