On Sun, Feb 20, 2005 at 10:44:21PM -0500, Alan Stern wrote:
> On Sun, 20 Feb 2005, Sergey Vlasov wrote:
>
> > On Thu, 17 Feb 2005 17:31:04 -0500 (EST) Alan Stern wrote:
> >
> > > James:
> > >
> > > This patch adds a NOREPORTLUN blacklist flag for a disk made by "WDC"
> > > (Winchester?). The drive's firmware crashes when it receives
> > > REPORT_LUNS, even though it claims to be SCSI rev 04.
> > >
> > > Alan Stern
> > >
> > >
> > >
> > > Signed-off-by: Alan Stern <[EMAIL PROTECTED]>
> > >
> > > ===== drivers/scsi/scsi_devinfo.c 1.14 vs edited =====
> > > --- 1.14/drivers/scsi/scsi_devinfo.c 2004-12-10 11:54:56 -05:00
> > > +++ edited/drivers/scsi/scsi_devinfo.c 2005-02-16 11:30:56 -05:00
> > > @@ -201,6 +201,7 @@
> > > {"WangDAT", "Model 2600", "01.7", BLIST_SELECT_NO_ATN},
> > > {"WangDAT", "Model 3200", "02.2", BLIST_SELECT_NO_ATN},
> > > {"WangDAT", "Model 1300", "02.4", BLIST_SELECT_NO_ATN},
> > > + {"WDC WD25", "00JB-00FUA0", NULL, BLIST_NOREPORTLUN},
> >
> > This does not look like a right solution. WD2500JB is an IDE disk made
> > by Western Digital:
> >
> > http://www.wdc.com/en/products/Products.asp?DriveID=42
> >
> > Therefore the problematic LaCie device is really an USB/Firewire-to-IDE
> > adapter with the WD2500JB drive in a box. And it is the USB-IDE part
> > which fails on the REPORT_LUNS command, not the IDE disk. Probably
> > there are other LaCie devices with the same adapter but different disks
> > inside; adding all such IDE disks to the SCSI blacklist does not seem
> > like a good solution. The usb-storage blacklist seems like a better
> > place for such entry.
>
> You are quite correct. In fact, another user with a different brand of
> USB-IDE converter just wrote in with exactly the same problem.
>
> James, please withdraw the patch above.
>
> Matt, it looks like the best way to solve this problem is to go back to
> the old strategy of always setting the SCSI revision to 2 (no matter what
> it might actually be), at least for Direct Access devices. That would
> suppress the REPORT_LUNS command. Would we lose anything by doing this?
Besides the use of REPORT_LUNS on devices which actually support it? I
don't think so...
I wonder if printing a warning if sdev->scsi_level > SCSI_2 would be
useful...
Matt
--
Matthew Dharm Home: [EMAIL PROTECTED]
Maintainer, Linux USB Mass Storage Driver
You are needink to look more evil. You likink very strong coffee?
-- Pitr to Dust Puppy
User Friendly, 10/16/1998
pgpmEVI8kSRky.pgp
Description: PGP signature
