Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 25 Aug 2020 12:02:55 +0200
Message-ID: <s5hsgcb59gw.wl-tiwai@suse.de>
From:   Takashi Iwai <tiwai@suse.de>
To:     Tiezhu Yang <yangtiezhu@loongson.cn>
Cc:     Takashi Iwai <tiwai@suse.com>, Jaroslav Kysela <perex@perex.cz>,
        alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
        Xuefeng Li <lixuefeng@loongson.cn>, <stable@vger.kernel.org>
Subject: Re: [PATCH v2] Revert "ALSA: hda: Add support for Loongson 7A1000 controller"
In-Reply-To: <1598348388-2518-1-git-send-email-yangtiezhu@loongson.cn>
References: <1598348388-2518-1-git-send-email-yangtiezhu@loongson.cn>
User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka)
 FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.3
 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO)
MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

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