virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
Cc: Jens Axboe <axboe@kernel.dk>,
	Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>,
	kvm-devel <kvm@vger.kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Zhi Yong Wu <wuzhy@cn.ibm.com>,
	Anthony Liguori <aliguori@linux.vnet.ibm.com>,
	target-devel <target-devel@vger.kernel.org>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	Paolo Bonzini <pbonzini@redhat.com>,
	lf-virt <virtualization@lists.linux-foundation.org>,
	Christoph Hellwig <hch@lst.de>
Subject: Re: [RFC-v3 0/4] tcm_vhost+cmwq fabric driver code for-3.6
Date: Wed, 18 Jul 2012 15:57:27 +0300	[thread overview]
Message-ID: <20120718125727.GA5589@redhat.com> (raw)
In-Reply-To: <1342573172-17421-1-git-send-email-nab@linux-iscsi.org>

On Wed, Jul 18, 2012 at 12:59:28AM +0000, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger <nab@linux-iscsi.org>
> 
> Hi folks,
> 
> The following is the RFC-v3 series of tcm_vhost target fabric driver code
> currently in-flight for-3.6 mainline code.
> 
> With the merge window opening soon, the tcm_vhost code has started seeing
> time in linux-next.  The v2 -> v3 changelog from the last week is currently
> looking like:
> 
>  *) Unlock on error in tcm_vhost_drop_nexus() (DanC)
>  *) Fix strlen() doesn't count the terminator (DanC)
>  *) Call kfree() on an error path (DanC)
>  *) Convert tcm_vhost_write_pending to use target_execute_cmd (hch + nab)
>  *) Fix another strlen() off by one in tcm_vhost_make_tport (DanC)
>  *) Add option under drivers/staging/Kconfig, and move to drivers/vhost/tcm/
>     as requested by MST (nab)

I actually only wanted to have a separate Kconfig (in a separate
directory or Kconfig.tcm), so you do not need to move code back to move
driver out of staging.  But if you prefer this, I'm fine with it too for
now.

> Thanks to Dan Carpenter for his smatch fixes this past round.
> 
> Also as requested by MST, the code has been moved to a seperate tcm/ subdirectory
> under drivers/vhost/ so that it can be included under staging's config options
> until we can settle on the necessary userspace bits for QEMU and kvm-tool.
> 
> The updated series will be going out shortly to target-pending/for-next-merge.
> 
> Please have another look and let us know if you have any concerned.
> 
> Thanks!
> 
> Nicholas Bellinger (2):
>   vhost: Add vhost_scsi specific defines
>   tcm_vhost: Initial merge for vhost level target fabric driver
> 
> Stefan Hajnoczi (2):
>   vhost: Separate vhost-net features from vhost features
>   vhost: make vhost work queue visible
> 
>  drivers/staging/Kconfig       |    2 +
>  drivers/vhost/Makefile        |    2 +
>  drivers/vhost/net.c           |    4 +-
>  drivers/vhost/tcm/Kconfig     |    6 +
>  drivers/vhost/tcm/Makefile    |    1 +
>  drivers/vhost/tcm/tcm_vhost.c | 1611 +++++++++++++++++++++++++++++++++++++++++
>  drivers/vhost/tcm/tcm_vhost.h |   74 ++
>  drivers/vhost/test.c          |    4 +-
>  drivers/vhost/vhost.c         |    5 +-
>  drivers/vhost/vhost.h         |    6 +-
>  include/linux/vhost.h         |    9 +
>  11 files changed, 1716 insertions(+), 8 deletions(-)
>  create mode 100644 drivers/vhost/tcm/Kconfig
>  create mode 100644 drivers/vhost/tcm/Makefile
>  create mode 100644 drivers/vhost/tcm/tcm_vhost.c
>  create mode 100644 drivers/vhost/tcm/tcm_vhost.h
> 
> -- 
> 1.7.2.5

  parent reply	other threads:[~2012-07-18 12:57 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-18  0:59 [RFC-v3 0/4] tcm_vhost+cmwq fabric driver code for-3.6 Nicholas A. Bellinger
2012-07-18  0:59 ` [RFC-v3 1/4] vhost: Separate vhost-net features from vhost features Nicholas A. Bellinger
2012-07-18  0:59 ` [RFC-v3 2/4] vhost: make vhost work queue visible Nicholas A. Bellinger
2012-07-18  0:59 ` [RFC-v3 3/4] vhost: Add vhost_scsi specific defines Nicholas A. Bellinger
2012-07-18 13:05   ` Michael S. Tsirkin
     [not found]   ` <20120718130518.GB5589@redhat.com>
2012-07-18 21:17     ` Nicholas A. Bellinger
     [not found]     ` <1342646225.18004.633.camel@haakon2.linux-iscsi.org>
2012-07-18 21:21       ` Michael S. Tsirkin
2012-07-18  0:59 ` [RFC-v3 4/4] tcm_vhost: Initial merge for vhost level target fabric driver Nicholas A. Bellinger
2012-07-18 16:09   ` Michael S. Tsirkin
2012-07-18 21:20     ` Nicholas A. Bellinger
     [not found]     ` <1342646458.18004.637.camel@haakon2.linux-iscsi.org>
2012-07-20 12:03       ` Michael S. Tsirkin
2012-07-20 18:00         ` Nicholas A. Bellinger
2012-07-20 18:02           ` Nicholas A. Bellinger
2012-07-18 12:57 ` Michael S. Tsirkin [this message]
2012-07-18 16:11 ` [RFC-v3 0/4] tcm_vhost+cmwq fabric driver code for-3.6 Michael S. Tsirkin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120718125727.GA5589@redhat.com \
    --to=mst@redhat.com \
    --cc=aliguori@linux.vnet.ibm.com \
    --cc=axboe@kernel.dk \
    --cc=gregkh@linuxfoundation.org \
    --cc=hch@lst.de \
    --cc=kvm@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=nab@linux-iscsi.org \
    --cc=pbonzini@redhat.com \
    --cc=stefanha@linux.vnet.ibm.com \
    --cc=target-devel@vger.kernel.org \
    --cc=virtualization@lists.linux-foundation.org \
    --cc=wuzhy@cn.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).