From: Luben Tuikov <linux-scsi@vger.kernel.org>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: linux-scsi@vger.kernel.org
Subject: Re: [PATCH] minimal SAS transport class
Date: Tue, 23 Aug 2005 20:02:39 -0400 [thread overview]
Message-ID: <430BB91F.7000406@vger.kernel.org> (raw)
In-Reply-To: <430B5CDB.9070204@s5r6.in-berlin.de>
On 08/23/05 13:28, Stefan Richter wrote:
> James.Smart@Emulex.Com wrote:
>
>>>As others stated, id is already a tag/label. You should be
>>>able to pass
>>>whatever id you want to scsi_scan_target, like the FC ID
>>>(port_id),
>
> [...]
>
>>If the port id changes during run time, what are you to do ?
>
> [...]
>
>>This approach only works as long as the transport's id fits within
>>the target id (an int). How would the SAS address(8byte wwn) utilize
>>such a scheme ?
>
>
> I don't understand exactly what is being discussed here. But if
> anybody's intention is to extend the concept of target ID to also
> provide the concept of transport specific, persistent, globally unique
> identifiers*, then the datatype of target ID needs to be extended
> accordingly.
Hi Stefan,
No, no, no -- this is exactly what we DO NOT WANT TO DO.
Why? Because it is the same mistake that was done when HCIL crept
up into SCSI Core 13 years ago.
Device addressing done by LLDD is completely not the business
of SCSI Core and vice versa. They should be *completely* separate.
In effect, a native support for targets (I know... a poor choice of
words, but not everyone reads specs), has been in need for SCSI Core
since one of the first iSCSI Target and Initiator for Linux was developed
5 years ago.
In effect FC tells SCSI Core: here is a device I do not know anything
about, other than the fact that it is on the fabric and it has
a SCSI Target (or Initiator) Port: how you get to is is _my_ business,
what is done with or to the device is _your_ business.
Then SCSI Core does LU discovery using SPC and SAM on the device
using Execute Task.
All this _completely_ eliminates the need for HCIL.
How the FC layer identifies the device is the FC's layer problem.
How SCSI Core identifies the device is SCSI Core's problem.
HCIL should have never been available to SCSI Core in the first place,
but then again SAM didn't exist back then.
> Otherwise the LLDs have to implement a mapping between target IDs and
LLDD should not have to invent HCIL just because SCSI Core hasn't
seen any inovation and advancement for the last 13 years.
Currently they do.
Target ID should NOT be extended to cover whatever new transport
protocols comes up. SCSI Core whould follow SAM if it wants to
transparently cover FC, iSCSI, SAS, etc.
> these unique identifiers. This mapping does not need to be persistent.
> Persistent mappings for whatever purpose can and should be implemented
> in userspace. (This is how it has always been e.g. with SBP-2.)
>
>
> *) examples of transport specific, persistent, globally unique
> identifiers: obviously 8 Byte address from SAS, 8 Byte EUI from SBP-2/
> IEEE 1394/ ISO/IEC 13213, or maybe 88 bit GUI from ISO/IEC 13213
Yes, you're right.
And if would be nice if all those IDs are just another label tacked
onto the domain device, opaque label. SCSI Core can add labels
on its own (e.g. HCIL ;-) ).
Luben
next prev parent reply other threads:[~2005-08-24 0:02 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-08-23 16:16 [PATCH] minimal SAS transport class James.Smart
2005-08-23 17:28 ` Stefan Richter
2005-08-24 0:02 ` Luben Tuikov [this message]
2005-08-24 9:12 ` Christoph Hellwig
2005-08-26 15:47 ` Luben Tuikov
2005-08-26 16:29 ` scsi device driver Yijian Wang
2005-08-26 19:24 ` [PATCH] minimal SAS transport class Jeff Garzik
2005-08-26 19:44 ` Luben Tuikov
2005-08-27 1:53 ` Jeff Garzik
2005-08-27 7:35 ` Stefan Richter
2005-08-28 22:27 ` Luben Tuikov
2005-08-29 5:16 ` Stefan Richter
2005-08-29 17:11 ` Christoph Hellwig
2005-08-29 17:20 ` Luben Tuikov
2005-08-29 17:25 ` Christoph Hellwig
2005-08-29 17:16 ` Christoph Hellwig
2005-08-29 17:31 ` Luben Tuikov
2005-08-29 18:34 ` Luben Tuikov
2005-08-29 18:09 ` James Bottomley
2005-08-23 17:57 ` Patrick Mansfield
-- strict thread matches above, loose matches on Subject: below --
2005-08-23 18:25 James.Smart
2005-08-22 23:08 Moore, Eric Dean
2005-08-24 8:59 ` Christoph Hellwig
2005-08-20 4:15 James.Smart
2005-08-20 4:57 ` Jeff Garzik
2005-08-20 17:23 ` Luben Tuikov
2005-08-21 17:03 ` Christoph Hellwig
2005-08-21 16:52 ` Christoph Hellwig
2005-08-21 18:23 ` Luben Tuikov
2005-08-22 4:55 ` Matt Domsch
2005-08-22 17:05 ` Luben Tuikov
2005-08-22 21:53 ` Mike Anderson
2005-08-23 23:55 ` Luben Tuikov
2005-08-24 17:12 ` Patrick Mansfield
2005-08-24 20:05 ` Luben Tuikov
2005-08-24 20:42 ` Patrick Mansfield
2005-08-24 21:48 ` Luben Tuikov
2005-08-23 11:10 ` Douglas Gilbert
2005-08-23 6:27 ` Hannes Reinecke
2005-08-23 15:42 ` Patrick Mansfield
2005-08-23 15:53 ` Matthew Wilcox
2005-08-24 0:13 ` Luben Tuikov
2005-08-25 19:32 ` Stefan Richter
2005-08-25 20:06 ` Jeff Garzik
2005-08-26 16:43 ` Luben Tuikov
2005-08-26 17:22 ` James Bottomley
2005-08-26 18:16 ` Luben Tuikov
2005-08-26 18:48 ` Jeff Garzik
2005-08-26 19:37 ` Luben Tuikov
2005-08-27 1:39 ` Jeff Garzik
2005-08-27 7:11 ` Stefan Richter
2005-08-28 22:13 ` Luben Tuikov
2005-08-18 18:48 James.Smart
2005-08-18 19:04 ` Jeff Garzik
2005-08-19 14:06 ` Christoph Hellwig
2005-08-19 17:51 ` Luben Tuikov
2005-08-19 17:54 ` Christoph Hellwig
2005-08-19 17:56 ` Luben Tuikov
2005-08-19 17:59 ` Christoph Hellwig
2005-08-19 18:07 ` Luben Tuikov
2005-08-19 19:59 ` James Bottomley
2005-08-19 20:32 ` Luben Tuikov
2005-08-19 20:54 ` Jeff Garzik
2005-08-20 9:18 ` Christoph Hellwig
2005-08-20 17:34 ` Luben Tuikov
2005-08-21 6:41 ` Arjan van de Ven
2005-08-21 17:07 ` Christoph Hellwig
2005-08-19 19:08 ` Luben Tuikov
2005-08-18 20:06 ` Luben Tuikov
2005-08-19 14:04 ` Christoph Hellwig
2005-08-18 14:43 James.Smart
2005-08-18 14:02 James.Smart
2005-08-18 17:56 ` Christoph Hellwig
2005-08-18 20:05 ` Luben Tuikov
2005-08-19 14:15 ` Christoph Hellwig
2005-08-18 11:57 James.Smart
2005-08-15 13:55 Christoph Hellwig
2005-08-15 14:19 ` Luben Tuikov
2005-08-15 14:35 ` Arjan van de Ven
2005-08-15 15:04 ` Luben Tuikov
2005-08-15 15:13 ` Luben Tuikov
2005-08-15 15:21 ` Christoph Hellwig
2005-08-15 15:33 ` Luben Tuikov
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=430BB91F.7000406@vger.kernel.org \
--to=linux-scsi@vger.kernel.org \
--cc=stefanr@s5r6.in-berlin.de \
/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).