From: Matthew Wilcox <matthew@wil.cx>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>
Cc: Hannes Reinecke <hare@novell.com>,
SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: Re: Buggy Firewire bridge 'Prolific PL3507'
Date: Tue, 8 Jan 2008 11:22:05 -0700 [thread overview]
Message-ID: <20080108182205.GG16309@parisc-linux.org> (raw)
In-Reply-To: <4783B93B.70307@s5r6.in-berlin.de>
On Tue, Jan 08, 2008 at 06:56:11PM +0100, Stefan Richter wrote:
> It's hald or something like that.
Can't we get hald to look at the cached results of the inquiry command,
handily available through sysfs?
> Matthew Wilcox wrote:
> > I have a hard time thinking this needs to be handled generically
> > because READ CAPACITY isn't in the Primary Command Set, unlike INQUIRY.
> > It's mandatory for SBC devices to implement it, but other devices
> > shouldn't even implement it.
>
> It is also mandatory in RBC, which most SBP-2 to IDE or SBP-2 to SATA
> bridges implement (besides MMC, if a CD/DVD-ROM/R/W is attached).
According to SPC4r05a, READ CAPACITY(10) (opcode 25) is Mandatory for D,
W and O. READ CAPACITY (opcode 25) is Optional for R. READ CARD
CAPACITY (also opcode 25) is Mandatory for K.
D - DIRECT ACCESS BLOCK DEVICE (SBC-3)
W - WRITE ONCE BLOCK DEVICE (SBC)
O - OPTICAL MEMORY BLOCK DEVICE (SBC)
R - CD/DVD DEVICE (MMC-6)
K - OPTICAL CARD READER/WRITER DEVICE (OCRW)
According to this table (D.2 for anyone following along at home), RBC
devices are not supposed to implement it.
> I would not at all like to see a respective workaround added to the sbp2
> driver. Sbp2 is a transport layer which does not touch commands. Maybe
> an in-kernel workaround would be considered by whoever maintains the
> point where that extra INQUIRY is injected (via sd or sg, I don't know
> which).
It would be thoroughly inappropriate to put this workaround in the
midlayer. sbp2 isn't a good place either, but it's better than
sg/sd/scsi_ioctl/...
--
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours. We can't possibly take such
a retrograde step."
next prev parent reply other threads:[~2008-01-08 18:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-08 13:43 Buggy Firewire bridge 'Prolific PL3507' Hannes Reinecke
2008-01-08 14:01 ` Matthew Wilcox
2008-01-08 17:56 ` Stefan Richter
2008-01-08 18:10 ` James Bottomley
2008-01-08 20:00 ` maximilian attems
2008-01-08 20:10 ` David Zeuthen
2008-01-08 20:25 ` David Zeuthen
2008-01-08 18:22 ` Matthew Wilcox [this message]
2008-01-08 19:32 ` 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=20080108182205.GG16309@parisc-linux.org \
--to=matthew@wil.cx \
--cc=hare@novell.com \
--cc=linux-scsi@vger.kernel.org \
--cc=stefanr@s5r6.in-berlin.de \
/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