alsa-lib: Fix incorrect appl pointer when mmap_commit() returns error.

The appl pointer needs to be updated only when snd_pcm_mmap_commit() is
successfully returned. Or it shouldn't be updated.
This is to fix the avail_update()'s result is incorrect when returns error.

Signed-off-by: Yuqing Zhu <carol.zhu@nxp.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>

1 files changed, 137 insertions, 0 deletions

diff --git a/meta/recipes-multimedia/alsa/alsa-lib/0001-pcm_plugin-fix-appl-pointer-not-correct-when-mmap_co.patch b/meta/recipes-multimedia/alsa/alsa-lib/0001-pcm_plugin-fix-appl-pointer-not-correct-when-mmap_co.patch
new file mode 100755
index 0000000000..9fd69f301d
--- /dev/null
+++ b/meta/recipes-multimedia/alsa/alsa-lib/0001-pcm_plugin-fix-appl-pointer-not-correct-when-mmap_co.patch
@@ -0,0 +1,137 @@
+From 7c424edd116e76eee6218a1e9a6ff6c4daaf2a4d Mon Sep 17 00:00:00 2001
+From: Shengjiu Wang <shengjiu.wang@freescale.com>
+Date: Wed, 6 Apr 2016 19:02:12 +0800
+Subject: [PATCH] pcm_plugin: fix appl pointer not correct when mmap_commit()
+ return error
+
+When snd_pcm_mmap_commit() return error, the appl pointer is also updated.
+which cause the avail_update()'s result wrong.
+This patch move the snd_pcm_mmap_appl_forward() to the place when
+snd_pcm_mmap_commit() is successfully returned.
+
+Upstream-Status: Submitted [https://patchwork.kernel.org/patch/8760881/]
+
+Signed-off-by: Shengjiu Wang <shengjiu.wang@freescale.com>
+Signed-off-by: Takashi Iwai <tiwai@suse.de>
+---
+ src/pcm/pcm_plugin.c | 48 ++++++++++++++++++++++++++++++++----------------
+ 1 file changed, 32 insertions(+), 16 deletions(-)
+
+diff --git a/src/pcm/pcm_plugin.c b/src/pcm/pcm_plugin.c
+index d007e8c..940491d 100644
+--- a/src/pcm/pcm_plugin.c
++++ b/src/pcm/pcm_plugin.c
+@@ -279,18 +279,22 @@ static snd_pcm_sframes_t snd_pcm_plugin_write_areas(snd_pcm_t *pcm,
+ 			return -EPIPE;
+ 		}
+ 		snd_atomic_write_begin(&plugin->watom);
+-		snd_pcm_mmap_appl_forward(pcm, frames);
+ 		result = snd_pcm_mmap_commit(slave, slave_offset, slave_frames);
+ 		if (result > 0 && (snd_pcm_uframes_t)result != slave_frames) {
+ 			snd_pcm_sframes_t res;
+ 			res = plugin->undo_write(pcm, slave_areas, slave_offset + result, slave_frames, slave_frames - result);
+-			if (res < 0)
++			if (res < 0) {
++				snd_atomic_write_end(&plugin->watom);
+ 				return xfer > 0 ? (snd_pcm_sframes_t)xfer : res;
++			}
+ 			frames -= res;
+ 		}
+-		snd_atomic_write_end(&plugin->watom);
+-		if (result <= 0)
++		if (result <= 0) {
++			snd_atomic_write_end(&plugin->watom);
+ 			return xfer > 0 ? (snd_pcm_sframes_t)xfer : result;
++		}
++		snd_pcm_mmap_appl_forward(pcm, frames);
++		snd_atomic_write_end(&plugin->watom);
+ 		offset += frames;
+ 		xfer += frames;
+ 		size -= frames;
+@@ -325,19 +329,23 @@ static snd_pcm_sframes_t snd_pcm_plugin_read_areas(snd_pcm_t *pcm,
+ 			return -EPIPE;
+ 		}
+ 		snd_atomic_write_begin(&plugin->watom);
+-		snd_pcm_mmap_appl_forward(pcm, frames);
+ 		result = snd_pcm_mmap_commit(slave, slave_offset, slave_frames);
+ 		if (result > 0 && (snd_pcm_uframes_t)result != slave_frames) {
+ 			snd_pcm_sframes_t res;
+ 			
+ 			res = plugin->undo_read(slave, areas, offset, frames, slave_frames - result);
+-			if (res < 0)
++			if (res < 0) {
++				snd_atomic_write_end(&plugin->watom);
+ 				return xfer > 0 ? (snd_pcm_sframes_t)xfer : res;
++			}
+ 			frames -= res;
+ 		}
+-		snd_atomic_write_end(&plugin->watom);
+-		if (result <= 0)
++		if (result <= 0) {
++			snd_atomic_write_end(&plugin->watom);
+ 			return xfer > 0 ? (snd_pcm_sframes_t)xfer : result;
++		}
++		snd_pcm_mmap_appl_forward(pcm, frames);
++		snd_atomic_write_end(&plugin->watom);
+ 		offset += frames;
+ 		xfer += frames;
+ 		size -= frames;
+@@ -423,19 +431,23 @@ snd_pcm_plugin_mmap_commit(snd_pcm_t *pcm,
+ 		frames = plugin->write(pcm, areas, appl_offset, frames,
+ 				       slave_areas, slave_offset, &slave_frames);
+ 		snd_atomic_write_begin(&plugin->watom);
+-		snd_pcm_mmap_appl_forward(pcm, frames);
+ 		result = snd_pcm_mmap_commit(slave, slave_offset, slave_frames);
+-		snd_atomic_write_end(&plugin->watom);
+ 		if (result > 0 && (snd_pcm_uframes_t)result != slave_frames) {
+ 			snd_pcm_sframes_t res;
+ 			
+ 			res = plugin->undo_write(pcm, slave_areas, slave_offset + result, slave_frames, slave_frames - result);
+-			if (res < 0)
++			if (res < 0) {
++				snd_atomic_write_end(&plugin->watom);
+ 				return xfer > 0 ? xfer : res;
++			}
+ 			frames -= res;
+ 		}
+-		if (result <= 0)
++		if (result <= 0) {
++			snd_atomic_write_end(&plugin->watom);
+ 			return xfer > 0 ? xfer : result;
++		}
++		snd_pcm_mmap_appl_forward(pcm, frames);
++		snd_atomic_write_end(&plugin->watom);
+ 		if (frames == cont)
+ 			appl_offset = 0;
+ 		else
+@@ -490,19 +502,23 @@ static snd_pcm_sframes_t snd_pcm_plugin_avail_update(snd_pcm_t *pcm)
+ 			frames = (plugin->read)(pcm, areas, hw_offset, frames,
+ 					      slave_areas, slave_offset, &slave_frames);
+ 			snd_atomic_write_begin(&plugin->watom);
+-			snd_pcm_mmap_hw_forward(pcm, frames);
+ 			result = snd_pcm_mmap_commit(slave, slave_offset, slave_frames);
+-			snd_atomic_write_end(&plugin->watom);
+ 			if (result > 0 && (snd_pcm_uframes_t)result != slave_frames) {
+ 				snd_pcm_sframes_t res;
+ 				
+ 				res = plugin->undo_read(slave, areas, hw_offset, frames, slave_frames - result);
+-				if (res < 0)
++				if (res < 0) {
++					snd_atomic_write_end(&plugin->watom);
+ 					return xfer > 0 ? (snd_pcm_sframes_t)xfer : res;
++				}
+ 				frames -= res;
+ 			}
+-			if (result <= 0)
++			if (result <= 0) {
++				snd_atomic_write_end(&plugin->watom);
+ 				return xfer > 0 ? (snd_pcm_sframes_t)xfer : result;
++			}
++			snd_pcm_mmap_hw_forward(pcm, frames);
++			snd_atomic_write_end(&plugin->watom);
+ 			if (frames == cont)
+ 				hw_offset = 0;
+ 			else
+-- 
+1.9.1
+