From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: "Martin K. Petersen" <martin.petersen@oracle.com>,
Matthew Wilcox <matthew@wil.cx>,
Matthew Wilcox <willy@linux.intel.com>,
SCSI development list <linux-scsi@vger.kernel.org>
Subject: Re: Bugs in scsi_vpd_inquiry()
Date: Mon, 10 Aug 2009 16:49:21 -0400 [thread overview]
Message-ID: <yq13a7zfl7y.fsf@sermon.lab.mkp.net> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0908101518020.4223-100000@iolanthe.rowland.org> (Alan Stern's message of "Mon, 10 Aug 2009 16:13:40 -0400 (EDT)")
>>>>> "Alan" == Alan Stern <stern@rowland.harvard.edu> writes:
>> First of all we're not going to send EVPD=1 out to devices reporting
>> SCSI_SPC_2 or lower anymore, making some of this discussion moot in
>> the short term.
Alan> Ah. Has this change been posted anywhere yet?
ffd4bc2a984fab40ed969163efdff321490e8032
Alan> Does Vista also do this querying? I've got access to a machine
Alan> running Vista, so I can test the USB storage stack behavior.
Dunno.
Alan> How will Windows 7 deal with all those old buggy USB devices?
Alan> (You probably don't have any idea...)
Nope :)
Alan> No, there isn't. There are USB device IDs, but they are assigned
Alan> more or less randomly by the manufacturers -- they don't form a
Alan> nicely increasing sequence of revision numbers.
But let's assume for a second that USB drives with 4KB physical blocks
will be USB 3.0. I think that's a likely scenario. Couldn't we do
something like?
/* Dumb down SCSI level for USB 1.1 and 2.0 devices */
if (le16_to_cpu(udev->descriptor.bcdUSB) <= 0x0200
&& sdev->scsi_level > SCSI_2)
sdev->sdev_target->scsi_level = sdev->scsi_level = SCSI_2;
I'm sure that won't catch all devices. But it would at least be a step
in the right direction. And if that breaks we can revisit.
Alan> Even worse, the device-reported SCSI rev. is completely
Alan> independent of the ID for the bridge chip, which is where the
Alan> failures seem to occur. (Although it's hard to be certain which
Alan> component is failing.)
If the bridge doesn't provide the SCSI rev. where does it come from? Or
are you saying there's a USB "target" chip and then a USB-SATA bridge
chip behind it?
Alan> I'm not keen on the idea of collecting more failure data by
Alan> changing the driver so as to cause failures!
I wasn't thinking a hard fail. More like a:
printk(KERN_ERR "Please mail this info to linux-scsi@vger...");
and have that sit in Fedora/Ubuntu for 6 months.
But if there's nothing we can key off of then there's no point.
--
Martin K. Petersen Oracle Linux Engineering
next prev parent reply other threads:[~2009-08-10 20:49 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-10 14:41 Bugs in scsi_vpd_inquiry() Alan Stern
2009-08-10 14:58 ` Matthew Wilcox
2009-08-10 15:32 ` Alan Stern
2009-08-10 17:08 ` Martin K. Petersen
2009-08-10 20:13 ` Alan Stern
2009-08-10 20:49 ` Martin K. Petersen [this message]
2009-08-10 21:14 ` Alan Stern
2009-08-10 22:47 ` Martin K. Petersen
2009-08-11 14:35 ` Alan Stern
2009-08-10 21:53 ` Douglas Gilbert
2009-08-10 22:52 ` Martin K. Petersen
2009-08-11 16:04 ` Matthew Wilcox
2009-08-11 7:07 ` Boaz Harrosh
2009-08-11 14:53 ` Alan Stern
2009-08-11 15:13 ` James Bottomley
2009-08-11 15:18 ` Boaz Harrosh
2009-08-11 15:27 ` James Bottomley
2009-08-11 15:38 ` Alan Stern
2009-08-11 15:57 ` Matthew Wilcox
2009-08-11 15:59 ` James Bottomley
2009-08-11 16:14 ` Alan Stern
2009-08-11 16:24 ` James Bottomley
2009-08-13 13:58 ` Boaz Harrosh
2009-08-13 14:15 ` James Bottomley
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=yq13a7zfl7y.fsf@sermon.lab.mkp.net \
--to=martin.petersen@oracle.com \
--cc=linux-scsi@vger.kernel.org \
--cc=matthew@wil.cx \
--cc=stern@rowland.harvard.edu \
--cc=willy@linux.intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox