All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pete Wyckoff <pw@osc.edu>
To: Boaz Harrosh <bharrosh@panasas.com>
Cc: James Bottomley <James.Bottomley@HansenPartnership.com>,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
	Christoph Hellwig <hch@infradead.org>,
	Jens Axboe <jens.axboe@oracle.com>, Jeff Garzik <jeff@garzik.org>,
	linux-scsi <linux-scsi@vger.kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 21/24][RFC] scsi_tgt: use of sense accessors
Date: Tue, 5 Feb 2008 11:21:33 -0500	[thread overview]
Message-ID: <20080205162133.GC1931@osc.edu> (raw)
In-Reply-To: <1202147595-5400-1-git-send-email-bharrosh@panasas.com>

bharrosh@panasas.com wrote on Mon, 04 Feb 2008 19:53 +0200:
>   FIXME: I need help with this driver (Pete?)
>     I used scsi_sense() in a none const way. But since
>     scsi_tgt is the ULD here, it can just access it's own sense
>     buffer directly. I did not use scsi_eh_cpy_sense() because
>     I did not want the extra copy. Pete will want to use a 260
>     bytes buffer here.
> 
> Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
> Need-help-from: Pete Wyckoff <pw@osc.edu>

FYI, I never use scsi_tgt.  Only just pure userspace on the target,
and a dumb ethernet NIC that does not know it is speaking any form
of SCSI.

People who need scsi_tgt have real target-enabled NICs like the
fancy qla4xxx.  Those act as SCSI targets across FC or IP or
whatever and bring commands into the kernel, which then relays them
to a userspace tgtd process, which does the read/write as necessary,
and returns a result code to the NIC to ship back across FC.

So sorry, I won't take a guess at what has to happen here.  But
yeah, you are right that an OSD target implementation would at times
need a sense buffer bigger than 96.  Protocol maximum length for all
sense data is 264.

		-- Pete

  reply	other threads:[~2008-02-05 16:21 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-04 17:53 [PATCH 21/24][RFC] scsi_tgt: use of sense accessors Boaz Harrosh
2008-02-05 16:21 ` Pete Wyckoff [this message]
2008-02-05 16:53   ` FUJITA Tomonori
2008-02-05 17:28     ` Jeff Garzik

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=20080205162133.GC1931@osc.edu \
    --to=pw@osc.edu \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=akpm@linux-foundation.org \
    --cc=bharrosh@panasas.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=hch@infradead.org \
    --cc=jeff@garzik.org \
    --cc=jens.axboe@oracle.com \
    --cc=linux-scsi@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.