diff options
author | Takashi Iwai <tiwai@suse.de> | 2006-04-28 09:13:41 -0400 |
---|---|---|
committer | Jaroslav Kysela <perex@suse.cz> | 2006-06-22 15:33:10 -0400 |
commit | d689e34b524b69c111db0b7c844d71c8e1a53b15 (patch) | |
tree | a37916ba70fd8d37d14bf546a656722fb4ed79ab /sound/core/timer.c | |
parent | bf850204a71a97eb5a6afaf27263bb667f9cab0a (diff) |
[ALSA] Remove spinlocks around proc prints
Don't lock during showing proc read. snd_iprintf() might sleep.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/core/timer.c')
-rw-r--r-- | sound/core/timer.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sound/core/timer.c b/sound/core/timer.c index 9a1e51c7c230..e37eab74572d 100644 --- a/sound/core/timer.c +++ b/sound/core/timer.c | |||
@@ -1095,7 +1095,6 @@ static void snd_timer_proc_read(struct snd_info_entry *entry, | |||
1095 | if (timer->hw.flags & SNDRV_TIMER_HW_SLAVE) | 1095 | if (timer->hw.flags & SNDRV_TIMER_HW_SLAVE) |
1096 | snd_iprintf(buffer, " SLAVE"); | 1096 | snd_iprintf(buffer, " SLAVE"); |
1097 | snd_iprintf(buffer, "\n"); | 1097 | snd_iprintf(buffer, "\n"); |
1098 | spin_lock_irqsave(&timer->lock, flags); | ||
1099 | list_for_each(q, &timer->open_list_head) { | 1098 | list_for_each(q, &timer->open_list_head) { |
1100 | ti = list_entry(q, struct snd_timer_instance, open_list); | 1099 | ti = list_entry(q, struct snd_timer_instance, open_list); |
1101 | snd_iprintf(buffer, " Client %s : %s\n", | 1100 | snd_iprintf(buffer, " Client %s : %s\n", |
@@ -1104,7 +1103,6 @@ static void snd_timer_proc_read(struct snd_info_entry *entry, | |||
1104 | SNDRV_TIMER_IFLG_RUNNING) | 1103 | SNDRV_TIMER_IFLG_RUNNING) |
1105 | ? "running" : "stopped"); | 1104 | ? "running" : "stopped"); |
1106 | } | 1105 | } |
1107 | spin_unlock_irqrestore(&timer->lock, flags); | ||
1108 | } | 1106 | } |
1109 | mutex_unlock(®ister_mutex); | 1107 | mutex_unlock(®ister_mutex); |
1110 | } | 1108 | } |