lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 13 Nov 2007 20:39:23 -0800
From:	Matthew Dharm <mdharm-kernel@...-eyed-alien.net>
To:	Greg KH <greg@...ah.com>
Cc:	Gabriel C <nix.or.die@...glemail.com>,
	Alan Stern <stern@...land.harvard.edu>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kernel@...r.kernel.org, bk@...e.de, khali@...ux-fr.org
Subject: Re: 2.6.24-rc2-mm1

On Tue, Nov 13, 2007 at 07:49:24PM -0800, Greg KH wrote:
> 
> Matt, are these the errors you were worried about with the patch we were
> just talking about tha tis in my tree?

I can't tell from these logs.

The key question (in relation to the allow_restart patch) is this:  Was
everything working fine until a START_STOP was sent to the device?

The SCSI layers used to send devices START_STOP to almost every device as
part of initialization.  I think we switched all of that to use
TEST_UNIT_READY instead.

The patch you've got should re-enable START_STOP to be sent.  The SCSI
layers (I'm told, but haven't verified myself) only send START_STOP if the
device reports that it needs a startup command.

CONFIG_USB_STORAGE_DEBUG will generate a *lot* of debug output.  But, it
should be pretty easy to see if START_STOP was sent at all, and if it
caused the problems.

Matt

P.S. Worst case, the issue we're talking about here would only cause the
device firmware to crash, which would eventually lead to a disconnect.
That shouldn't have caused the much more severe problems shown in the log
you sent.

-- 
Matthew Dharm                              Home: mdharm-usb@...-eyed-alien.net 
Maintainer, Linux USB Mass Storage Driver

DP: And judging from the scores, Stef has the sma...  
T:  LET'S NOT GO THERE!
					-- Dust Puppy and Tanya
User Friendly, 12/11/1997

Content of type "application/pgp-signature" skipped

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ