All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: "Martin K. Petersen" <martin.petersen@oracle.com>,
	linux-scsi@vger.kernel.org
Subject: Re: RFC: Transport identifier
Date: Thu, 26 Feb 2009 15:32:24 -0500	[thread overview]
Message-ID: <yq1prh5ey7b.fsf@sermon.lab.mkp.net> (raw)
In-Reply-To: <1235663301.19035.44.camel@localhost.localdomain> (James Bottomley's message of "Thu\, 26 Feb 2009 15\:48\:21 +0000")

>>>>> "James" == James Bottomley <James.Bottomley@HansenPartnership.com> writes:

James> I'd really rather not put transport specific knowledge back into
James> the mid-layer ... the whole idea of the transport classes was to
James> take it out as much as possible.

I felt that implementing transport classes for everything else was a
huge overkill for a simple identifier.


James> The other thought is that a lot of devices nowadays are bridged
James> (all SCSI DVDs have SPI to ATA bridges; a lot of high end USB
James> storage or enclosures has USB to ATA bridges), so a single
James> transport identifier doesn't quite cover it.

Nope.  And it was not means to be concise.  It was meant to be a hint as
to what kind of technology was at play.

To a large extent a dubious_transport bit would suffice.  But I also
wanted to remedy the lsscsi problem while I was at it.  As Doug
mentioned the current heuristics are icky.


James> The final thought is that a lot of what you're looking for is
James> actually in the PROTOCOL field of a VPD inquiry, so it might be
James> possible to use that to obviate a lot of this.

You mean the protocol mode page?  Or the version descriptors in INQUIRY?

I don't have a single device that provides the version descriptors.
Sadly.

I'm fine with the mode page approach, although a quick test shows only
very recent drives fill it out.  I'll try it on all my spindles in the
lab and see what I discover.

Maybe we can reverse the logic a bit and key off of whether the protocol
mode page is provided.  And consider devices dubious if it's not?

The thing is we need to start sending RC16 to a lot of drives very soon
because of the 4KB thing.  And as we've seen RC16 breaks a lot of junk
devices.  So we need a good indicator other than the SCSI rev. because
that unfortunately doesn't cut it.

-- 
Martin K. Petersen	Oracle Linux Engineering

  reply	other threads:[~2009-02-26 20:32 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-26  4:31 RFC: Transport identifier Martin K. Petersen
2009-02-26  4:54 ` Julian Calaby
2009-02-26  5:32   ` Joel Becker
2009-02-26 20:22     ` Martin K. Petersen
2009-02-26  9:53 ` Douglas Gilbert
2009-02-26 15:39 ` Mike Christie
2009-02-26 15:48 ` James Bottomley
2009-02-26 20:32   ` Martin K. Petersen [this message]
2009-02-27  7:33     ` FUJITA Tomonori
2009-02-28  4:13       ` Martin K. Petersen
2009-02-28  4:50         ` Matthew Wilcox
2009-02-28  5:19           ` FUJITA Tomonori
2009-02-28 15:40             ` Martin K. Petersen
2009-02-28 14:53     ` Stefan Richter
2009-02-28 15:06       ` Matthew Wilcox
2009-02-28 15:19         ` Stefan Richter
2009-02-28 15:36           ` James Bottomley
2009-02-28 15:54           ` Martin K. Petersen
2009-02-28 15:42       ` Martin K. Petersen

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=yq1prh5ey7b.fsf@sermon.lab.mkp.net \
    --to=martin.petersen@oracle.com \
    --cc=James.Bottomley@HansenPartnership.com \
    --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 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.