public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: linux-scsi@vger.kernel.org
Cc: James.Smart@Emulex.Com, hch@lst.de
Subject: Re: [PATCH] SAS transport class
Date: Tue, 06 Sep 2005 18:00:00 +0200	[thread overview]
Message-ID: <431DBD00.2060309@s5r6.in-berlin.de> (raw)
In-Reply-To: <9BB4DECD4CFE6D43AA8EA8D768ED51C21D7A75@xbl3.ma.emulex.com>

James Smart wrote:
> I'll point out, but not delve into - that you've avoided the target
> id bindings again, thus forcing the lldd's/adapters to maintain tables
> of port#/SASaddr to target id (if they care, and I believe enterprise
> storage will care!).  I can understand if you're going to eventually
> insert a generic abstraction layer for scsi's b/c/t/l addressing
> scheme, but since it's not here today, I expect you to hold SAS to the
> same "code sharing" level as FC.

b/c/t/l has no place in layers below scsi core (except in SPI low-level 
where it has actual meaning).

I don't understand exactly what should be accomplished by a table of 
WWNs to target IDs --- but any kind of mapping between WWN and device 
file (and thereby between WWN and a faked b/c/t/l) belongs into 
userspace, not into the kernel.

Furthermore, there cannot be an "abstraction layer for scsi's b/c/t/l 
addressing scheme". It is impossible to abstract something which is 
without meaning.
-- 
Stefan Richter
-=====-=-=-= =--= --==-
http://arcgraph.de/sr/

  parent reply	other threads:[~2005-09-06 15:59 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-09-06 13:40 [PATCH] SAS transport class James.Smart
2005-09-06 14:00 ` Christoph Hellwig
2005-09-06 16:00 ` Stefan Richter [this message]
  -- strict thread matches above, loose matches on Subject: below --
2005-09-09 14:22 Christoph Hellwig
2005-09-09 14:35 ` James Bottomley
2005-09-09 14:38   ` Christoph Hellwig
2005-09-09 23:38 ` James Bottomley
2005-09-06 14:24 James.Smart
2005-09-04 23:50 Christoph Hellwig

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=431DBD00.2060309@s5r6.in-berlin.de \
    --to=stefanr@s5r6.in-berlin.de \
    --cc=James.Smart@Emulex.Com \
    --cc=hch@lst.de \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox