All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <jejb@steeleye.com>
To: Luben Tuikov <luben_tuikov@adaptec.com>
Cc: Christoph Hellwig <hch@lst.de>,
	SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: Re: [PATCH] allow a transport to pre-initialize starget_data
Date: Mon, 15 Aug 2005 11:32:13 -0500	[thread overview]
Message-ID: <1124123533.5089.24.camel@mulgrave> (raw)
In-Reply-To: <4300BE8D.4070002@adaptec.com>

On Mon, 2005-08-15 at 12:10 -0400, Luben Tuikov wrote:
> Isn't DV an SPI paradigm?

Yes, but that's only one of the reasons to expose the underlying device.

> If the whole point of the LLDD+controller is to _give_ you this,
> "an LU which is actually RAID", abstraction, why does SCSI Core
> need to get to the underlying devices?

Because lots of the RAID drivers have internal code that performs
actions, like DV, on their underlying physical discs.

> What will SCSI Core do with the RAID members?  Why is SCSI Core
> concerned with this information?  What kind of parameters will
> it have to set, without the RAID device server knowing about it
> (which is implemented on the chip if HW RAID)?

Nothing except expose them ... that's the idea.  The parameters are set
from user level after that.  This elimiates need for the pass through
ioctl that all of these drivers have.

> > It's not a SPI leftover.  It's used by HBA's that have two channels that
> > genuinely share resources.  Although most modern SCSI cards simply have
> > two chips (and hence two separate HBAs) per channel, there are a few, of
> > which the qlogic fc card is one, that still have this one chip per two
> > channels model.
> 
> Ah, I see.
> 
> Please help me understand this:
> 
> You say that "channel" is not an SPI leftover but it is used
> by SCSI Core because 
> 
> 	"HBA's that have two channels that genuinely share resources".
> 
> Why is SCSI Core concerned with how HBAs are _built_?  Isn't this
> the LLDD's job?

Because resource management belongs in the mid-layer.  In particular
available commands per host.

> Is "channel" in SCSI Core used to _address_ SCSI devices?
> How many of those "channel" will we need?

On the wire? No.  In the mid-layer, not really, since devices are
addressed by struct scsi_device.  The channel makes a useful
discriminator for the LLD in the scsi device, however.

James



  reply	other threads:[~2005-08-15 16:32 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-15 13:42 [PATCH] allow a transport to pre-initialize starget_data Christoph Hellwig
2005-08-15 15:14 ` James Bottomley
2005-08-15 15:18 ` Luben Tuikov
2005-08-15 15:23   ` James Bottomley
2005-08-15 15:41     ` Luben Tuikov
2005-08-15 15:53       ` James Bottomley
2005-08-15 16:10         ` Luben Tuikov
2005-08-15 16:32           ` James Bottomley [this message]
2005-08-15 16:40             ` Luben Tuikov
2005-08-15 15:25   ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2005-08-18 14:17 James.Smart

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=1124123533.5089.24.camel@mulgrave \
    --to=jejb@steeleye.com \
    --cc=hch@lst.de \
    --cc=linux-scsi@vger.kernel.org \
    --cc=luben_tuikov@adaptec.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.