All of lore.kernel.org
 help / color / mirror / Atom feed
From: Douglas Gilbert <dougg@torque.net>
To: Luben Tuikov <luben@splentec.com>
Cc: linux-scsi <linux-scsi@vger.kernel.org>
Subject: Re: Autosense requirement
Date: Mon, 29 Jul 2002 13:48:35 -0400	[thread overview]
Message-ID: <3D457FF3.C7A02C57@torque.net> (raw)
In-Reply-To: 3D453CE8.5A15FCD6@splentec.com

Luben Tuikov wrote:
> 
> How safe am I to assume that SCSI LLDD will return
> autosense for a failed/incomplete command, as per
> SCSI core (mid-layer) error handling requrement?
> (e.g. ide-scsi doesn't return autosense)
> 
> In drafts I'm more and more seeing
> ``targets MUST support and enable autosense.''
> 
> If indeed we put a ``MUST'' requirement on SCSI LLDD
> to support and enable autosense, which will indeed
> streamline things, then this will save me a an _irq_
> spin lock over a simple spin lock...
> 
> What is the future of this requrement?

Luben,
In the lk 2.4 series we have both older "non-eh" lower
level drivers and a few, newer "eh" drivers. In the lk
2.5 series the mid level interface to the older "non-eh"
drivers was dropped.

Hence in the development lk 2.5 series there are many stranded
drivers, including ide-scsi, some of which have been converted "on
the fly". His is an incomplete list of issues to be addressed
in lower level (HBA) driver conversions:
  - "eh" interface [mandatory]
  - io_request_lock displaced by host_lock [mandatory]
  - PCI memory/DMA interface
  - driverfs aware
  - autosense for high performance drivers (perhaps all)

As you point out autosense support (at the device level)
is now mandatory in recent scsi drafts/standards. For
simplicity of queueing logic, the sense buffer should not
be left for the mid level to fetch (with a REQUEST SENSE).

I have attempted the conversion on several HBA drivers
(of which scsi_debug is the only one I maintain). As Ben
LaHaise has pointed out, I have skipped addressing the
"PCI memory/DMA interface" so far.

Until Doug Ledford pointed out recently what lower level 
drivers should do with regard to autosense, I was unaware
what to do. Since that time I have converted scsi_debug
to do autosense properly and submitted a patch for the
drivers/scsi/scsi_mid_low_api.txt documentation file.

Now would someone else like to step up to the plate
(or "wicket" to use the cricket metaphor) and try and
get the ide-scsi driver in lk 2.5 to do autosense?


If kernel people think that a ide-scsi "like" driver is 
going away then perhaps they should consider the new SAS
(SCSI over Serial ATA infrastructure) draft. They have
one mode illustrated in that document where the initiator
is a SCSI host and it talks to multiple serial ATA
disks via an "expander". Could be interesting.

Doug Gilbert


  reply	other threads:[~2002-07-29 17:48 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-07-29 13:02 Autosense requirement Luben Tuikov
2002-07-29 17:48 ` Douglas Gilbert [this message]
2002-07-29 19:55   ` Doug Ledford
2002-07-29 21:53     ` Mike Anderson
2002-07-29 21:58       ` Doug Ledford
2002-07-29 22:10         ` Patrick Mansfield
2002-07-29 23:00           ` Doug Ledford
2002-07-29 23:18             ` Patrick Mansfield
2002-07-31 16:04               ` James Bottomley
2002-07-31 18:02                 ` Patrick Mansfield
2002-07-30 19:58     ` Luben Tuikov
  -- strict thread matches above, loose matches on Subject: below --
2002-07-29 22:28 Ravi Anand
2002-07-29 22:36 ` Doug Ledford

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=3D457FF3.C7A02C57@torque.net \
    --to=dougg@torque.net \
    --cc=linux-scsi@vger.kernel.org \
    --cc=luben@splentec.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.