diff options
author | Jonathan Corbet <corbet@lwn.net> | 2008-06-23 17:40:43 -0600 |
---|---|---|
committer | Jonathan Corbet <corbet@lwn.net> | 2008-07-02 15:06:28 -0600 |
commit | 2db9f0a35a4e4ad7cee756a4e0dabb7c1c8b6c72 (patch) | |
tree | 32dd2d06d650305c6bff129a92545c73d6bc9295 /sound | |
parent | 70b028b7ea94f1b36c61f3ee1c921cc3a87812e6 (diff) | |
download | op-kernel-dev-2db9f0a35a4e4ad7cee756a4e0dabb7c1c8b6c72.zip op-kernel-dev-2db9f0a35a4e4ad7cee756a4e0dabb7c1c8b6c72.tar.gz |
snd/PCM: fasync BKL pushdown
Chances are this is unneeded, but the code is twisty enough that it is hard
to tell.
Signed-off-by: Jonathan Corbet <corbet@lwn.net>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/core/pcm_native.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c index 61f5d42..c49b9d9 100644 --- a/sound/core/pcm_native.c +++ b/sound/core/pcm_native.c @@ -22,6 +22,7 @@ #include <linux/mm.h> #include <linux/file.h> #include <linux/slab.h> +#include <linux/smp_lock.h> #include <linux/time.h> #include <linux/pm_qos_params.h> #include <linux/uio.h> @@ -3249,14 +3250,17 @@ static int snd_pcm_fasync(int fd, struct file * file, int on) struct snd_pcm_file * pcm_file; struct snd_pcm_substream *substream; struct snd_pcm_runtime *runtime; - int err; + int err = -ENXIO; + lock_kernel(); pcm_file = file->private_data; substream = pcm_file->substream; - snd_assert(substream != NULL, return -ENXIO); + snd_assert(substream != NULL, goto out); runtime = substream->runtime; err = fasync_helper(fd, file, on, &runtime->fasync); +out: + unlock_kernel(); if (err < 0) return err; return 0; |