Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Vinod Koul <vinod.koul@intel.com>
Cc: patches.audio@intel.com, jeeja.kp@intel.com,
	alsa-devel@alsa-project.org, broonie@kernel.org,
	liam.r.girdwood@intel.com
Subject: Re: [PATCH] ALSA: hda: Fix cpu lockup when stopping the cmd dmas
Date: Wed, 10 May 2017 08:47:52 +0200	[thread overview]
Message-ID: <s5ha86l5ht3.wl-tiwai@suse.de> (raw)
In-Reply-To: <20170510061731.GA6263@localhost>

On Wed, 10 May 2017 08:17:31 +0200,
Vinod Koul wrote:
> 
> On Wed, May 10, 2017 at 11:51:58AM +0530, jeeja.kp@intel.com wrote:
> > From: Jeeja KP <jeeja.kp@intel.com>
> > 
> > Using jiffies in hdac_wait_for_cmd_dmas() to determine when to time out
> > when interrupts are off (snd_hdac_bus_stop_cmd_io()/spin_lock_irq())
> > causes hard lockup so unlock while waiting using jiffies.
> > 
> > ---<-snip->---
> > <0>[ 1211.603046] NMI watchdog: Watchdog detected hard LOCKUP on cpu 3
> > <4>[ 1211.603047] Modules linked in: snd_hda_intel i915 vgem
> > <4>[ 1211.603053] irq event stamp: 13366
> > <4>[ 1211.603053] hardirqs last  enabled at (13365):
> > ...
> > <4>[ 1211.603059] Call Trace:
> > <4>[ 1211.603059]  ? delay_tsc+0x3d/0xc0
> > <4>[ 1211.603059]  __delay+0xa/0x10
> > <4>[ 1211.603060]  __const_udelay+0x31/0x40
> > <4>[ 1211.603060]  snd_hdac_bus_stop_cmd_io+0x96/0xe0 [snd_hda_core]
> > <4>[ 1211.603060]  ? azx_dev_disconnect+0x20/0x20 [snd_hda_intel]
> > <4>[ 1211.603061]  snd_hdac_bus_stop_chip+0xb1/0x100 [snd_hda_core]
> > <4>[ 1211.603061]  azx_stop_chip+0x9/0x10 [snd_hda_codec]
> > <4>[ 1211.603061]  azx_suspend+0x72/0x220 [snd_hda_intel]
> > <4>[ 1211.603061]  pci_pm_suspend+0x71/0x140
> > <4>[ 1211.603062]  dpm_run_callback+0x6f/0x330
> > <4>[ 1211.603062]  ? pci_pm_freeze+0xe0/0xe0
> > <4>[ 1211.603062]  __device_suspend+0xf9/0x370
> > <4>[ 1211.603062]  ? dpm_watchdog_set+0x60/0x60
> > <4>[ 1211.603063]  async_suspend+0x1a/0x90
> > <4>[ 1211.603063]  async_run_entry_fn+0x34/0x160
> > <4>[ 1211.603063]  process_one_work+0x1f4/0x6d0
> > <4>[ 1211.603063]  ? process_one_work+0x16e/0x6d0
> > <4>[ 1211.603064]  worker_thread+0x49/0x4a0
> > <4>[ 1211.603064]  kthread+0x107/0x140
> > <4>[ 1211.603064]  ? process_one_work+0x6d0/0x6d0
> > <4>[ 1211.603065]  ? kthread_create_on_node+0x40/0x40
> > <4>[ 1211.603065]  ret_from_fork+0x2e/0x40
> > 
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100419
> > Reported-by: Marta Lofstedt <marta.lofstedt@intel.com>
> > Suggested-by: Takashi Iwai <tiwai@suse.de>
> > Signed-off-by: Jeeja KP <jeeja.kp@intel.com>
> 
> Fixes: 38b19ed7f81ec ("ALSA: hda: fix to wait for RIRB & CORB DMA to set")
> CC: stable <stable@vger.kernel.org> # 4.7
> Acked-by: Vinod Koul <vinod.koul@intel.com>

Applied, thanks.


Takashi

      reply	other threads:[~2017-05-10  6:47 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-10  6:21 [PATCH] ALSA: hda: Fix cpu lockup when stopping the cmd dmas jeeja.kp
2017-05-10  6:17 ` Vinod Koul
2017-05-10  6:47   ` Takashi Iwai [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5ha86l5ht3.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=jeeja.kp@intel.com \
    --cc=liam.r.girdwood@intel.com \
    --cc=patches.audio@intel.com \
    --cc=vinod.koul@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox