[PATCH] Revert "ALSA: hda: Add support for Loongson 7A1000 controller"
This reverts commit 61eee4a7fc40 ("ALSA: hda: Add support for Loongson 7A1000 controller").
With this patch, there exists the following error on the Loongson LS7A platform:
[ 216.639938] rcu: INFO: rcu_preempt self-detected stall on CPU [ 216.645685] rcu: 0-....: (1 GPs behind) idle=d5a/1/0x4000000000000004 softirq=562/563 fqs=16476 [ 216.654565] (t=53772 jiffies g=-463 q=11976) [ 216.658923] NMI backtrace for cpu 0 [ 216.662417] CPU: 0 PID: 68 Comm: kworker/0:2 Not tainted 5.8.0+ #3 [ 216.668587] Hardware name: , BIOS [ 216.672174] Workqueue: events azx_probe_work [snd_hda_intel] [ 216.677829] Stack : 0000000000000000 0000000000000000 ffffffff95004ce0 d786f9efa2288403 [ 216.685848] d786f9efa2288403 0000000000000000 98000001102638c8 ffffffff80cee270 [ 216.693866] 0000000000000000 0000000000000000 0000000000000000 00000000000002b4 [ 216.701883] 206b726f775f6562 0000000000000001 0000000000000000 ffffffff80f30000 [ 216.709902] ffffffff80f30000 ffffffff80d90000 0000000000000000 0000000000000000 [ 216.717919] 0000000000000000 0000000000000000 0000000000000000 ffffffff80d90000 [ 216.725937] ffffffff80d90000 0000000000000007 ffffffff806aff18 0000000000000000 [ 216.733955] ffffffff80f00000 9800000110cc4000 98000001102638c0 ffffffff80d9db80 [ 216.741974] ffffffff8065a740 0000000000000000 0000000000000000 0000000000000000 [ 216.749991] 000073746e657665 0000000000000000 ffffffff80211a64 d786f9efa2288403 [ 216.758009] ... [ 216.760464] Call Trace: [ 216.762920] [<ffffffff80211a64>] show_stack+0x9c/0x130 [ 216.768058] [<ffffffff8065a740>] dump_stack+0xb0/0xf0 [ 216.773110] [<ffffffff80665774>] nmi_cpu_backtrace+0x134/0x140 [ 216.778939] [<ffffffff80665910>] nmi_trigger_cpumask_backtrace+0x190/0x200 [ 216.785805] [<ffffffff802b1abc>] rcu_dump_cpu_stacks+0x12c/0x190 [ 216.791806] [<ffffffff802b08cc>] rcu_sched_clock_irq+0xa2c/0xfc8 [ 216.797808] [<ffffffff802b91d4>] update_process_times+0x2c/0xb8 [ 216.803724] [<ffffffff802cad80>] tick_sched_timer+0x40/0xb8 [ 216.809293] [<ffffffff802ba5f0>] __hrtimer_run_queues+0x118/0x1d0 [ 216.815380] [<ffffffff802bab74>] hrtimer_interrupt+0x12c/0x2d8 [ 216.821208] [<ffffffff8021547c>] c0_compare_interrupt+0x74/0xa0 [ 216.827124] [<ffffffff80296bd0>] __handle_irq_event_percpu+0xa8/0x198 [ 216.833558] [<ffffffff80296cf0>] handle_irq_event_percpu+0x30/0x90 [ 216.839732] [<ffffffff8029d958>] handle_percpu_irq+0x88/0xb8 [ 216.845388] [<ffffffff80296124>] generic_handle_irq+0x44/0x60 [ 216.851131] [<ffffffff80b3cfd0>] do_IRQ+0x18/0x28 [ 216.855838] [<ffffffff8067ace4>] plat_irq_dispatch+0x64/0x100 [ 216.861579] [<ffffffff80209a20>] handle_int+0x140/0x14c [ 216.866802] [<ffffffff802402e8>] irq_exit+0xf8/0x100
This is because AZX_DRIVER_GENERIC can not work well for Loongson LS7A HDA controller, it needs some workarounds which are not merged into the upstream kernel, so it should revert this patch now.
There will be a better solution that has been tested carefully to support Loongson LS7A HDA controller.
Fixes: 61eee4a7fc40 ("ALSA: hda: Add support for Loongson 7A1000 controller") Cc: stable@vger.kernel.org # 5.9-rc1+ Signed-off-by: Tiezhu Yang yangtiezhu@loongson.cn --- sound/pci/hda/hda_intel.c | 2 -- 1 file changed, 2 deletions(-)
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index e34a4d5..0f86e37 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -2745,8 +2745,6 @@ static const struct pci_device_id azx_ids[] = { .driver_data = AZX_DRIVER_GENERIC | AZX_DCAPS_PRESET_ATI_HDMI }, /* Zhaoxin */ { PCI_DEVICE(0x1d17, 0x3288), .driver_data = AZX_DRIVER_ZHAOXIN }, - /* Loongson */ - { PCI_DEVICE(0x0014, 0x7a07), .driver_data = AZX_DRIVER_GENERIC }, { 0, } }; MODULE_DEVICE_TABLE(pci, azx_ids);
On Tue, 25 Aug 2020 10:25:03 +0200, Tiezhu Yang wrote:
This reverts commit 61eee4a7fc40 ("ALSA: hda: Add support for Loongson 7A1000 controller").
With this patch, there exists the following error on the Loongson LS7A platform:
[ 216.639938] rcu: INFO: rcu_preempt self-detected stall on CPU [ 216.645685] rcu: 0-....: (1 GPs behind) idle=d5a/1/0x4000000000000004 softirq=562/563 fqs=16476 [ 216.654565] (t=53772 jiffies g=-463 q=11976) [ 216.658923] NMI backtrace for cpu 0 [ 216.662417] CPU: 0 PID: 68 Comm: kworker/0:2 Not tainted 5.8.0+ #3 [ 216.668587] Hardware name: , BIOS [ 216.672174] Workqueue: events azx_probe_work [snd_hda_intel] [ 216.677829] Stack : 0000000000000000 0000000000000000 ffffffff95004ce0 d786f9efa2288403 [ 216.685848] d786f9efa2288403 0000000000000000 98000001102638c8 ffffffff80cee270 [ 216.693866] 0000000000000000 0000000000000000 0000000000000000 00000000000002b4 [ 216.701883] 206b726f775f6562 0000000000000001 0000000000000000 ffffffff80f30000 [ 216.709902] ffffffff80f30000 ffffffff80d90000 0000000000000000 0000000000000000 [ 216.717919] 0000000000000000 0000000000000000 0000000000000000 ffffffff80d90000 [ 216.725937] ffffffff80d90000 0000000000000007 ffffffff806aff18 0000000000000000 [ 216.733955] ffffffff80f00000 9800000110cc4000 98000001102638c0 ffffffff80d9db80 [ 216.741974] ffffffff8065a740 0000000000000000 0000000000000000 0000000000000000 [ 216.749991] 000073746e657665 0000000000000000 ffffffff80211a64 d786f9efa2288403 [ 216.758009] ... [ 216.760464] Call Trace: [ 216.762920] [<ffffffff80211a64>] show_stack+0x9c/0x130 [ 216.768058] [<ffffffff8065a740>] dump_stack+0xb0/0xf0 [ 216.773110] [<ffffffff80665774>] nmi_cpu_backtrace+0x134/0x140 [ 216.778939] [<ffffffff80665910>] nmi_trigger_cpumask_backtrace+0x190/0x200 [ 216.785805] [<ffffffff802b1abc>] rcu_dump_cpu_stacks+0x12c/0x190 [ 216.791806] [<ffffffff802b08cc>] rcu_sched_clock_irq+0xa2c/0xfc8 [ 216.797808] [<ffffffff802b91d4>] update_process_times+0x2c/0xb8 [ 216.803724] [<ffffffff802cad80>] tick_sched_timer+0x40/0xb8 [ 216.809293] [<ffffffff802ba5f0>] __hrtimer_run_queues+0x118/0x1d0 [ 216.815380] [<ffffffff802bab74>] hrtimer_interrupt+0x12c/0x2d8 [ 216.821208] [<ffffffff8021547c>] c0_compare_interrupt+0x74/0xa0 [ 216.827124] [<ffffffff80296bd0>] __handle_irq_event_percpu+0xa8/0x198 [ 216.833558] [<ffffffff80296cf0>] handle_irq_event_percpu+0x30/0x90 [ 216.839732] [<ffffffff8029d958>] handle_percpu_irq+0x88/0xb8 [ 216.845388] [<ffffffff80296124>] generic_handle_irq+0x44/0x60 [ 216.851131] [<ffffffff80b3cfd0>] do_IRQ+0x18/0x28 [ 216.855838] [<ffffffff8067ace4>] plat_irq_dispatch+0x64/0x100 [ 216.861579] [<ffffffff80209a20>] handle_int+0x140/0x14c [ 216.866802] [<ffffffff802402e8>] irq_exit+0xf8/0x100
Could you edit the Oops message to drop unnecessary hex numbers, timestamps, and whatever redundant for readers?
thanks,
Takashi
On 08/25/2020 04:30 PM, Takashi Iwai wrote:
On Tue, 25 Aug 2020 10:25:03 +0200, Tiezhu Yang wrote:
This reverts commit 61eee4a7fc40 ("ALSA: hda: Add support for Loongson 7A1000 controller").
With this patch, there exists the following error on the Loongson LS7A platform:
[ 216.639938] rcu: INFO: rcu_preempt self-detected stall on CPU [ 216.645685] rcu: 0-....: (1 GPs behind) idle=d5a/1/0x4000000000000004 softirq=562/563 fqs=16476 [ 216.654565] (t=53772 jiffies g=-463 q=11976) [ 216.658923] NMI backtrace for cpu 0 [ 216.662417] CPU: 0 PID: 68 Comm: kworker/0:2 Not tainted 5.8.0+ #3 [ 216.668587] Hardware name: , BIOS [ 216.672174] Workqueue: events azx_probe_work [snd_hda_intel] [ 216.677829] Stack : 0000000000000000 0000000000000000 ffffffff95004ce0 d786f9efa2288403 [ 216.685848] d786f9efa2288403 0000000000000000 98000001102638c8 ffffffff80cee270 [ 216.693866] 0000000000000000 0000000000000000 0000000000000000 00000000000002b4 [ 216.701883] 206b726f775f6562 0000000000000001 0000000000000000 ffffffff80f30000 [ 216.709902] ffffffff80f30000 ffffffff80d90000 0000000000000000 0000000000000000 [ 216.717919] 0000000000000000 0000000000000000 0000000000000000 ffffffff80d90000 [ 216.725937] ffffffff80d90000 0000000000000007 ffffffff806aff18 0000000000000000 [ 216.733955] ffffffff80f00000 9800000110cc4000 98000001102638c0 ffffffff80d9db80 [ 216.741974] ffffffff8065a740 0000000000000000 0000000000000000 0000000000000000 [ 216.749991] 000073746e657665 0000000000000000 ffffffff80211a64 d786f9efa2288403 [ 216.758009] ... [ 216.760464] Call Trace: [ 216.762920] [<ffffffff80211a64>] show_stack+0x9c/0x130 [ 216.768058] [<ffffffff8065a740>] dump_stack+0xb0/0xf0 [ 216.773110] [<ffffffff80665774>] nmi_cpu_backtrace+0x134/0x140 [ 216.778939] [<ffffffff80665910>] nmi_trigger_cpumask_backtrace+0x190/0x200 [ 216.785805] [<ffffffff802b1abc>] rcu_dump_cpu_stacks+0x12c/0x190 [ 216.791806] [<ffffffff802b08cc>] rcu_sched_clock_irq+0xa2c/0xfc8 [ 216.797808] [<ffffffff802b91d4>] update_process_times+0x2c/0xb8 [ 216.803724] [<ffffffff802cad80>] tick_sched_timer+0x40/0xb8 [ 216.809293] [<ffffffff802ba5f0>] __hrtimer_run_queues+0x118/0x1d0 [ 216.815380] [<ffffffff802bab74>] hrtimer_interrupt+0x12c/0x2d8 [ 216.821208] [<ffffffff8021547c>] c0_compare_interrupt+0x74/0xa0 [ 216.827124] [<ffffffff80296bd0>] __handle_irq_event_percpu+0xa8/0x198 [ 216.833558] [<ffffffff80296cf0>] handle_irq_event_percpu+0x30/0x90 [ 216.839732] [<ffffffff8029d958>] handle_percpu_irq+0x88/0xb8 [ 216.845388] [<ffffffff80296124>] generic_handle_irq+0x44/0x60 [ 216.851131] [<ffffffff80b3cfd0>] do_IRQ+0x18/0x28 [ 216.855838] [<ffffffff8067ace4>] plat_irq_dispatch+0x64/0x100 [ 216.861579] [<ffffffff80209a20>] handle_int+0x140/0x14c [ 216.866802] [<ffffffff802402e8>] irq_exit+0xf8/0x100
Could you edit the Oops message to drop unnecessary hex numbers, timestamps, and whatever redundant for readers?
OK, thanks for your reply. I will do it and then send v2 soon.
Thanks, Tiezhu
thanks,
Takashi
participants (2)
-
Takashi Iwai
-
Tiezhu Yang