Re: [PATCH] SCSI error handling on some USB disks needs allow_restartcp

On Tue, Nov 13, 2007 at 11:57:59AM -0500, Alan Stern wrote:
> On Tue, 13 Nov 2007, Matthew Dharm wrote:
> 
> > On Tue, Nov 13, 2007 at 03:41:21PM +0100, Jean Delvare wrote:
> > > > +	vendor = us->pusb_dev->manufacturer;
> > > > +	product = us->pusb_dev->product;
> > > > +
> > > > +	if (vendor && product &&
> > > > +	    !strcmp(vendor, "Seagate") &&
> > > > +	    (!strcmp(product, "FreeAgent Pro") ||
> > > > +	     !strcmp(product, "FreeAgentDesktop")))
> > > > +		sdev->allow_restart = 1;
> > > > +
> > > >   	/*
> > > >   	 * The UFI spec treates the Peripheral Qualifier bits in an
> > > >   	 * INQUIRY result as reserved and requires devices to set them
> > 
> > This is overkill.  It's almost certainly safe to just set allow_restart = 1
> > always, since the SCSI layer won't send a START_STOP unless it gets an
> > error, anyway.
> > 
> > Long ago, every device got a START_STOP, and a lot of them broke.  But now,
> > the devices that can't tollerate a START_STOP shouldn't trigger those code
> > paths.
> 
> Right.  In fact the patch has already been written.  See
> 
> http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-03-usb/usb-storage-always-set-the-allow_restart-flag.patch

I thought this looked familiar.  My bad.

Matt

-- 
Matthew Dharm                              Home: [email protected] 
Maintainer, Linux USB Mass Storage Driver

Hi.  I have my back hairs caught in my computer fan.
					-- Customer
User Friendly, 8/20/1998

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/

_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.7 (GNU/Linux)

iD8DBQFHOdpQHL9iwnUZqnkRAi9DAJ9y20Z1R56ipwwRA0PFeXFggKs1jwCcCADm
cCodxEX1ByVasC+QJQS06lo=
=ntcu
-----END PGP SIGNATURE-----