linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Brian King <brking@us.ibm.com>
To: James Bottomley <James.Bottomley@SteelEye.com>
Cc: jgarzik@pobox.com, linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [PATCH 1/2] scsi: Add scsi_device max_cmd_len
Date: Tue, 28 Mar 2006 18:03:00 -0600	[thread overview]
Message-ID: <4429CEB4.4050608@us.ibm.com> (raw)
In-Reply-To: <1143586146.3353.51.camel@mulgrave.il.steeleye.com>

James Bottomley wrote:
> On Tue, 2006-03-28 at 16:38 -0600, Brian King wrote:
>> My direction at this point has been to move away from
>> the virtual scsi host device for SATA devices attached
>> to SAS HBAs. This seemed to me to be the clearest implementation.
>> Then you have 1 SAS HBA = 1 struct scsi_host, no matter
>> how many SATA devices are attached underneath it either
>> via direct attach or via an expander.
> 
> That's what a transport class implementation would allow you to do.  As
> long as the representation of the host controller device (be it PCI,
> SCSI IDE or whatever) has an embedded generic device, the class can
> attach and carry attributes.

I'm still struggling a little with where you want to head here. Are you
proposing a scsi_transport_sata in addition to the existing scsi transports,
or are you proposing adding to the existing scsi_transport_sas code?
I assume it is the latter, since a single SAS HBA will be supporting
both SAS and SATA devices at the same time and currently scsi core only
handles a single transport per scsi_host.


Thanks,

Brian

-- 
Brian King
eServer Storage I/O
IBM Linux Technology Center

  reply	other threads:[~2006-03-29  0:03 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-28 22:17 [PATCH 1/2] scsi: Add scsi_device max_cmd_len Brian King
2006-03-28 22:29 ` James Bottomley
2006-03-28 22:38   ` Brian King
2006-03-28 22:49     ` James Bottomley
2006-03-29  0:03       ` Brian King [this message]
2006-03-29  0:12         ` James Bottomley
2006-03-29  4:42           ` Brian King
2006-03-29 23:22   ` Jeff Garzik
2006-03-29  9:11 ` Christoph Hellwig
2006-03-29 14:15   ` Brian King
2006-03-29 15:05     ` Christoph Hellwig
2006-03-29 23:19   ` Jeff Garzik
2006-03-30 16:39     ` Brian King
2006-03-30 16:42       ` Brian King
2006-04-04  9:25       ` Jeff Garzik
2006-04-07 13:36         ` Brian King
2006-04-01 10:31     ` Stefan Richter

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=4429CEB4.4050608@us.ibm.com \
    --to=brking@us.ibm.com \
    --cc=James.Bottomley@SteelEye.com \
    --cc=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --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;
as well as URLs for NNTP newsgroup(s).