linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@pobox.com>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org,
	ltuikov@yahoo.com, Mike Anderson <andmike@us.ibm.com>,
	Luben Tuikov <luben_tuikov@adaptec.com>,
	James Bottomley <James.Bottomley@steeleye.com>,
	Christoph Hellwig <hch@infradead.org>
Subject: Re: Adaptec SAS integration notes
Date: Tue, 18 Oct 2005 16:51:48 -0400	[thread overview]
Message-ID: <43556064.20104@pobox.com> (raw)
In-Reply-To: <43555CC1.8040200@s5r6.in-berlin.de>

Stefan Richter wrote:
> All transports implement HCIL mappings. (SPI drivers enjoy the easiest 
> mapping, hcil<->HCIL.) They are all faced with the necessity of mapping 
> and solve it in different ways, for different reasons. Why stop at FC + 
> SAS and try to find a unified solution for just those two? Also, when 
> you are working out how FC and SAS could interface to a common HCIL 
> mapping library (or HCIL mapping layer?), you are effectively designing 
> aspects of how an interface between FC|SAS and SCSI core should look 
> like. Or, why not provide the HCIL mapping helpers in the core and 
> subsequently make users of this new interface completely unaware of the 
> existence of HCIL...?

Ignoring some legacy ioctls and bits of existing code...   The only 
mapping that's _really_ required is an internal C pointer, used by the 
device class to send messages to the transport class.

Adaptec's code does a bit of that on the transport side, but we'd need 
to update all the existing device classes (sd, sr, st, ...) before they 
had been converted to a purely using pointers to connect the transport 
to the device.

Getting to the point where device classes need only to do

	transport_instance->send_message(scsi command | TMF)

and where the transport class replies with

	device_instance->send_message(scsi response | TMF response)

may be a long term goal.  Depends on where evolution takes us...


> Moreover, HCIL mapping is but one aspect of a common problem. As an 
> example, I think yet another aspect of the same problem is that device 
> properties like target names and LUNs get lost below the 
> transport<->core interface. We want to present these device properties 
> to userspace in a unified manner. Here it becomes even more clear: 
> Neither is it a problem for FC and SAS alone, nor are transport-level 
> helpers a good solution to the problem.

Target names are already in the generic device.  Luben has a good point, 
though, that there may be multiple "labels" for the same addressible object.


> I freely admit that I don't have a picture of where the real 
> difficulties lie; I am far too unfamiliar with the innards of scsi core. 
> But time spent on transport-level helpers which are meant to ease 
> symptoms of the transport<->core difficulties seems to be spent rather 
> ineffectively to me. More concrete, what is the ratio of benefit to cost 
> of HCIL mapping code sharing between SAS and FC? Especially since that 
> mapping is deemed obsolete already (in the long term or perhaps even mid 
> term). And to completely go back to the subject: Will attempts on code 
> sharing between FC layer and SAS layer bring things forward in the 
> short-term goal of SAS integration?

HCIL mapping is a simple and very small part of this task :)  There are 
a bunch of line items to tackle, including SMP and SDI details, all 
bundled up in this.

	Jeff



  reply	other threads:[~2005-10-18 20:51 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20051012184029.GA10786@havoc.gtf.org>
2005-10-14 18:06 ` Adaptec SAS integration notes Luben Tuikov
2005-10-15 14:19   ` Stefan Richter
2005-10-16  1:05     ` Douglas Gilbert
2005-10-17  6:35       ` Stefan Richter
2005-10-18 16:44   ` Jeff Garzik
2005-10-18 19:30     ` Luben Tuikov
2005-10-18 20:16       ` Jeff Garzik
2005-10-18 20:55         ` Luben Tuikov
2005-10-18 20:36     ` Stefan Richter
2005-10-18 20:51       ` Jeff Garzik [this message]
2005-10-15 14:21 James.Smart
2005-10-15 16:01 ` Stefan Richter
2005-10-18 15:06 ` Luben Tuikov
2005-10-18 16:12 ` 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=43556064.20104@pobox.com \
    --to=jgarzik@pobox.com \
    --cc=James.Bottomley@steeleye.com \
    --cc=andmike@us.ibm.com \
    --cc=hch@infradead.org \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=ltuikov@yahoo.com \
    --cc=luben_tuikov@adaptec.com \
    --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).