Re: [RFC-v3 4/4] tcm_vhost: Initial merge for vhost level target fabric driver

["Michael S. Tsirkin" <mst@redhat.com>]

On Wed, Jul 18, 2012 at 02:20:58PM -0700, Nicholas A. Bellinger wrote:
> On Wed, 2012-07-18 at 19:09 +0300, Michael S. Tsirkin wrote:
> > On Wed, Jul 18, 2012 at 12:59:32AM +0000, Nicholas A. Bellinger wrote:
> 
> <SNIP>
> 
> > > 
> > > Changelog v2 -> v3:
> > > 
> > >   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)
> > > 
> > > ---
> > >  drivers/staging/Kconfig       |    2 +
> > >  drivers/vhost/Makefile        |    2 +
> > >  drivers/vhost/tcm/Kconfig     |    6 +
> > >  drivers/vhost/tcm/Makefile    |    1 +
> > >  drivers/vhost/tcm/tcm_vhost.c | 1611 +++++++++++++++++++++++++++++++++++++++++
> > >  drivers/vhost/tcm/tcm_vhost.h |   74 ++
> > >  6 files changed, 1696 insertions(+), 0 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
> > > 
> > 
> > Really sorry about making you run around like that,
> > I did not mean moving all of tcm to a directory,
> > just adding tcm/Kconfig or adding drivers/vhost/Kconfig.tcm
> > because eventually it's easier to keep it all together
> > in one place.
> > 
> 
> Er, apologies for the slight mis-understanding here..  Moving back now +
> fixing up the Kbuild bits.

I'm going offline in several hours and am on vacation for a week
starting tomorrow. So to make 3.6, and if you intend to merge through my
tree, the best bet is if you can send the final version real soon now.

-- 
MST