Re: [linux-usb-devel] usb-storage autosuspend bug?
On Sun, Jul 29, 2007 at 08:00:01PM -0700, Linus Torvalds wrote: > > > On Fri, 27 Jul 2007, Alan Stern wrote: > > > > I don't think it's a refcounting problem. My guess is that the > > underlying cause is the bug in your urb->status removal patch for > > usb_start_wait_urb() -- the one I fixed here: > > > > http://marc.info/?l=linux-usb-devel=118531582013355=2 > > > > Of course, my guess could be wrong. I haven't tried to recreate > > Linus's bug with that patch reverted. But with the patch present I was > > not able to duplicate the bug. > > I finally got around to testing this, and while I still have problems with > that reader, I can't reproduce the oops with that patch in place either. > > Of course, I only ever saw the oops once, so I really don't know how > reproducible it was. Maybe you have to remove the USB reader at _just_ the > right moment. But I did get > > ... > usb 2-6: reset high speed USB device using ehci_hcd and address 4 > usb 2-6: USB disconnect, address 4 > sd 6:0:0:1: [sdc] Result: hostbyte=0x07 driverbyte=0x00 > end_request: I/O error, dev sdc, sector 1999855 > ... > scsi 6:0:0:1: rejecting I/O to dead device > > so the testing was done with the kind of behaviour that resulted in an > oops and a dead khubd last time around. > > So I guess I'll ack the patch for whatever that is worth. Ok, the patch is in the series I just sent to you. If you run into that bug again, please let us know. thanks, greg k-h - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
On Sun, Jul 29, 2007 at 08:00:01PM -0700, Linus Torvalds wrote: On Fri, 27 Jul 2007, Alan Stern wrote: I don't think it's a refcounting problem. My guess is that the underlying cause is the bug in your urb-status removal patch for usb_start_wait_urb() -- the one I fixed here: http://marc.info/?l=linux-usb-develm=118531582013355w=2 Of course, my guess could be wrong. I haven't tried to recreate Linus's bug with that patch reverted. But with the patch present I was not able to duplicate the bug. I finally got around to testing this, and while I still have problems with that reader, I can't reproduce the oops with that patch in place either. Of course, I only ever saw the oops once, so I really don't know how reproducible it was. Maybe you have to remove the USB reader at _just_ the right moment. But I did get ... usb 2-6: reset high speed USB device using ehci_hcd and address 4 usb 2-6: USB disconnect, address 4 sd 6:0:0:1: [sdc] Result: hostbyte=0x07 driverbyte=0x00 end_request: I/O error, dev sdc, sector 1999855 ... scsi 6:0:0:1: rejecting I/O to dead device so the testing was done with the kind of behaviour that resulted in an oops and a dead khubd last time around. So I guess I'll ack the patch for whatever that is worth. Ok, the patch is in the series I just sent to you. If you run into that bug again, please let us know. thanks, greg k-h - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
On Fri, 27 Jul 2007, Alan Stern wrote: > > I don't think it's a refcounting problem. My guess is that the > underlying cause is the bug in your urb->status removal patch for > usb_start_wait_urb() -- the one I fixed here: > > http://marc.info/?l=linux-usb-devel=118531582013355=2 > > Of course, my guess could be wrong. I haven't tried to recreate > Linus's bug with that patch reverted. But with the patch present I was > not able to duplicate the bug. I finally got around to testing this, and while I still have problems with that reader, I can't reproduce the oops with that patch in place either. Of course, I only ever saw the oops once, so I really don't know how reproducible it was. Maybe you have to remove the USB reader at _just_ the right moment. But I did get ... usb 2-6: reset high speed USB device using ehci_hcd and address 4 usb 2-6: USB disconnect, address 4 sd 6:0:0:1: [sdc] Result: hostbyte=0x07 driverbyte=0x00 end_request: I/O error, dev sdc, sector 1999855 ... scsi 6:0:0:1: rejecting I/O to dead device so the testing was done with the kind of behaviour that resulted in an oops and a dead khubd last time around. So I guess I'll ack the patch for whatever that is worth. Over to you, Greg.. Linus - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
On Fri, 27 Jul 2007, Alan Stern wrote: I don't think it's a refcounting problem. My guess is that the underlying cause is the bug in your urb-status removal patch for usb_start_wait_urb() -- the one I fixed here: http://marc.info/?l=linux-usb-develm=118531582013355w=2 Of course, my guess could be wrong. I haven't tried to recreate Linus's bug with that patch reverted. But with the patch present I was not able to duplicate the bug. I finally got around to testing this, and while I still have problems with that reader, I can't reproduce the oops with that patch in place either. Of course, I only ever saw the oops once, so I really don't know how reproducible it was. Maybe you have to remove the USB reader at _just_ the right moment. But I did get ... usb 2-6: reset high speed USB device using ehci_hcd and address 4 usb 2-6: USB disconnect, address 4 sd 6:0:0:1: [sdc] Result: hostbyte=0x07 driverbyte=0x00 end_request: I/O error, dev sdc, sector 1999855 ... scsi 6:0:0:1: rejecting I/O to dead device so the testing was done with the kind of behaviour that resulted in an oops and a dead khubd last time around. So I guess I'll ack the patch for whatever that is worth. Over to you, Greg.. Linus - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
On Fri, 27 Jul 2007, Oliver Neukum wrote: > Am Donnerstag 26 Juli 2007 schrieb Greg KH: > > Alan and Oliver, was this caused by the autosuspend changes for > > usb-storage? > > The oops itself looks like refcounting. What caused the initial io error > does not become clear from the log. It is possible that the device cannot > stand suspension. But there's no evidence from that. On the contrary, > these devices usually do an unplug/plug cycle, which is not in the log. I don't think it's a refcounting problem. My guess is that the underlying cause is the bug in your urb->status removal patch for usb_start_wait_urb() -- the one I fixed here: http://marc.info/?l=linux-usb-devel=118531582013355=2 Of course, my guess could be wrong. I haven't tried to recreate Linus's bug with that patch reverted. But with the patch present I was not able to duplicate the bug. Alan Stern - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
Am Donnerstag 26 Juli 2007 schrieb Greg KH: > Alan and Oliver, was this caused by the autosuspend changes for > usb-storage? The oops itself looks like refcounting. What caused the initial io error does not become clear from the log. It is possible that the device cannot stand suspension. But there's no evidence from that. On the contrary, these devices usually do an unplug/plug cycle, which is not in the log. Regards Oliver - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
Am Donnerstag 26 Juli 2007 schrieb Greg KH: Alan and Oliver, was this caused by the autosuspend changes for usb-storage? The oops itself looks like refcounting. What caused the initial io error does not become clear from the log. It is possible that the device cannot stand suspension. But there's no evidence from that. On the contrary, these devices usually do an unplug/plug cycle, which is not in the log. Regards Oliver - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [linux-usb-devel] usb-storage autosuspend bug?
On Fri, 27 Jul 2007, Oliver Neukum wrote: Am Donnerstag 26 Juli 2007 schrieb Greg KH: Alan and Oliver, was this caused by the autosuspend changes for usb-storage? The oops itself looks like refcounting. What caused the initial io error does not become clear from the log. It is possible that the device cannot stand suspension. But there's no evidence from that. On the contrary, these devices usually do an unplug/plug cycle, which is not in the log. I don't think it's a refcounting problem. My guess is that the underlying cause is the bug in your urb-status removal patch for usb_start_wait_urb() -- the one I fixed here: http://marc.info/?l=linux-usb-develm=118531582013355w=2 Of course, my guess could be wrong. I haven't tried to recreate Linus's bug with that patch reverted. But with the patch present I was not able to duplicate the bug. Alan Stern - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/