linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: stgt@vger.kernel.org, linux-scsi@vger.kernel.org,
	linux-kernel@vger.kernel.org, michaelc@cs.wisc.edu,
	bharrosh@panasas.com, James.Bottomley@suse.de,
	dgilbert@interlog.com
Subject: Re: [PATCH 0/3] Add proper STGT <-> bs_sg passthrough v3
Date: Mon, 07 Jun 2010 02:40:52 -0700	[thread overview]
Message-ID: <1275903652.26888.73.camel@haakon2.linux-iscsi.org> (raw)
In-Reply-To: <20100607183438Y.fujita.tomonori@lab.ntt.co.jp>

On Mon, 2010-06-07 at 18:35 +0900, FUJITA Tomonori wrote:
> On Mon, 07 Jun 2010 02:07:09 -0700
> "Nicholas A. Bellinger" <nab@linux-iscsi.org> wrote:
> 
> > On Mon, 2010-06-07 at 17:56 +0900, FUJITA Tomonori wrote:
> > > On Mon, 07 Jun 2010 01:27:16 -0700
> > > "Nicholas A. Bellinger" <nab@linux-iscsi.org> wrote:
> > > 
> > > > Ok, I just fixed some minor breakage and applied patch 3 on top of
> > > > tgt.git/pass in my local tree and everything appears to work with
> > > > '-E bsg -Y pt'.
> > > > 
> > > > Please let me know if you have any other comments on the BSG bits or if
> > > > you would like me to send along a updated patch 3 seperately.
> > > 
> > > Please send patches except for bsg first.
> > > 
> > > About bsg support, we need both sg and bsg? Just bsg is not enough?
> > 
> > I figured you would still like legacy support w/o bsg for the forseeable
> > future with older production kernels.  I don't recall which kernel BSG
> > support was added or if this code has already been backported to the
> > various distro kernels.
> 
> I prefer to support only bsg (until someone tells me to support sg).
> 

Ok, if that is the case I will respin a patch on top of tgt.git/pass to
remove SG and use BSG exclusively.  Please let me know if you have any
more recommendations in this area.

> 
> btw, if you look for a tool to generate a bidi command, my sgv4-tools
> might work for you.
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/tomo/sgv4-tools.git
> 
> try sgv4_xdwriteread (note that it's not useful at all except for
> generating bidi).

Ah perfect, this is exactly what I am looking for.  Many thanks.

Best,

--nab


  reply	other threads:[~2010-06-07  9:40 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-07  3:50 [PATCH 0/3] Add proper STGT <-> bs_sg passthrough v3 Nicholas A. Bellinger
2010-06-07  6:45 ` FUJITA Tomonori
2010-06-07  8:27   ` Nicholas A. Bellinger
2010-06-07  8:56     ` FUJITA Tomonori
2010-06-07  9:07       ` Nicholas A. Bellinger
2010-06-07  9:35         ` FUJITA Tomonori
2010-06-07  9:40           ` Nicholas A. Bellinger [this message]
2010-06-07 23:45     ` FUJITA Tomonori
2010-06-08  0:19       ` Nicholas A. Bellinger
2010-07-20 23:17         ` James Bottomley
2010-07-20 23:39           ` Nicholas A. Bellinger

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=1275903652.26888.73.camel@haakon2.linux-iscsi.org \
    --to=nab@linux-iscsi.org \
    --cc=James.Bottomley@suse.de \
    --cc=bharrosh@panasas.com \
    --cc=dgilbert@interlog.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=michaelc@cs.wisc.edu \
    --cc=stgt@vger.kernel.org \
    /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).