summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorEero Nurkkala <ext-eero.nurkkala@nokia.com>2009-10-12 08:41:59 +0300
committerMark Brown <broonie@opensource.wolfsonmicro.com>2009-10-13 13:33:02 +0100
commit8e8b2d676f3f7c1246b108793fb5690e6c6fcd26 (patch)
tree12953d54da24f6a92a143303258987e51ac8878b /sound
parent6f775ba0154cdf10108071acc71f26c4efdb8d57 (diff)
downloadlinux-8e8b2d676f3f7c1246b108793fb5690e6c6fcd26.tar.gz
linux-8e8b2d676f3f7c1246b108793fb5690e6c6fcd26.tar.bz2
linux-8e8b2d676f3f7c1246b108793fb5690e6c6fcd26.zip
ASoC: Serialize access to dapm_power_widgets()
Access to damp_power_widgets() is assumed to be single-threaded. Concurrent accesses to dapm_power_widgets() may result in unpredictable behavior. Calls from: close_delayed_work() soc_codec_close() soc_pcm_prepare() soc_suspend() soc_resume_deferred() to snd_soc_dapm_stream_event() do not have the codec->mutex taken to cover the call to dapm_power_widgets(). Thus, take the mutex in these paths also to assure single-threaded use of dapm_power_widgets(). Signed-off-by: Eero Nurkkala <ext-eero.nurkkala@nokia.com> Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound')
-rw-r--r--sound/soc/soc-dapm.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/soc-dapm.c b/sound/soc/soc-dapm.c
index 8de6f9dec4a2..d89f6dc00908 100644
--- a/sound/soc/soc-dapm.c
+++ b/sound/soc/soc-dapm.c
@@ -2072,9 +2072,9 @@ int snd_soc_dapm_stream_event(struct snd_soc_codec *codec,
}
}
}
- mutex_unlock(&codec->mutex);
dapm_power_widgets(codec, event);
+ mutex_unlock(&codec->mutex);
dump_dapm(codec, __func__);
return 0;
}