[PATCH v2 00/11] vhost cleanups
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
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
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
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
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
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