February 2016 - Alsa-devel - mailman.alsa-project.org

[alsa-devel] sound: spinlock lockup in snd_seq_oss_write
by Dmitry Vyukov 15 Feb '16

15 Feb '16

Hello, The following program hangs machine: https://gist.githubusercontent.com/dvyukov/1c9eeb67414a9e7c5e52/raw/127b660… BUG: spinlock lockup suspected on CPU#2, a.out/7470 lock: 0xffff880036a9ef50, .magic: dead4ead, .owner: a.out/7466, .owner_cpu: 3 CPU: 2 PID: 7470 Comm: a.out Not tainted 4.5.0-rc2+ #319 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 00000000ffffffff ffff88005f5ff660 ffffffff82be30ad ffff880036a9ef50 ffff88005d3d5f00 ffff88005d0b4740 ffff88005f5ff698 ffffffff8146768d ffffffff85241dce ffff880000000000 ffff880036a9ef50 ffff880036a9ef60 Call Trace: [< inline >] __dump_stack lib/dump_stack.c:15 [<ffffffff82be30ad>] dump_stack+0x6f/0xa2 lib/dump_stack.c:50 [<ffffffff8146768d>] spin_dump+0x14d/0x280 kernel/locking/spinlock_debug.c:67 [< inline >] __spin_lock_debug kernel/locking/spinlock_debug.c:117 [<ffffffff814679ad>] do_raw_spin_lock+0x15d/0x2b0 kernel/locking/spinlock_debug.c:137 [< inline >] __raw_spin_lock_irqsave include/linux/spinlock_api_smp.h:119 [<ffffffff8665ea57>] _raw_spin_lock_irqsave+0xa7/0xd0 kernel/locking/spinlock.c:159 [<ffffffff85241dce>] snd_timer_start+0x10e/0x2b0 sound/core/timer.c:491 [< inline >] seq_timer_start sound/core/seq/seq_timer.c:393 [<ffffffff852bf860>] snd_seq_timer_start+0x1a0/0x2b0 sound/core/seq/seq_timer.c:405 [< inline >] snd_seq_queue_process_event sound/core/seq/seq_queue.c:687 [<ffffffff852ba514>] snd_seq_control_queue+0x304/0x8b0 sound/core/seq/seq_queue.c:748 [<ffffffff852c01b5>] event_input_timer+0x25/0x30 sound/core/seq/seq_system.c:118 [<ffffffff852b2de4>] snd_seq_deliver_single_event.constprop.11+0x3f4/0x740 sound/core/seq/seq_clientmgr.c:634 [<ffffffff852b3248>] snd_seq_deliver_event+0x118/0x800 sound/core/seq/seq_clientmgr.c:831 [<ffffffff852b4636>] snd_seq_kernel_client_dispatch+0x126/0x170 sound/core/seq/seq_clientmgr.c:2404 [<ffffffff852c92eb>] send_timer_event.isra.0+0x10b/0x150 sound/core/seq/oss/seq_oss_timer.c:153 [<ffffffff852c981a>] snd_seq_oss_timer_start+0x1ca/0x310 sound/core/seq/oss/seq_oss_timer.c:174 [<ffffffff852c9c04>] snd_seq_oss_process_timer_event+0x2a4/0x340 sound/core/seq/oss/seq_oss_timer.c:106 [< inline >] insert_queue sound/core/seq/oss/seq_oss_rw.c:170 [<ffffffff852cf7ab>] snd_seq_oss_write+0x20b/0x810 sound/core/seq/oss/seq_oss_rw.c:148 [<ffffffff852c74f9>] odev_write+0x59/0xa0 sound/core/seq/oss/seq_oss.c:179 [<ffffffff817bacd3>] __vfs_write+0x113/0x4b0 fs/read_write.c:528 [<ffffffff817bc797>] vfs_write+0x167/0x4a0 fs/read_write.c:577 [< inline >] SYSC_write fs/read_write.c:624 [<ffffffff817bfa81>] SyS_write+0x111/0x220 fs/read_write.c:616 [<ffffffff8665f0b6>] entry_SYSCALL_64_fastpath+0x16/0x7a arch/x86/entry/entry_64.S:185 Sending NMI to all CPUs: NMI backtrace for cpu 0 CPU: 0 PID: 7473 Comm: a.out Not tainted 4.5.0-rc2+ #319 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 task: ffff88003376df00 ti: ffff880032440000 task.ti: ffff880032440000 RIP: 0010:[<ffffffff82c103a7>] [<ffffffff82c103a7>] delay_tsc+0x27/0x70 RSP: 0018:ffff8800324479f8 EFLAGS: 00000006 RAX: 00000000a0066ab3 RBX: ffff880036a9ef50 RCX: 000000000000001e RDX: 0000000000000100 RSI: 00000100a0066a95 RDI: 0000000000000001 RBP: ffff8800324479f8 R08: 0000000000000000 R09: 0000000000000001 R10: ffff88003376df00 R11: ffff880036a9ef68 R12: ffff880036a9ef60 R13: 000000009a9d2d40 R14: ffff880036a9ef58 R15: 0000000099db9d6d FS: 00007f94a370e700(0000) GS:ffff88003ec00000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b CR2: 000000c82010e120 CR3: 000000003334f000 CR4: 00000000000006f0 Stack: ffff880032447a08 ffffffff82c102ea ffff880032447a40 ffffffff81467999 ffff880036a9ef50 ffff880036a9ee20 0000000000000282 ffff880035d8ab70 0000000000000000 ffff880032447a60 ffffffff8665dfbb ffffffff8523dd64 Call Trace: [<ffffffff82c102ea>] __delay+0xa/0x10 arch/x86/lib/delay.c:153 [< inline >] __spin_lock_debug kernel/locking/spinlock_debug.c:114 [<ffffffff81467999>] do_raw_spin_lock+0x149/0x2b0 kernel/locking/spinlock_debug.c:137 [< inline >] __raw_spin_lock include/linux/spinlock_api_smp.h:145 [<ffffffff8665dfbb>] _raw_spin_lock+0x3b/0x50 kernel/locking/spinlock.c:151 [< inline >] spin_lock include/linux/spinlock.h:302 [<ffffffff8523dd64>] snd_timer_start_slave+0xd4/0x200 sound/core/timer.c:460 [<ffffffff85241880>] snd_timer_continue+0x200/0x290 sound/core/timer.c:598 [< inline >] snd_timer_user_continue sound/core/timer.c:1822 [< inline >] __snd_timer_user_ioctl sound/core/timer.c:1889 [<ffffffff85246a9d>] snd_timer_user_ioctl+0xe1d/0x25f0 sound/core/timer.c:1904 [< inline >] vfs_ioctl fs/ioctl.c:43 [<ffffffff817faf7c>] do_vfs_ioctl+0x18c/0xfb0 fs/ioctl.c:674 [< inline >] SYSC_ioctl fs/ioctl.c:689 [<ffffffff817fbe2f>] SyS_ioctl+0x8f/0xc0 fs/ioctl.c:680 [<ffffffff8665f0b6>] entry_SYSCALL_64_fastpath+0x16/0x7a arch/x86/entry/entry_64.S:185 Code: 00 00 00 00 55 65 ff 05 c8 4a 40 7d 48 89 e5 65 44 8b 05 8d 2d 40 7d 0f ae e8 0f 31 48 c1 e2 20 48 89 d6 48 09 c6 0f ae e8 0f 31 <48> c1 e2 20 48 09 d0 48 89 c1 48 29 f1 48 39 f9 73 27 65 ff 0d NMI backtrace for cpu 1 CPU: 1 PID: 7421 Comm: stress Not tainted 4.5.0-rc2+ #319 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 task: ffff88003330df00 ti: ffff880032230000 task.ti: ffff880032230000 RIP: 0010:[<ffffffff82c103a7>] [<ffffffff82c103a7>] delay_tsc+0x27/0x70 RSP: 0018:ffff88003ed07c80 EFLAGS: 00000006 RAX: 00000000a0066d38 RBX: ffff880036a9ef50 RCX: 000000000000001e RDX: 0000000000000100 RSI: 00000100a0066d0f RDI: 0000000000000001 RBP: ffff88003ed07c80 R08: 0000000000000001 R09: 0000000000000001 R10: ffff88003330df00 R11: ffff880036a9ef68 R12: ffff880036a9ef60 R13: 000000009a9d2d40 R14: ffff880036a9ef58 R15: 000000009a82fe90 FS: 000000c820127700(0000) GS:ffff88003ed00000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 CR2: 000000c8201004e0 CR3: 0000000032076000 CR4: 00000000000006e0 Stack: ffff88003ed07c90 ffffffff82c102ea ffff88003ed07cc8 ffffffff81467999 ffff880036a9ef50 0000000000000082 ffff880036a9ee20 00000000000f4240 ffff880032795068 ffff88003ed07cf0 ffffffff8665ea57 ffffffff85242019 Call Trace: d [<ffffffff82c102ea>] __delay+0xa/0x10 arch/x86/lib/delay.c:153 [< inline >] __spin_lock_debug kernel/locking/spinlock_debug.c:114 [<ffffffff81467999>] do_raw_spin_lock+0x149/0x2b0 kernel/locking/spinlock_debug.c:137 [< inline >] __raw_spin_lock_irqsave include/linux/spinlock_api_smp.h:119 [<ffffffff8665ea57>] _raw_spin_lock_irqsave+0xa7/0xd0 kernel/locking/spinlock.c:159 [<ffffffff85242019>] snd_timer_interrupt+0xa9/0xc80 sound/core/timer.c:718 [<ffffffff85248cb9>] snd_hrtimer_callback+0x169/0x230 sound/core/hrtimer.c:54 [< inline >] __run_hrtimer kernel/time/hrtimer.c:1248 [<ffffffff814b7dc3>] __hrtimer_run_queues+0x363/0xc20 kernel/time/hrtimer.c:1312 [<ffffffff814b9de2>] hrtimer_interrupt+0x182/0x430 kernel/time/hrtimer.c:1346 [<ffffffff8124e1d2>] local_apic_timer_interrupt+0x72/0xe0 arch/x86/kernel/apic/apic.c:907 [<ffffffff812516f9>] smp_apic_timer_interrupt+0x79/0xa0 arch/x86/kernel/apic/apic.c:931 [<ffffffff8665feac>] apic_timer_interrupt+0x8c/0xa0 arch/x86/entry/entry_64.S:520 [<ffffffff817ab804>] create_object+0x34/0x2d0 mm/kmemleak.c:546 [<ffffffff86641fb3>] kmemleak_alloc+0x63/0xa0 mm/kmemleak.c:916 [< inline >] kmemleak_alloc_recursive include/linux/kmemleak.h:47 [< inline >] slab_post_alloc_hook mm/slub.c:1337 [< inline >] slab_alloc_node mm/slub.c:2596 [< inline >] slab_alloc mm/slub.c:2604 [<ffffffff81760843>] kmem_cache_alloc+0x153/0x2e0 mm/slub.c:2609 [<ffffffff8181b1b5>] dup_fd+0x85/0xa90 fs/file.c:298 [< inline >] copy_files kernel/fork.c:1033 [<ffffffff8134efed>] copy_process.part.35+0x188d/0x57a0 kernel/fork.c:1436 [< inline >] copy_process kernel/fork.c:1274 [<ffffffff8135322c>] _do_fork+0x1bc/0xcb0 kernel/fork.c:1723 [< inline >] SYSC_clone kernel/fork.c:1832 [<ffffffff81353df7>] SyS_clone+0x37/0x50 kernel/fork.c:1826 [<ffffffff8665f0b6>] entry_SYSCALL_64_fastpath+0x16/0x7a arch/x86/entry/entry_64.S:185 Code: 00 00 00 00 55 65 ff 05 c8 4a 40 7d 48 89 e5 65 44 8b 05 8d 2d 40 7d 0f ae e8 0f 31 48 c1 e2 20 48 89 d6 48 09 c6 0f ae e8 0f 31 <48> c1 e2 20 48 09 d0 48 89 c1 48 29 f1 48 39 f9 73 27 65 ff 0d NMI backtrace for cpu 2 CPU: 2 PID: 7470 Comm: a.out Not tainted 4.5.0-rc2+ #319 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 task: ffff88005d0b4740 ti: ffff88005f5f8000 task.ti: ffff88005f5f8000 RIP: 0010:[<ffffffff81262066>] [<ffffffff81262066>] flat_send_IPI_mask+0x156/0x290 RSP: 0018:ffff88005f5ff5d0 EFLAGS: 00000046 RAX: 0000000000000000 RBX: 0000000000000c00 RCX: 0000000000000000 RDX: 0000000000000c00 RSI: 0000000000000002 RDI: ffffffffff5fc300 RBP: ffff88005f5ff5f8 R08: 0000000000000001 R09: 0000000000000000 R10: fffffbfff11c14aa R11: 1ffffffff13552ed R12: 0000000000000082 BUG: spinlock lockup suspected on CPU#1, stress/7421 lock: 0xffff880036a9ef50, .magic: dead4ead, .owner: a.out/7466, .owner_cpu: 3 CPU: 1 PID: 7421 Comm: stress Not tainted 4.5.0-rc2+ #319 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 00000000ffffffff ffff88003ed07c58 ffffffff82be30ad ffff880036a9ef50 ffff88005d3d5f00 ffff88003330df00 ffff88003ed07c90 ffffffff8146768d ffffffff85242019 0000000000000000 ffff880036a9ef50 ffff880036a9ef60 Call Trace: <IRQ> [< inline >] __dump_stack lib/dump_stack.c:15 <IRQ> [<ffffffff82be30ad>] dump_stack+0x6f/0xa2 lib/dump_stack.c:50 [<ffffffff8146768d>] spin_dump+0x14d/0x280 kernel/locking/spinlock_debug.c:67 [< inline >] __spin_lock_debug kernel/locking/spinlock_debug.c:117 [<ffffffff814679ad>] do_raw_spin_lock+0x15d/0x2b0 kernel/locking/spinlock_debug.c:137 [< inline >] __raw_spin_lock_irqsave include/linux/spinlock_api_smp.h:119 [<ffffffff8665ea57>] _raw_spin_lock_irqsave+0xa7/0xd0 kernel/locking/spinlock.c:159 [<ffffffff85242019>] snd_timer_interrupt+0xa9/0xc80 sound/core/timer.c:718 [<ffffffff85248cb9>] snd_hrtimer_callback+0x169/0x230 sound/core/hrtimer.c:54 [< inline >] __run_hrtimer kernel/time/hrtimer.c:1248 [<ffffffff814b7dc3>] __hrtimer_run_queues+0x363/0xc20 kernel/time/hrtimer.c:1312 [<ffffffff814b9de2>] hrtimer_interrupt+0x182/0x430 kernel/time/hrtimer.c:1346 [<ffffffff8124e1d2>] local_apic_timer_interrupt+0x72/0xe0 arch/x86/kernel/apic/apic.c:907 [<ffffffff812516f9>] smp_apic_timer_interrupt+0x79/0xa0 arch/x86/kernel/apic/apic.c:931 [<ffffffff8665feac>] apic_timer_interrupt+0x8c/0xa0 arch/x86/entry/entry_64.S:520 [<ffffffff817ab804>] create_object+0x34/0x2d0 mm/kmemleak.c:546 [<ffffffff86641fb3>] kmemleak_alloc+0x63/0xa0 mm/kmemleak.c:916 [< inline >] kmemleak_alloc_recursive include/linux/kmemleak.h:47 [< inline >] slab_post_alloc_hook mm/slub.c:1337 [< inline >] slab_alloc_node mm/slub.c:2596 [< inline >] slab_alloc mm/slub.c:2604 [<ffffffff81760843>] kmem_cache_alloc+0x153/0x2e0 mm/slub.c:2609 [<ffffffff8181b1b5>] dup_fd+0x85/0xa90 fs/file.c:298 [< inline >] copy_files kernel/fork.c:1033 [<ffffffff8134efed>] copy_process.part.35+0x188d/0x57a0 kernel/fork.c:1436 [< inline >] copy_process kernel/fork.c:1274 [<ffffffff8135322c>] _do_fork+0x1bc/0xcb0 kernel/fork.c:1723 [< inline >] SYSC_clone kernel/fork.c:1832 [<ffffffff81353df7>] SyS_clone+0x37/0x50 kernel/fork.c:1826 [<ffffffff8665f0b6>] entry_SYSCALL_64_fastpath+0x16/0x7a arch/x86/entry/entry_64.S:185 R13: 000000000f000000 R14: ffffffff87b00c80 R15: 0000000000000002 FS: 00007fcabc427700(0000) GS:ffff88006d600000(0000) knlGS:0000000000000000 CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b CR2: 0000000020028073 CR3: 000000005c829000 CR4: 00000000000006e0 Stack: ffffffff87b00c80 ffffffff888be0a0 0000000000000007 fffffbfff111761c ffff88006d71a540 ffff88005f5ff618 ffffffff81257bfe ffffffff86ac7020 dffffc0000000000 ffff88005f5ff688 ffffffff82bee6ac ffffffff811aa610 Call Trace: [<ffffffff81257bfe>] nmi_raise_cpu_backtrace+0x5e/0x80 arch/x86/kernel/apic/hw_nmi.c:33 [<ffffffff82bee6ac>] nmi_trigger_all_cpu_backtrace+0x54c/0x5f0 lib/nmi_backtrace.c:85 [<ffffffff81257c54>] arch_trigger_all_cpu_backtrace+0x14/0x20 arch/x86/kernel/apic/hw_nmi.c:38 [< inline >] trigger_all_cpu_backtrace include/linux/nmi.h:41 [< inline >] __spin_lock_debug kernel/locking/spinlock_debug.c:119 [<ffffffff814679b7>] do_raw_spin_lock+0x167/0x2b0 kernel/locking/spinlock_debug.c:137 [< inline >] __raw_spin_lock_irqsave include/linux/spinlock_api_smp.h:119 [<ffffffff8665ea57>] _raw_spin_lock_irqsave+0xa7/0xd0 kernel/locking/spinlock.c:159 [<ffffffff85241dce>] snd_timer_start+0x10e/0x2b0 sound/core/timer.c:491 [< inline >] seq_timer_start sound/core/seq/seq_timer.c:393 [<ffffffff852bf860>] snd_seq_timer_start+0x1a0/0x2b0 sound/core/seq/seq_timer.c:405 [< inline >] snd_seq_queue_process_event sound/core/seq/seq_queue.c:687 [<ffffffff852ba514>] snd_seq_control_queue+0x304/0x8b0 sound/core/seq/seq_queue.c:748 [<ffffffff852c01b5>] event_input_timer+0x25/0x30 sound/core/seq/seq_system.c:118 [<ffffffff852b2de4>] snd_seq_deliver_single_event.constprop.11+0x3f4/0x740 sound/core/seq/seq_clientmgr.c:634 [<ffffffff852b3248>] snd_seq_deliver_event+0x118/0x800 sound/core/seq/seq_clientmgr.c:831 On commit b37a05c083c85c2657dca9bbe1f5d79dccf756d5 + latest Takashi fixes.

2 2

[alsa-devel] SGTL5000 startup error on reboot
by Erik Friesen 15 Feb '16

15 Feb '16

I find that the codec is getting a write failure at sgtl5000_fill_defaults at { SGTL5000_CHIP_ANA_POWER, 0x7060 }, about half the time following a software reboot, and with external vddd powered with 1.8v. Reading this register shows that it is already at 0x7060, so I am not sure what is up with that, but I would suspect a hardware bug in the codec. Using this code has fixed this so far. static int sgtl5000_fill_defaults(struct sgtl5000_priv *sgtl5000) { int i, ret, val, index; int reg; for (i = 0; i < ARRAY_SIZE(sgtl5000_reg_defaults); i++) { val = sgtl5000_reg_defaults[i].def; index = sgtl5000_reg_defaults[i].reg; ret = regmap_read(sgtl5000->regmap, index, &reg); if (ret) { printk(KERN_WARNING "SGTL5000 probe read failed at %i\n", i); return ret; } if (reg != val) { ret = regmap_write(sgtl5000->regmap, index, val); } if (ret) { printk(KERN_WARNING "SGTL5000 probe write failed %x at %i index %i @ %x\n", val, i, index, reg); return ret; } } return 0; }

1 1

[alsa-devel] kernel crash in snd_hda_intel
by Gabriel Feceoru 15 Feb '16

15 Feb '16

!!! This caused a regression in the i-g-t drv_module_reload_basic test. Reproducible easily on HSW (i5-4460) with: #rmmod snd_hda_intel The bisect shows this as the offending commit: commit 25e4abb33df3aafa7d1efba8f82f9178268efab1 Author: Libin Yang <libin.yang(a)linux.intel.com> Date: Tue Jan 12 11:13:27 2016 +0800 ALSA: hda - hdmi jack created based on pcm Jack is created based on pcm. Apply the acomp jack rule to dyn_pcm_assign. For dyn_pcm_assign: Driver does not use hda_jack. It operates snd_jack directly. snd_jack pointer will be stored in spec->pcm.jack instead of the current spec->acomp_jack. When pcm is assigned to pin, jack will be assigned to pin automatically. For !dyn_pcm_assign: Driver continues using hda_jack for less impact on the old cases. Pcm is statically assigned to pin. So is jack. spec->pcm.jack saves the snd_jack pointer created in hda_jack. Signed-off-by: Libin Yang <libin.yang(a)linux.intel.com> Signed-off-by: Takashi Iwai <tiwai(a)suse.de> [ 79.020523] BUG: unable to handle kernel paging request at 0000000000015d80 [ 79.021314] IP: [<ffffffff810a486b>] queued_spin_lock_slowpath+0xeb/0x180 [ 79.022125] PGD 0 [ 79.022881] Oops: 0002 [#1] SMP [ 79.023644] Modules linked in: snd_hda_codec_hdmi snd_hda_codec_realtek snd_hda_codec_generic snd_hda_intel(-) snd_hda_codec x86_pkg_temp_thermal snd_hwdep snd_hda_core i915 i2c_algo_bit drm_kms_helper syscopyarea sysfillrect sysimgblt ehci_pci fb_sys_fops r8169 ehci_hcd mii drm xhci_pci xhci_hcd video [ 79.026790] CPU: 0 PID: 1294 Comm: rmmod Tainted: G U 4.5.0-rc1+ #73 [ 79.028322] Hardware name: Dell Inc. Inspiron 3847/088DT1 , BIOS A06 01/15/2015 [ 79.029854] task: ffff8800dac90000 ti: ffff880212700000 task.ti: ffff880212700000 [ 79.031384] RIP: 0010:[<ffffffff810a486b>] [<ffffffff810a486b>] queued_spin_lock_slowpath+0xeb/0x180 [ 79.032924] RSP: 0018:ffff8802127039b0 EFLAGS: 00010002 [ 79.033698] RAX: 000000000000350c RBX: ffff8800d434ef90 RCX: ffff88021fa15d80 [ 79.035224] RDX: 0000000000015d80 RSI: 00000000d434ef88 RDI: ffff8800d434ef90 [ 79.036751] RBP: ffff8802127039b0 R08: 0000000000040000 R09: 0000000000000000 [ 79.038277] R10: ffff8800daef8998 R11: 0000000000000004 R12: 0000000000000282 [ 79.039802] R13: 0000000000000000 R14: 0000000000000006 R15: ffff8800d434ef90 [ 79.041331] FS: 00007f75be1ad700(0000) GS:ffff88021fa00000(0000) knlGS:0000000000000000 [ 79.042863] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 [ 79.043638] CR2: 0000000000015d80 CR3: 00000002101b1000 CR4: 00000000001406f0 [ 79.045166] Stack: [ 79.045926] ffff8802127039d0 ffffffff81748e59 ffff8800d434ed80 0000000000000005 [ 79.047462] ffff880212703a10 ffffffff81597e73 ffffffff817472d2 0000000000000003 [ 79.048995] ffff8800d434ecc0 0000000000000000 0000000000000001 ffff8800daef8800 [ 79.050528] Call Trace: [ 79.051293] [<ffffffff81748e59>] _raw_spin_lock_irqsave+0x39/0x50 [ 79.052071] [<ffffffff81597e73>] input_event+0x43/0x80 [ 79.052844] [<ffffffff817472d2>] ? mutex_lock+0x12/0x30 [ 79.053619] [<ffffffff8160d9fe>] snd_jack_report+0xee/0x110 [ 79.054396] [<ffffffffc02da4ea>] hdmi_present_sense+0x13a/0x390 [snd_hda_codec_hdmi] [ 79.055921] [<ffffffff8150ffee>] ? regmap_unlock_mutex+0xe/0x10 [ 79.056699] [<ffffffffc02da78d>] generic_hdmi_resume+0x4d/0x60 [snd_hda_codec_hdmi] [ 79.058235] [<ffffffffc02a8bb0>] ? snd_hda_add_imux_item+0x120/0x120 [snd_hda_codec] [ 79.059771] [<ffffffffc02a85ae>] hda_call_codec_resume+0xce/0x120 [snd_hda_codec] [ 79.061303] [<ffffffffc02a8bb0>] ? snd_hda_add_imux_item+0x120/0x120 [snd_hda_codec] [ 79.062835] [<ffffffffc02a8be5>] hda_codec_runtime_resume+0x35/0x50 [snd_hda_codec] [ 79.064368] [<ffffffff81503d68>] __rpm_callback+0x28/0x70 [ 79.065143] [<ffffffffc02a8bb0>] ? snd_hda_add_imux_item+0x120/0x120 [snd_hda_codec] [ 79.066676] [<ffffffff81503dd4>] rpm_callback+0x24/0x80 [ 79.067449] [<ffffffffc02a8bb0>] ? snd_hda_add_imux_item+0x120/0x120 [snd_hda_codec] [ 79.068983] [<ffffffff81504646>] rpm_resume+0x426/0x620 [ 79.069758] [<ffffffff815055ae>] __pm_runtime_resume+0x4e/0x70 [ 79.070536] [<ffffffff814f9453>] __device_release_driver+0x43/0x160 [ 79.071314] [<ffffffff814f9593>] device_release_driver+0x23/0x30 [ 79.072085] [<ffffffff814f8b71>] bus_remove_device+0x101/0x170 [ 79.072862] [<ffffffff814f50f9>] device_del+0x139/0x270 [ 79.073638] [<ffffffffc00a51cb>] ? widget_tree_free.isra.1+0x8b/0xa0 [snd_hda_core] [ 79.075170] [<ffffffffc00a3831>] snd_hdac_device_unregister+0x21/0x30 [snd_hda_core] [ 79.076703] [<ffffffffc02a708d>] snd_hda_codec_dev_free+0x1d/0x40 [snd_hda_codec] [ 79.078235] [<ffffffff8160ac5c>] __snd_device_free+0x4c/0x80 [ 79.079010] [<ffffffff8160afb0>] snd_device_free_all+0x30/0x60 [ 79.079787] [<ffffffff816058a4>] release_card_device+0x34/0x90 [ 79.080563] [<ffffffff814f4892>] device_release+0x32/0x90 [ 79.081338] [<ffffffff813f74da>] kobject_release+0x7a/0x190 [ 79.082113] [<ffffffff813f7397>] kobject_put+0x27/0x50 [ 79.082886] [<ffffffff814f4b67>] put_device+0x17/0x20 [ 79.083659] [<ffffffff816068e8>] snd_card_free+0x58/0x70 [ 79.084434] [<ffffffffc00b3fd1>] azx_remove+0x31/0x40 [snd_hda_intel] [ 79.085214] [<ffffffff81433da9>] pci_device_remove+0x39/0xc0 [ 79.085990] [<ffffffff814f94b1>] __device_release_driver+0xa1/0x160 [ 79.086767] [<ffffffff814f9fe6>] driver_detach+0xa6/0xb0 [ 79.087540] [<ffffffff814f8eb5>] bus_remove_driver+0x55/0xd0 [ 79.088314] [<ffffffff814fa5dc>] driver_unregister+0x2c/0x50 [ 79.089084] [<ffffffff814324d1>] pci_unregister_driver+0x21/0x90 [ 79.089863] [<ffffffffc00b5d88>] azx_driver_exit+0x10/0x288 [snd_hda_intel] [ 79.090644] [<ffffffff810dde01>] SyS_delete_module+0x1b1/0x250 [ 79.091422] [<ffffffff81002017>] ? trace_hardirqs_on_thunk+0x17/0x19 [ 79.092200] [<ffffffff8174925b>] entry_SYSCALL_64_fastpath+0x16/0x6e [ 79.092977] Code: c1 e0 10 45 31 c9 85 c0 74 44 48 89 c2 c1 e8 12 48 c1 ea 0c 83 e8 01 83 e2 30 48 98 48 81 c2 80 5d 01 00 48 03 14 c5 e0 b4 d4 81 <48> 89 0a 8b 41 08 85 c0 75 09 f3 90 8b 41 08 85 c0 74 f7 4c 8b [ 79.095373] RIP [<ffffffff810a486b>] queued_spin_lock_slowpath+0xeb/0x180 [ 79.096155] RSP <ffff8802127039b0> [ 79.096920] CR2: 0000000000015d80 [ 79.097687] ---[ end trace 5594cbffe0158d7f ]--- $lspci -nn 00:00.0 Host bridge [0600]: Intel Corporation 4th Gen Core Processor DRAM Controller [8086:0c00] (rev 06) 00:01.0 PCI bridge [0604]: Intel Corporation Xeon E3-1200 v3/4th Gen Core Processor PCI Express x16 Controller [8086:0c01] (rev 06) 00:02.0 VGA compatible controller [0300]: Intel Corporation Xeon E3-1200 v3/4th Gen Core Processor Integrated Graphics Controller [8086:0412] (rev 06) 00:03.0 Audio device [0403]: Intel Corporation Xeon E3-1200 v3/4th Gen Core Processor HD Audio Controller [8086:0c0c] (rev 06) 00:14.0 USB controller [0c03]: Intel Corporation 8 Series/C220 Series Chipset Family USB xHCI [8086:8c31] (rev 05) 00:16.0 Communication controller [0780]: Intel Corporation 8 Series/C220 Series Chipset Family MEI Controller #1 [8086:8c3a] (rev 04) 00:1a.0 USB controller [0c03]: Intel Corporation 8 Series/C220 Series Chipset Family USB EHCI #2 [8086:8c2d] (rev 05) 00:1b.0 Audio device [0403]: Intel Corporation 8 Series/C220 Series Chipset High Definition Audio Controller [8086:8c20] (rev 05) 00:1c.0 PCI bridge [0604]: Intel Corporation 8 Series/C220 Series Chipset Family PCI Express Root Port #1 [8086:8c10] (rev d5) 00:1c.3 PCI bridge [0604]: Intel Corporation 8 Series/C220 Series Chipset Family PCI Express Root Port #4 [8086:8c16] (rev d5) 00:1c.4 PCI bridge [0604]: Intel Corporation 8 Series/C220 Series Chipset Family PCI Express Root Port #5 [8086:8c18] (rev d5) 00:1d.0 USB controller [0c03]: Intel Corporation 8 Series/C220 Series Chipset Family USB EHCI #1 [8086:8c26] (rev 05) 00:1f.0 ISA bridge [0601]: Intel Corporation C220 Series Chipset Family H81 Express LPC Controller [8086:8c5c] (rev 05) 00:1f.2 SATA controller [0106]: Intel Corporation 8 Series/C220 Series Chipset Family 6-port SATA Controller 1 [AHCI mode] [8086:8c02] (rev 05) 00:1f.3 SMBus [0c05]: Intel Corporation 8 Series/C220 Series Chipset Family SMBus Controller [8086:8c22] (rev 05) 03:00.0 Ethernet controller [0200]: Realtek Semiconductor Co., Ltd. RTL8111/8168/8411 PCI Express Gigabit Ethernet Controller [10ec:8168] (rev 0c) 04:00.0 Network controller [0280]: Qualcomm Atheros QCA9565 / AR9565 Wireless Network Adapter [168c:0036] (rev 01) Regards, Gabriel

2 5

[alsa-devel] [RFC][PATCH 0/5] alsa-lib: add APIs for control element set
by Takashi Sakamoto 15 Feb '16

15 Feb '16

Hi, This patchset adds some APIs for control element set, according to design of ALSA Ctl core. In ALSA Ctl core, each sound card can have control elements. The elements are managed according to below design. * element set * A set of elements with the same attribute (i.e. name, get/put operations). * element * An element can be identified by userspace applications. * channel * An element has channels which can be changed by userspace applications or hardware device itself. * value * Each channel has its own value. The place to kept the value depends on design of each device driver (i.e. in hardware register or kernel space). There're two ways to identify the element; by identification number (numid), or by a combination of name of the element set and index from the first element in the element set. Both of device driver in kernel land and ALSA Ctl application in user land can add the element set. When userspace applications add them, ALSA Ctl core works as Inter Process Communication mechanism between event-listening applications and usual control applications. Such element is refered as 'user-defined control elements' in alsa-lib. Typically, device driver in kernel land adds one element set for a single element. So users and userspace developers are hard to realize that an existence of the element set. There're more items to describe the design. But here I focus on what relates to this patchset. Well, current alsa-lib implementation includes small bugs, some misleadings, a lack of feature related to user-defined control elements. This patchset is my attempt to improves such issues. Regards Takashi Sakamoto (5): control: initialize element channel values according to identification number control: rename variables so that it represents the number of channels which an element has control: improve API documentation control: add a function to add an element of bytes type control: add functions to add element set include/control.h | 35 +++- src/control/control.c | 484 +++++++++++++++++++++++++++++++++++++++----------- 2 files changed, 413 insertions(+), 106 deletions(-) -- 2.5.0

1 5

[alsa-devel] Plantronics DA45 cannot get freq at ep 0x4
by denk 15 Feb '16

15 Feb '16

Hi, using the Plantronics DA45 Headset Adapter leads to many lines of cannot get freq at ep 0x4 and cannot get freq at ep 0x84. I found a similar error in the following discussion: http://www.spinics.net/lists/linux-usb/msg125029.html I added the USB ID of the DA45, recompiled the kernel and the messages are gone. I would be happy if you could accept this simple addition made by the following patch: --- a/sound/usb/quirks.c 2015-12-15 06:41:43.000000000 +0100 +++ b/sound/usb/quirks.c 2016-01-28 08:15:35.207785555 +0100 @@ -1122,6 +1122,7 @@ case USB_ID(0x045E, 0x0779): /* MS Lifecam HD-3000 */ case USB_ID(0x04D8, 0xFEEA): /* Benchmark DAC1 Pre */ case USB_ID(0x074D, 0x3553): /* Outlaw RR2150 (Micronas UAC3553B) */ + case USB_ID(0x047F, 0xAA05): /* Plantronics DA45 */ return true; } return false; If there are any questions or something missing, please ask. Thanks in advance. Best denk

2 1

[alsa-devel] sound compress_driver.h: missing kernel-doc notation for a few fields
by Randy Dunlap 15 Feb '16

15 Feb '16

Hi Richard, Please add kernel-doc notation for these fields: ..//include/sound/compress_driver.h:160: warning: No description found for parameter 'id[64]' ..//include/sound/compress_driver.h:160: warning: No description found for parameter 'proc_root' ..//include/sound/compress_driver.h:160: warning: No description found for parameter 'proc_info_entry' thanks, -- ~Randy

1 0

[alsa-devel] [PATCH] Provide sequencer sound card number / PID via alsa-lib
by Martin Koegler 13 Feb '16

13 Feb '16

From: Martin Koegler <martin.koegler(a)chello.at> Signed-off-by: Martin Koegler <martin.koegler(a)chello.at> --- include/seq.h | 1 + include/sound/asequencer.h | 5 +++-- src/seq/seq.c | 11 +++++++++++ src/seq/seq_hw.c | 8 +++++++- 4 files changed, 22 insertions(+), 3 deletions(-) diff --git a/include/seq.h b/include/seq.h index 9576822..77996e5 100644 --- a/include/seq.h +++ b/include/seq.h @@ -143,6 +143,7 @@ snd_seq_client_type_t snd_seq_client_info_get_type(const snd_seq_client_info_t * const char *snd_seq_client_info_get_name(snd_seq_client_info_t *info); int snd_seq_client_info_get_broadcast_filter(const snd_seq_client_info_t *info); int snd_seq_client_info_get_error_bounce(const snd_seq_client_info_t *info); +int snd_seq_client_info_get_owner(const snd_seq_client_info_t *info); const unsigned char *snd_seq_client_info_get_event_filter(const snd_seq_client_info_t *info); int snd_seq_client_info_get_num_ports(const snd_seq_client_info_t *info); int snd_seq_client_info_get_event_lost(const snd_seq_client_info_t *info); diff --git a/include/sound/asequencer.h b/include/sound/asequencer.h index 09c8a00..7ebf7fd 100644 --- a/include/sound/asequencer.h +++ b/include/sound/asequencer.h @@ -24,7 +24,7 @@ /** version of the sequencer */ -#define SNDRV_SEQ_VERSION SNDRV_PROTOCOL_VERSION (1, 0, 1) +#define SNDRV_SEQ_VERSION SNDRV_PROTOCOL_VERSION (1, 0, 2) /** * definition of sequencer event types @@ -356,7 +356,8 @@ struct snd_seq_client_info { unsigned char event_filter[32]; /* event filter bitmap */ int num_ports; /* RO: number of ports */ int event_lost; /* number of lost events */ - char reserved[64]; /* for future use */ + int owner; /* RO: card number[kernel] / PID[user] */ + char reserved[64 - sizeof(int)]; /* for future use */ }; diff --git a/src/seq/seq.c b/src/seq/seq.c index 620ca3f..2505293 100644 --- a/src/seq/seq.c +++ b/src/seq/seq.c @@ -1522,6 +1522,17 @@ int snd_seq_client_info_get_error_bounce(const snd_seq_client_info_t *info) } /** + * \brief Get the sound card number (kernel) or owning PID + * \param info client_info container + * \return card number/PID/-1 if value is not available. + */ +int snd_seq_client_info_get_owner(const snd_seq_client_info_t *info) +{ + assert(info); + return info->owner; +} + +/** * \brief (DEPRECATED) Get the event filter bitmap of a client_info container * \param info client_info container * \return NULL if no event filter, or pointer to event filter bitmap diff --git a/src/seq/seq_hw.c b/src/seq/seq_hw.c index d033367..a4c083a 100644 --- a/src/seq/seq_hw.c +++ b/src/seq/seq_hw.c @@ -32,10 +32,11 @@ const char *_snd_module_seq_hw = ""; #ifndef DOC_HIDDEN #define SNDRV_FILE_SEQ ALSA_DEVICE_DIRECTORY "seq" #define SNDRV_FILE_ALOADSEQ ALOAD_DEVICE_DIRECTORY "aloadSEQ" -#define SNDRV_SEQ_VERSION_MAX SNDRV_PROTOCOL_VERSION(1, 0, 1) +#define SNDRV_SEQ_VERSION_MAX SNDRV_PROTOCOL_VERSION(1, 0, 2) typedef struct { int fd; + int micro_version; } snd_seq_hw_t; #endif /* DOC_HIDDEN */ @@ -100,6 +101,8 @@ static int snd_seq_hw_get_client_info(snd_seq_t *seq, snd_seq_client_info_t * in /*SYSERR("SNDRV_SEQ_IOCTL_GET_CLIENT_INFO failed");*/ return -errno; } + if (hw->micro_version < SNDRV_PROTOCOL_MICRO(SNDRV_SEQ_VERSION_MAX)) + info->owner = -1; return 0; } @@ -368,6 +371,8 @@ static int snd_seq_hw_query_next_client(snd_seq_t *seq, snd_seq_client_info_t *i /*SYSERR("SNDRV_SEQ_IOCTL_QUERY_NEXT_CLIENT failed");*/ return -errno; } + if (hw->micro_version < SNDRV_PROTOCOL_MICRO(SNDRV_SEQ_VERSION_MAX)) + info->owner = -1; return 0; } @@ -480,6 +485,7 @@ int snd_seq_hw_open(snd_seq_t **handle, const char *name, int streams, int mode) return -ENOMEM; } hw->fd = fd; + hw->micro_version = SNDRV_PROTOCOL_MICRO(ver); if (streams & SND_SEQ_OPEN_OUTPUT) { seq->obuf = (char *) malloc(seq->obufsize = SND_SEQ_OBUF_SIZE); if (!seq->obuf) { -- 2.1.4

2 2

[alsa-devel] [PATCH] Show sequencer sound card numer/PID via aconnect
by Martin Koegler 13 Feb '16

13 Feb '16

From: Martin Koegler <martin.koegler(a)chello.at> Signed-off-by: Martin Koegler <martin.koegler(a)chello.at> --- seq/aconnect/aconnect.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/seq/aconnect/aconnect.c b/seq/aconnect/aconnect.c index 8d6cebb..3508e1b 100644 --- a/seq/aconnect/aconnect.c +++ b/seq/aconnect/aconnect.c @@ -166,11 +166,19 @@ static void print_port(snd_seq_t *seq, snd_seq_client_info_t *cinfo, snd_seq_port_info_t *pinfo, int count) { if (! count) { - printf(_("client %d: '%s' [type=%s]\n"), + printf(_("client %d: '%s' [type=%s"), snd_seq_client_info_get_client(cinfo), snd_seq_client_info_get_name(cinfo), (snd_seq_client_info_get_type(cinfo) == SND_SEQ_USER_CLIENT ? _("user") : _("kernel"))); + int owner = snd_seq_client_info_get_owner(cinfo); + if (owner != -1) { + if (snd_seq_client_info_get_type(cinfo) == SND_SEQ_USER_CLIENT) + printf(",pid=%d", owner); + else + printf(",card=%d", owner); + } + printf("]\n"); } printf(" %3d '%-16s'\n", snd_seq_port_info_get_port(pinfo), -- 2.1.4

2 1

[alsa-devel] [PATCH V2 0/3] ASOC: bcm2835: move bcm2835-i2s to use clock framework
by kernel＠martin.sperl.org 13 Feb '16

13 Feb '16

From: Martin Sperl <kernel(a)martin.sperl.org> This patchset enables the bcm2835-i2s driver to use the clock framework which was introduced with commit 94cb7f76caa0 ("ARM: bcm2835: Switch to using the new clock driver support."). This commit resulted in the fact that the bcm2835-i2s driver was no longer working due to some register addresses used by 2 drivers (clk-bcm2835 and bcm2835-i2s). This patchset requires that the patchset for PCM-clock support as well as fractional/mash support is applied to the clk-bcm2835 driver and the corresponding device tree, but as the current version of the driver is not working, that should not be a problem. Note that there is one change: right now the current driver tries to calculate an optimal bclk_ratio based on its knowledge that it is using the 19.2Mhz oscillator. This computation would recommend the use of 40 or 80 bits instead of 32/64 bits that are required. Some of the DACs can not handle this, so most downstream drivers would set snd_soc_dai_set_bclk_ratio explicitly to disable this "non-power-of-2" automatic selection. So it seems wise to leave it out of the current patchset. If it is deemed necessary, then I can provide a separate patch that implements this again. Changelog: V1->V2: * moving clock patches into a separate patchset, which fixes also other issues in the clock framework * remove unnecessary bclk_size assignments Martin Sperl (3): ASoC: bcm2835: move to use the clock framework ARM: bcm2835: I2S: use new register-range and clock framework dt-bindings: bsm2835: fix bindings documentation to use new clock framework .../devicetree/bindings/sound/brcm,bcm2835-i2s.txt | 7 +- arch/arm/boot/dts/bcm2835.dtsi | 5 +- sound/soc/bcm/bcm2835-i2s.c | 284 +++++--------------- 3 files changed, 69 insertions(+), 227 deletions(-) -- 1.7.10.4

6 14

[alsa-devel] [PATCH] midi: avoid freeing umidi object twice
by Andrey Konovalov 13 Feb '16

13 Feb '16

The 'umidi' object will be free'd on the error path by snd_usbmidi_free() when tearing down the rawmidi interface. So we shouldn't try to free it in snd_usbmidi_create() after having registered the rawmidi interface. Found by KASAN. Acked-by: Clemens Ladisch <clemens(a)ladisch.de> Signed-off-by: Andrey Konovalov <andreyknvl(a)gmail.com> --- sound/usb/midi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/usb/midi.c b/sound/usb/midi.c index cc39f63..007cf58 100644 --- a/sound/usb/midi.c +++ b/sound/usb/midi.c @@ -2455,7 +2455,6 @@ int snd_usbmidi_create(struct snd_card *card, else err = snd_usbmidi_create_endpoints(umidi, endpoints); if (err < 0) { - snd_usbmidi_free(umidi); return err; } -- 1.9.1

2 1