Re: [PATCH v2] Revert "ALSA: hda: Add support for Loongson 7A1000 controller"

From: Tiezhu Yang <yangtiezhu@loongson.cn>
To: Huacai Chen <chenhuacai@gmail.com>, Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org, LKML <linux-kernel@vger.kernel.org>,
	stable <stable@vger.kernel.org>, Takashi Iwai <tiwai@suse.com>,
	Xuefeng Li <lixuefeng@loongson.cn>
Subject: Re: [PATCH v2] Revert "ALSA: hda: Add support for Loongson 7A1000 controller"
Date: Tue, 8 Sep 2020 11:09:31 +0800	[thread overview]
Message-ID: <bf50515b-a018-a66d-188a-4901428e66a6@loongson.cn> (raw)
In-Reply-To: <CAAhV-H5V5adhY2OjJLxW7x3zDaHGgBxxy45hjf22+qMSEBQuww@mail.gmail.com>

On 09/08/2020 08:37 AM, Huacai Chen wrote:
> Hi, all
>
> This patch should be backported to 5.4.

Hi,

Commit 61eee4a7fc40 ("ALSA: hda: Add support for Loongson
7A1000 controller") has been not yet merged into 5.4, so no
need to backport.

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/sound/pci/hda/hda_intel.c?h=v5.4

Thanks,
Tiezhu

>
> Huacai
>
> On Tue, Aug 25, 2020 at 6:03 PM Takashi Iwai <tiwai@suse.de> wrote:
>> On Tue, 25 Aug 2020 11:39:48 +0200,
>> Tiezhu Yang wrote:
>>> This reverts commit 61eee4a7fc40 ("ALSA: hda: Add support for Loongson
>>> 7A1000 controller") to fix the following error on the Loongson LS7A
>>> platform:
>>>
>>> rcu: INFO: rcu_preempt self-detected stall on CPU
>>> <SNIP>
>>> NMI backtrace for cpu 0
>>> CPU: 0 PID: 68 Comm: kworker/0:2 Not tainted 5.8.0+ #3
>>> Hardware name:  , BIOS
>>> Workqueue: events azx_probe_work [snd_hda_intel]
>>> <SNIP>
>>> Call Trace:
>>> [<ffffffff80211a64>] show_stack+0x9c/0x130
>>> [<ffffffff8065a740>] dump_stack+0xb0/0xf0
>>> [<ffffffff80665774>] nmi_cpu_backtrace+0x134/0x140
>>> [<ffffffff80665910>] nmi_trigger_cpumask_backtrace+0x190/0x200
>>> [<ffffffff802b1abc>] rcu_dump_cpu_stacks+0x12c/0x190
>>> [<ffffffff802b08cc>] rcu_sched_clock_irq+0xa2c/0xfc8
>>> [<ffffffff802b91d4>] update_process_times+0x2c/0xb8
>>> [<ffffffff802cad80>] tick_sched_timer+0x40/0xb8
>>> [<ffffffff802ba5f0>] __hrtimer_run_queues+0x118/0x1d0
>>> [<ffffffff802bab74>] hrtimer_interrupt+0x12c/0x2d8
>>> [<ffffffff8021547c>] c0_compare_interrupt+0x74/0xa0
>>> [<ffffffff80296bd0>] __handle_irq_event_percpu+0xa8/0x198
>>> [<ffffffff80296cf0>] handle_irq_event_percpu+0x30/0x90
>>> [<ffffffff8029d958>] handle_percpu_irq+0x88/0xb8
>>> [<ffffffff80296124>] generic_handle_irq+0x44/0x60
>>> [<ffffffff80b3cfd0>] do_IRQ+0x18/0x28
>>> [<ffffffff8067ace4>] plat_irq_dispatch+0x64/0x100
>>> [<ffffffff80209a20>] handle_int+0x140/0x14c
>>> [<ffffffff802402e8>] irq_exit+0xf8/0x100
>>>
>>> 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 at this time, so it should revert this patch now.
>>>
>>> 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>
>>> ---
>>>
>>> v2: update commit message
>> Applied now.  Thanks.
>>
>>
>> Takashi