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
next prev parent 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).