summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2006-07-13 20:08:01 +0200
committerGreg Kroah-Hartman <gregkh@suse.de>2006-07-24 20:35:26 -0700
commit62d99ac2aeb0a91d7f5f3a4f1532c686b736b37e (patch)
tree954ce7a24255c06ebe2750ec2d36a9da31d76407
parent954d53d3ea28bcd12396f7d26b78dd16719246c6 (diff)
downloadlinux-stable-62d99ac2aeb0a91d7f5f3a4f1532c686b736b37e.tar.gz
linux-stable-62d99ac2aeb0a91d7f5f3a4f1532c686b736b37e.tar.bz2
linux-stable-62d99ac2aeb0a91d7f5f3a4f1532c686b736b37e.zip
ALSA: Fix a deadlock in snd-rtctimer
Fix an occasional deadlock occuring with snd-rtctimer driver, added irqsave to the lock in tasklet (ALSA bug#952). Signed-off-by: Takashi Iwai <tiwai@suse.de> Signed-off-by: Jaroslav Kysela <perex@suse.cz> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--sound/core/timer.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/sound/core/timer.c b/sound/core/timer.c
index cdeeb639b675..4585600ba0be 100644
--- a/sound/core/timer.c
+++ b/sound/core/timer.c
@@ -628,8 +628,9 @@ static void snd_timer_tasklet(unsigned long arg)
struct snd_timer_instance *ti;
struct list_head *p;
unsigned long resolution, ticks;
+ unsigned long flags;
- spin_lock(&timer->lock);
+ spin_lock_irqsave(&timer->lock, flags);
/* now process all callbacks */
while (!list_empty(&timer->sack_list_head)) {
p = timer->sack_list_head.next; /* get first item */
@@ -649,7 +650,7 @@ static void snd_timer_tasklet(unsigned long arg)
spin_lock(&timer->lock);
ti->flags &= ~SNDRV_TIMER_IFLG_CALLBACK;
}
- spin_unlock(&timer->lock);
+ spin_unlock_irqrestore(&timer->lock, flags);
}
/*