On Sat, May 15, 2010 at 11:26 AM, Felipe Contreras <felipe.contreras@xxxxxxxxx> wrote: > On Fri, May 14, 2010 at 10:49 PM, Omar Ramirez Luna <omar.ramirez@xxxxxx> wrote: >> On 5/14/2010 2:27 PM, Felipe Contreras wrote: >> [...] >>> >>> So, I tried your patches, and a simple test app worked fine without >>> modification, but a real video decoding hanged the device >>> completely... some spinlock was stuck. I don't know if it's because of >>> your patches, or because of the state of the bridge at that point. >>> I'll try first to rebase to the latest to have a better idea of what's >>> happening. >> >> You may want to check if you have this patch "DSPBRIDGE: Fix declaration and >> initialization of sync objects"[1] > > I used what Ohad suggested: 13e2573. So no, that patch is not there. > >> This is the one I know fixes a spinlock issue and which description is not >> clear enough to state that. >> >> File fixed was: drivers/dsp/bridge/wmd/msg_sm.c >> >> [1]http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=b3900e6df1f4e16b59d506a299cd5084c67a6ede > > I tried to cherry-pick that commit... didn't help: > > BUG: spinlock lockup on CPU#0, gst-launch-0.10/534, c6404304 > Backtrace: > [<c0040984>] (dump_backtrace+0x0/0xf8) from [<c036d204>] (dump_stack+0x18/0x1c) > r6:00000000 r5:c644a000 r4:c6404304 r3:00000000 > [<c036d1ec>] (dump_stack+0x0/0x1c) from [<c0213a08>] > (__spin_lock_debug+0xbc/0xd0) > [<c021394c>] (__spin_lock_debug+0x0/0xd0) from [<c0213aa0>] > (do_raw_spin_lock+0x84/0xb4) > r8:c644be54 r7:00000064 r6:c005f178 r5:80000013 r4:c6404304 > [<c0213a1c>] (do_raw_spin_lock+0x0/0xb4) from [<c036ff6c>] > (_raw_spin_lock_irqsave+0x5c/0x68) > r4:c6404304 r3:c7865a00 > [<c036ff10>] (_raw_spin_lock_irqsave+0x0/0x68) from [<c005f178>] > (completion_done+0x1c/0x38) > r6:00000001 r5:c6404304 r4:c6404300 > [<c005f15c>] (completion_done+0x0/0x38) from [<bf00eb10>] > (sync_wait_on_multiple_events+0x58/0x14c [bridgedriver]) > r5:00000002 r4:c644be4c > [<bf00eab8>] (sync_wait_on_multiple_events+0x0/0x14c [bridgedriver]) > from [<bf01158c>] (bridge_msg_get+0x154/0x240 [bridgedriver]) > [<bf011438>] (bridge_msg_get+0x0/0x240 [bridgedriver]) from > [<bf02220c>] (node_get_message+0x94/0x128 [bridgedriver]) > [<bf022178>] (node_get_message+0x0/0x128 [bridgedriver]) from > [<bf019968>] (nodewrap_get_message+0x28/0x8c [bridgedriver]) > r7:c004db48 r6:bf02e7d6 r5:c644bebc r4:c644bf04 > [<bf019940>] (nodewrap_get_message+0x0/0x8c [bridgedriver]) from > [<bf018d94>] (wcd_call_dev_io_ctl+0xf8/0x120 [bridgedriver]) > r5:00000040 r4:c644bf1c > [<bf018c9c>] (wcd_call_dev_io_ctl+0x0/0x120 [bridgedriver]) from > [<bf02a058>] (bridge_ioctl+0xac/0xcc [bridgedriver]) > r6:421e7d64 r5:c004db48 r4:c6419f00 r3:c65bb500 > [<bf029fac>] (bridge_ioctl+0x0/0xcc [bridgedriver]) from [<c0113eb4>] > (vfs_ioctl+0x34/0xb4) > r6:421e7d64 r5:bf029fac r4:c6419f00 > [<c0113e80>] (vfs_ioctl+0x0/0xb4) from [<c01142b8>] (do_vfs_ioctl+0x1c4/0x1e0) > r7:00000005 r6:c004db48 r5:421e7d64 r4:421e7d64 > [<c01140f4>] (do_vfs_ioctl+0x0/0x1e0) from [<c0114314>] (sys_ioctl+0x40/0x64) > r4:c6419f00 > [<c01142d4>] (sys_ioctl+0x0/0x64) from [<c003cfc0>] (ret_fast_syscall+0x0/0x38) > r7:00000036 r6:0006ef10 r5:4055d860 r4:0006a000 > > I'll try to rebase the patches to the latest head. Nope, didn't help. I've put the updated patches here: http://people.freedesktop.org/~felipec/dspbridge/ -- Felipe Contreras -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
