[PATCH v2 00/11] vhost cleanups

2013-05-06 Thread Asias He
MST, This is on top of [PATCH 0/2] vhost-net fix ubuf.

Asias He (11):
  vhost: Remove vhost_enable_zcopy in vhost.h
  vhost: Move VHOST_NET_FEATURES to net.c
  vhost: Make vhost a separate module
  vhost: Remove comments for hdr in vhost.h
  vhost: Simplify dev-vqs[i] access
  vhost-net: Cleanup vhost_ubuf and vhost_zcopy
  vhost-scsi: Remove unnecessary forward struct vhost_scsi declaration
  vhost-scsi: Rename struct vhost_scsi *s to *vs
  vhost-scsi: Make func indention more consistent
  vhost-scsi: Rename struct tcm_vhost_tpg *tv_tpg to *tpg
  vhost-scsi: Rename struct tcm_vhost_cmd *tv_cmd to *cmd

 drivers/vhost/Kconfig  |   8 +
 drivers/vhost/Makefile |   3 +-
 drivers/vhost/net.c|  64 ---
 drivers/vhost/scsi.c   | 470 ++---
 drivers/vhost/vhost.c  |  86 +++--
 drivers/vhost/vhost.h  |  11 +-
 6 files changed, 361 insertions(+), 281 deletions(-)

-- 
1.8.1.4

--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 00/11] vhost cleanups

2013-05-06 Thread Michael S. Tsirkin
On Mon, May 06, 2013 at 04:38:18PM +0800, Asias He wrote:
 MST, This is on top of [PATCH 0/2] vhost-net fix ubuf.

Acked-by: Michael S. Tsirkin m...@redhat.com

Once -rc1 is out I'll fork -next and apply them.
Thanks a lot!

Nicholas, recently attempts to push patches through both net and target
trees resulted in a bit of a mess, so let's stick to the common tree
(unless there's a dependency that makes us not to) until rate of changes
in the common code calms down a bit.  OK?

 Asias He (11):
   vhost: Remove vhost_enable_zcopy in vhost.h
   vhost: Move VHOST_NET_FEATURES to net.c
   vhost: Make vhost a separate module
   vhost: Remove comments for hdr in vhost.h
   vhost: Simplify dev-vqs[i] access
   vhost-net: Cleanup vhost_ubuf and vhost_zcopy
   vhost-scsi: Remove unnecessary forward struct vhost_scsi declaration
   vhost-scsi: Rename struct vhost_scsi *s to *vs
   vhost-scsi: Make func indention more consistent
   vhost-scsi: Rename struct tcm_vhost_tpg *tv_tpg to *tpg
   vhost-scsi: Rename struct tcm_vhost_cmd *tv_cmd to *cmd
 
  drivers/vhost/Kconfig  |   8 +
  drivers/vhost/Makefile |   3 +-
  drivers/vhost/net.c|  64 ---
  drivers/vhost/scsi.c   | 470 
 ++---
  drivers/vhost/vhost.c  |  86 +++--
  drivers/vhost/vhost.h  |  11 +-
  6 files changed, 361 insertions(+), 281 deletions(-)
 
 -- 
 1.8.1.4
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 00/11] vhost cleanups

2013-05-06 Thread Michael S. Tsirkin
On Mon, May 06, 2013 at 04:38:18PM +0800, Asias He wrote:
 MST, This is on top of [PATCH 0/2] vhost-net fix ubuf.

Okay, how about making EVENT_IDX work for virtio-scsi?
I'm guessing it's some messup with feature negotiation,
that's what all event-idx bugs came down to so far.

 Asias He (11):
   vhost: Remove vhost_enable_zcopy in vhost.h
   vhost: Move VHOST_NET_FEATURES to net.c
   vhost: Make vhost a separate module
   vhost: Remove comments for hdr in vhost.h
   vhost: Simplify dev-vqs[i] access
   vhost-net: Cleanup vhost_ubuf and vhost_zcopy
   vhost-scsi: Remove unnecessary forward struct vhost_scsi declaration
   vhost-scsi: Rename struct vhost_scsi *s to *vs
   vhost-scsi: Make func indention more consistent
   vhost-scsi: Rename struct tcm_vhost_tpg *tv_tpg to *tpg
   vhost-scsi: Rename struct tcm_vhost_cmd *tv_cmd to *cmd
 
  drivers/vhost/Kconfig  |   8 +
  drivers/vhost/Makefile |   3 +-
  drivers/vhost/net.c|  64 ---
  drivers/vhost/scsi.c   | 470 
 ++---
  drivers/vhost/vhost.c  |  86 +++--
  drivers/vhost/vhost.h  |  11 +-
  6 files changed, 361 insertions(+), 281 deletions(-)
 
 -- 
 1.8.1.4
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 00/11] vhost cleanups

2013-05-06 Thread Asias He
On Mon, May 06, 2013 at 01:07:46PM +0300, Michael S. Tsirkin wrote:
 On Mon, May 06, 2013 at 04:38:18PM +0800, Asias He wrote:
  MST, This is on top of [PATCH 0/2] vhost-net fix ubuf.
 
 Okay, how about making EVENT_IDX work for virtio-scsi?
 I'm guessing it's some messup with feature negotiation,
 that's what all event-idx bugs came down to so far.

Yes, IIRC, EVENT_IDX works for vhost-scsi now. Will cook a patch to
enable it. It should go 3.10, right?

  Asias He (11):
vhost: Remove vhost_enable_zcopy in vhost.h
vhost: Move VHOST_NET_FEATURES to net.c
vhost: Make vhost a separate module
vhost: Remove comments for hdr in vhost.h
vhost: Simplify dev-vqs[i] access
vhost-net: Cleanup vhost_ubuf and vhost_zcopy
vhost-scsi: Remove unnecessary forward struct vhost_scsi declaration
vhost-scsi: Rename struct vhost_scsi *s to *vs
vhost-scsi: Make func indention more consistent
vhost-scsi: Rename struct tcm_vhost_tpg *tv_tpg to *tpg
vhost-scsi: Rename struct tcm_vhost_cmd *tv_cmd to *cmd
  
   drivers/vhost/Kconfig  |   8 +
   drivers/vhost/Makefile |   3 +-
   drivers/vhost/net.c|  64 ---
   drivers/vhost/scsi.c   | 470 
  ++---
   drivers/vhost/vhost.c  |  86 +++--
   drivers/vhost/vhost.h  |  11 +-
   6 files changed, 361 insertions(+), 281 deletions(-)
  
  -- 
  1.8.1.4

-- 
Asias
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 00/11] vhost cleanups

2013-05-06 Thread Michael S. Tsirkin
On Mon, May 06, 2013 at 08:05:26PM +0800, Asias He wrote:
 On Mon, May 06, 2013 at 01:07:46PM +0300, Michael S. Tsirkin wrote:
  On Mon, May 06, 2013 at 04:38:18PM +0800, Asias He wrote:
   MST, This is on top of [PATCH 0/2] vhost-net fix ubuf.
  
  Okay, how about making EVENT_IDX work for virtio-scsi?
  I'm guessing it's some messup with feature negotiation,
  that's what all event-idx bugs came down to so far.
 
 Yes, IIRC, EVENT_IDX works for vhost-scsi now. Will cook a patch to
 enable it. It should go 3.10, right?

If it's early in the cycle, I think it can.

   Asias He (11):
 vhost: Remove vhost_enable_zcopy in vhost.h
 vhost: Move VHOST_NET_FEATURES to net.c
 vhost: Make vhost a separate module
 vhost: Remove comments for hdr in vhost.h
 vhost: Simplify dev-vqs[i] access
 vhost-net: Cleanup vhost_ubuf and vhost_zcopy
 vhost-scsi: Remove unnecessary forward struct vhost_scsi declaration
 vhost-scsi: Rename struct vhost_scsi *s to *vs
 vhost-scsi: Make func indention more consistent
 vhost-scsi: Rename struct tcm_vhost_tpg *tv_tpg to *tpg
 vhost-scsi: Rename struct tcm_vhost_cmd *tv_cmd to *cmd
   
drivers/vhost/Kconfig  |   8 +
drivers/vhost/Makefile |   3 +-
drivers/vhost/net.c|  64 ---
drivers/vhost/scsi.c   | 470 
   ++---
drivers/vhost/vhost.c  |  86 +++--
drivers/vhost/vhost.h  |  11 +-
6 files changed, 361 insertions(+), 281 deletions(-)
   
   -- 
   1.8.1.4
 
 -- 
 Asias
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v2 00/11] vhost cleanups

2013-05-06 Thread Asias He
On Mon, May 06, 2013 at 04:15:35PM +0300, Michael S. Tsirkin wrote:
 On Mon, May 06, 2013 at 08:05:26PM +0800, Asias He wrote:
  On Mon, May 06, 2013 at 01:07:46PM +0300, Michael S. Tsirkin wrote:
   On Mon, May 06, 2013 at 04:38:18PM +0800, Asias He wrote:
MST, This is on top of [PATCH 0/2] vhost-net fix ubuf.
   
   Okay, how about making EVENT_IDX work for virtio-scsi?
   I'm guessing it's some messup with feature negotiation,
   that's what all event-idx bugs came down to so far.
  
  Yes, IIRC, EVENT_IDX works for vhost-scsi now. Will cook a patch to
  enable it. It should go 3.10, right?
 
 If it's early in the cycle, I think it can.

Well, let's queue it for 3.11.

Asias He (11):
  vhost: Remove vhost_enable_zcopy in vhost.h
  vhost: Move VHOST_NET_FEATURES to net.c
  vhost: Make vhost a separate module
  vhost: Remove comments for hdr in vhost.h
  vhost: Simplify dev-vqs[i] access
  vhost-net: Cleanup vhost_ubuf and vhost_zcopy
  vhost-scsi: Remove unnecessary forward struct vhost_scsi declaration
  vhost-scsi: Rename struct vhost_scsi *s to *vs
  vhost-scsi: Make func indention more consistent
  vhost-scsi: Rename struct tcm_vhost_tpg *tv_tpg to *tpg
  vhost-scsi: Rename struct tcm_vhost_cmd *tv_cmd to *cmd

 drivers/vhost/Kconfig  |   8 +
 drivers/vhost/Makefile |   3 +-
 drivers/vhost/net.c|  64 ---
 drivers/vhost/scsi.c   | 470 
++---
 drivers/vhost/vhost.c  |  86 +++--
 drivers/vhost/vhost.h  |  11 +-
 6 files changed, 361 insertions(+), 281 deletions(-)

-- 
1.8.1.4
  
  -- 
  Asias

-- 
Asias
--
To unsubscribe from this list: send the line unsubscribe kvm in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html