Hi,
I realized a part of this patchset brings regressions due to missing to
handle appl_ptr correctly for driver with SNDRV_PCM_INFO_SYNC_APPLPTR.
Below patches are invalid.
* pcm: hw: use heler function to query status/control data after
reading/writing PCM frames
* pcm: hw: use helper function to query status/control data after
PREPARE/RESET call
* pcm: hw: use helper function to query status/control data after
REWIND/FORWARD call
On these patches, a helper function, query_status() is used to execute
ioctl(2) with SNDRV_PCM_IOCTL_SYNC_PTR, however it is done in a case
that status data mapping is failed. However, for the drivers, ALSA PCM
core allows to map status data and the ioctl is not executed against
my expectation. Although these operations changes appl_ptr in kernel
space, appl_ptr in user space is not synchronized to the one in kernel
space.
This regression brings some operation failures. For example, as I
confirmed, pulseaudio gets unexpected appl_ptr when it operates REWIND.
In the program, this is detected underrun and PCM frames are sent doubly
as a recovery. Users hear sound with short repetition after the
operation.
Please abandon this patchset. Later, I'll send second version of this
patchset.
On Jun 30 2017 08:58, Takashi Sakamoto wrote:
Hi,
This patchset is an optimization for version 2.0.14 of ALSA PCM
protocol/interface.
In this version, for some devices/drivers, applications are disallowed
to map control data of runtime of PCM substream. On the other hand,
mapping status data is still available.
In current implementation of alsa-lib, execution of ioctl(2) with
SNDRV_PCM_IOCTL_SYNC_PTR is still done without enough care of the
change. For example, even if status data is successfully mapped,
the ioctl is executed to query the status data. This is inefficient.
This patchset is to arrange the execution.
Takashi Sakamoto (12):
pcm: hw: fix to initialize function local variable
pcm: hw: add a helper function just to query status/control data of
PCM substream
pcm: hw: use heler function to query status/control data after
reading/writing PCM frames
pcm: hw: use helper function to query status/control data after
HW_PARAMS call
pcm: hw: use helper function to query status/control data after
PREPARE/RESET call
pcm: hw: use helper function to query status/control data after
REWIND/FORWARD call
pcm: hw: use helper function to query status/control data for
calculation of available space on PCM buffer
pcm: hw: add a helper function to request hwsync without side-effects
pcm: hw: add a helper function to issue appl_ptr without sub-effects
pcm: hw: add a helper function to issue avail_min without sub-effects
pcm: hw: remove superfluous code to call of SNDRV_PCM_IOCTL_SYNC_PTR
in snd_pcm_hw_forward()
pcm: hw: minor refactoring for initialization of control data
src/pcm/pcm_hw.c | 109 ++++++++++++++++++++++++++++++++++++++-----------------
1 file changed, 75 insertions(+), 34 deletions(-)
Regards
Takashi Sakamoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
