[Cbe-oss-dev] [PATCH 1/5] usb: Fix PS3 EHCI suspend

[Geoff Levand]

On Wed, 2011-11-23 at 22:40 -0500, Alan Stern wrote: 
> On Wed, 23 Nov 2011, Geoff Levand wrote:
> > > > > >   kernel BUG at drivers/usb/host/ehci-mem.c:74!
> > > > > 
> > > > > This means some qh was still linked (or something similar) when it was 
> > > > > removed.  I don't see how these are connected to the root hub stopping.
> > > > 
> > > > I think it is because when the HC frame index stops incrementing the qh
> > > > processing does not finish up, so it is still linked.
> > > 
> > > I would expect the IAA watchdog timer to catch something like that.
> > 
> > I'll do some more investigation into this error and give a more
> > detailed report.

I have not been able to reproduce this with current kernels.  It could
be due to a USB code change, or due to a change in kernel config options
selected in my .config.  I'll continue to try to reproduce.

Here is what was seen:

<6>ps3-ehci-driver sb_08: USB bus 4 deregistered
<6>ps3-ohci-driver sb_07: remove, state 4
<6>usb usb1: USB disconnect, address 1
<6>ps3-ohci-driver sb_07: USB bus 1 deregistered
<6>ps3-ehci-driver sb_06: remove, state 4
<6>usb usb2: USB disconnect, address 1
<6>usb 2-1: USB disconnect, address 2
<6>usb 2-2: USB disconnect, address 3
<0>------------[ cut here ]------------
<2>kernel BUG at ps3-linux-dev/drivers/usb/host/ehci-mem.c:74!
Oops: Exception in kernel mode, sig: 5 [#1]
SMP NR_CPUS=2 PS3
Modules linked in: rfcomm sco bnep l2cap btusb bluetooth usb_storage snd_ps3 snd_pcm ps3_lpm snd_page_alloc snd_timer snd ehci_hcd ohci_hcd sg soundcore usbcore [last unloaded: scsi_wait_scan]
NIP: d0000000005452a8 LR: d0000000005454a8 CTR: 0000000000000000
REGS: c00000001e1d35b0 TRAP: 0700   Not tainted  (2.6.28-rc3-00872-ga6d1ff4)
MSR: 8000000000028032 <EE,IR,DR>  CR: 24000428  XER: 00000000
TASK = c000000006b122c0[1049] 'reboot' THREAD: c00000001e1d0000 CPU: 0
<6>GPR00: c000000006f74140 c00000001e1d3830 d0000000005526f8 c000000006d9f570 
<6>GPR04: 0000000000000000 d00000000054549c 0000000000000183 c000000006b122c0 
<6>GPR08: 0000000000000000 c000000006f74058 0000000000000000 c000000000b259b0 
<6>GPR12: 0000000024000424 c00000000059d300 00000000100f6210 0000000000000000 
<6>GPR16: 00000000100f4a38 00000000100cfb88 00000000100f4bdc 00000000100f4b4c 
<6>GPR20: 00000000100f495c 00000000100f496c 00000000100f4a60 0000000000000000 
<6>GPR24: 0000000000000000 0000000000000000 d0000000003036c8 0000000000000000 
<6>GPR28: c000000006d9f570 c000000006f74000 d000000000552080 c000000006d9f570 
NIP [d0000000005452a8] .ehci_mem_cleanup+0x60/0x198 [ehci_hcd]
LR [d0000000005454a8] .ehci_stop+0xc8/0x120 [ehci_hcd]
Call Trace:
[c00000001e1d3830] [c00000000033ad20] ._spin_unlock_irq+0x44/0x5c (unreliable)
[c00000001e1d38c0] [d0000000005454a8] .ehci_stop+0xc8/0x120 [ehci_hcd]
[c00000001e1d3950] [d0000000002dbfec] .usb_remove_hcd+0x10c/0x184 [usbcore]
[c00000001e1d39f0] [d00000000053eba0] .ps3_ehci_remove+0x48/0xe0 [ehci_hcd]
[c00000001e1d3a90] [c000000000041508] .ps3_system_bus_shutdown+0x70/0x90
[c00000001e1d3b00] [c000000000210fc4] .device_shutdown+0x88/0xf0
[c00000001e1d3b90] [c000000000066824] .kernel_restart_prepare+0x38/0x58
[c00000001e1d3c10] [c0000000000668b0] .kernel_restart+0x20/0x6c
[c00000001e1d3c90] [c000000000066ad8] .sys_reboot+0x1d4/0x238
[c00000001e1d3e30] [c0000000000074dc] syscall_exit+0x0/0x40
Instruction dump:
79290020 2fa90000 913d0080 409e005c 393d0058 e81d0058 eb9d0078 7fa04800 
409e0010 e81d0050 2fa00000 419e000c <0fe00000> 48000000 e89d0068 2fa40000 
<4>---[ end trace b4f42db0532e8a81 ]---

-Geoff