linux-kernel - Re: 2.6.22-rc4-mm1

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]

Message-ID: <Pine.LNX.4.44L0.0706071042410.3516-100000@iolanthe.rowland.org>
Date:	Thu, 7 Jun 2007 10:53:29 -0400 (EDT)
From:	Alan Stern <stern@...land.harvard.edu>
To:	Grant Wilson <gww@...ndive.no-ip.com>,
	Andrew Morton <akpm@...ux-foundation.org>
cc:	Kernel development list <linux-kernel@...r.kernel.org>,
	USB development list <linux-usb-devel@...ts.sourceforge.net>
Subject: Re: 2.6.22-rc4-mm1

On Wed, 6 Jun 2007, Andrew Morton wrote:

> On Wed, 6 Jun 2007 21:58:38 +0100 Grant Wilson <gww@...ndive.no-ip.com> wrote:
> 
> > On Wednesday 06 June 2007 10:07:37 Andrew Morton wrote:
> > > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.22-rc4/2.6.22-rc4-mm1/
> > 
> > Patch 'usb-try-to-debug-bug-8561' triggers when I plug in a usb flash drive:
> 
> Cool, thanks.
> 
> > [10998.881000] usb 1-10: new high speed USB device using ehci_hcd and address 3
> > [10999.001000] usb 1-10: new device found, idVendor=13fe, idProduct=1a00
> > [10999.002000] usb 1-10: new device strings: Mfr=1, Product=2, SerialNumber=3
> > [10999.016000] usb 1-10: Product: USB DISK 2.0
> > [10999.025000] usb 1-10: Manufacturer:
> > [10999.033000] usb 1-10: SerialNumber: 07720947018D
> > [10999.034000] usb 1-10: configuration #1 chosen from 1 choice
> > [10999.047000] scsi8 : SCSI emulation for USB Mass Storage devices
> > [11004.055000] WARNING: at drivers/usb/core/urb.c:293 usb_submit_urb()
> > [11004.055000]
> > [11004.055000] Call Trace:
> > [11004.055000]  [<ffffffff8020d30f>] dump_trace+0x43f/0x480
> > [11004.055000]  [<ffffffff8020d393>] show_trace+0x43/0x70
> > [11004.055000]  [<ffffffff8020d3d5>] dump_stack+0x15/0x20
> > [11004.055000]  [<ffffffff804b4314>] usb_submit_urb+0x224/0x240
> > [11004.055000]  [<ffffffff804b5ff5>] usb_sg_wait+0xd5/0x180
> > [11004.055000]  [<ffffffff804cf464>] usb_stor_bulk_transfer_sg+0xc4/0x120
> > [11004.055000]  [<ffffffff804cf611>] usb_stor_Bulk_transport+0x151/0x2e0
> > [11004.055000]  [<ffffffff804cfb57>] usb_stor_invoke_transport+0x37/0x380
> > [11004.055000]  [<ffffffff804ce9f9>] usb_stor_transparent_scsi_command+0x9/0x10
> > [11004.055000]  [<ffffffff804d0aea>] usb_stor_control_thread+0x18a/0x230
> > [11004.055000]  [<ffffffff8024927d>] kthread+0x4d/0x80
> > [11004.055000]  [<ffffffff8020c868>] child_rip+0xa/0x12
> > 
> 
> Alan, you got a bite - reel her in!

Thanks for forwarding the message.  Unforunately it's a false alarm.  
As mentioned in the original patch, the test it uses isn't precise.

To tell you the truth, I rather think there's not much point in keeping
usb-try-to-debug-bug-8561.patch around.  Anything seriously wrong that
it could catch ought to have shown up long ago.  And it is now clear
that bug 8561 has nothing to do with this; it is a programming error
common to many of the USB serial drivers.  (Still waiting to hear back 
from Paulo Pereira whether the fix to the USB Option driver works...)

My vote goes toward reverting usb-try-to-debug-bug-8561.patch.  
However just to be thorough, for anyone wants to keep it here's an
untested patch to remove those false alarms.

Alan Stern

---------------------------------

Remove some false alarms generated by usb-try-to-debug-bug-8561.patch.

Signed-off-by: Alan Stern <stern@...land.harvard.edu>

---

Index: usb-2.6/drivers/usb/core/message.c
===================================================================
--- usb-2.6.orig/drivers/usb/core/message.c
+++ usb-2.6/drivers/usb/core/message.c
@@ -404,8 +404,6 @@ int usb_sg_init (
 
 		io->urbs [i]->complete = sg_complete;
 		io->urbs [i]->context = io;
-		io->urbs [i]->status = -EINPROGRESS;
-		io->urbs [i]->actual_length = 0;
 
 		/*
 		 * Some systems need to revert to PIO when DMA is temporarily


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/