Re: [RFC-v3 4/4] tcm_vhost: Initial merge for vhost level target fabric driver
From: Nicholas A. Bellinger <hidden>
Date: 2012-07-20 18:00:47
Also in:
kvm, linux-scsi
On Fri, 2012-07-20 at 15:03 +0300, Michael S. Tsirkin wrote:
On Wed, Jul 18, 2012 at 02:20:58PM -0700, Nicholas A. Bellinger wrote:quoted
On Wed, 2012-07-18 at 19:09 +0300, Michael S. Tsirkin wrote:quoted
On Wed, Jul 18, 2012 at 12:59:32AM +0000, Nicholas A. Bellinger wrote:<SNIP>quoted
quoted
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.hReally 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.
Ok, thanks for the heads up here.. So aside from Greg-KH's feedback to avoid the drivers/staging/ Kconfig include usage, and one more bugfix from DanC from this morning those are the only pending changes for RFC-v4. If it's OK I'd prefer to take these via target-pending with the necessary Acked-By's, especially if you'll be AFK next week.. Would you like to see a RFC-v4 with these changes included..? Thank you, --nab