Commit 912e4c33 authored by Jeffery Miller's avatar Jeffery Miller Committed by Takashi Iwai

ALSA: pcm: Return negative delays from SNDRV_PCM_IOCTL_DELAY.

The commit c2c86a97 ("ALSA: pcm: Remove set_fs() in PCM core code")
changed SNDRV_PCM_IOCTL_DELAY to return an inconsistent error instead of a
negative delay.  Originally the call would succeed and return the negative
delay.  The Chromium OS Audio Server (CRAS) gets confused and hangs when
the error is returned instead of the negative delay.

Help CRAS avoid the issue by rolling back the behavior to return a
negative delay instead of an error.

Fixes: c2c86a97 ("ALSA: pcm: Remove set_fs() in PCM core code")
Signed-off-by: default avatarJeffery Miller <jmiller@neverware.com>
Cc: <stable@vger.kernel.org> # v4.13+
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 6d08b06e
...@@ -27,10 +27,11 @@ static int snd_pcm_ioctl_delay_compat(struct snd_pcm_substream *substream, ...@@ -27,10 +27,11 @@ static int snd_pcm_ioctl_delay_compat(struct snd_pcm_substream *substream,
s32 __user *src) s32 __user *src)
{ {
snd_pcm_sframes_t delay; snd_pcm_sframes_t delay;
int err;
delay = snd_pcm_delay(substream); err = snd_pcm_delay(substream, &delay);
if (delay < 0) if (err)
return delay; return err;
if (put_user(delay, src)) if (put_user(delay, src))
return -EFAULT; return -EFAULT;
return 0; return 0;
......
...@@ -2692,7 +2692,8 @@ static int snd_pcm_hwsync(struct snd_pcm_substream *substream) ...@@ -2692,7 +2692,8 @@ static int snd_pcm_hwsync(struct snd_pcm_substream *substream)
return err; return err;
} }
static snd_pcm_sframes_t snd_pcm_delay(struct snd_pcm_substream *substream) static int snd_pcm_delay(struct snd_pcm_substream *substream,
snd_pcm_sframes_t *delay)
{ {
struct snd_pcm_runtime *runtime = substream->runtime; struct snd_pcm_runtime *runtime = substream->runtime;
int err; int err;
...@@ -2708,7 +2709,9 @@ static snd_pcm_sframes_t snd_pcm_delay(struct snd_pcm_substream *substream) ...@@ -2708,7 +2709,9 @@ static snd_pcm_sframes_t snd_pcm_delay(struct snd_pcm_substream *substream)
n += runtime->delay; n += runtime->delay;
} }
snd_pcm_stream_unlock_irq(substream); snd_pcm_stream_unlock_irq(substream);
return err < 0 ? err : n; if (!err)
*delay = n;
return err;
} }
static int snd_pcm_sync_ptr(struct snd_pcm_substream *substream, static int snd_pcm_sync_ptr(struct snd_pcm_substream *substream,
...@@ -2916,11 +2919,13 @@ static int snd_pcm_common_ioctl(struct file *file, ...@@ -2916,11 +2919,13 @@ static int snd_pcm_common_ioctl(struct file *file,
return snd_pcm_hwsync(substream); return snd_pcm_hwsync(substream);
case SNDRV_PCM_IOCTL_DELAY: case SNDRV_PCM_IOCTL_DELAY:
{ {
snd_pcm_sframes_t delay = snd_pcm_delay(substream); snd_pcm_sframes_t delay;
snd_pcm_sframes_t __user *res = arg; snd_pcm_sframes_t __user *res = arg;
int err;
if (delay < 0) err = snd_pcm_delay(substream, &delay);
return delay; if (err)
return err;
if (put_user(delay, res)) if (put_user(delay, res))
return -EFAULT; return -EFAULT;
return 0; return 0;
...@@ -3008,13 +3013,7 @@ int snd_pcm_kernel_ioctl(struct snd_pcm_substream *substream, ...@@ -3008,13 +3013,7 @@ int snd_pcm_kernel_ioctl(struct snd_pcm_substream *substream,
case SNDRV_PCM_IOCTL_DROP: case SNDRV_PCM_IOCTL_DROP:
return snd_pcm_drop(substream); return snd_pcm_drop(substream);
case SNDRV_PCM_IOCTL_DELAY: case SNDRV_PCM_IOCTL_DELAY:
{ return snd_pcm_delay(substream, frames);
result = snd_pcm_delay(substream);
if (result < 0)
return result;
*frames = result;
return 0;
}
default: default:
return -EINVAL; return -EINVAL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment