[alsa-devel] hw_params ignores ioplug's buffer size requirements

Takashi Iwai tiwai at suse.de
Tue Aug 7 18:06:13 CEST 2012


At Tue, 7 Aug 2012 09:32:11 -0500,
Andrew Eikum wrote:
> 
> Hi folks,
> 
> I've got either a bug or a misunderstanding in how ALSA handles
> ioplugs that provide hw_params restrictions. My understanding is that
> the snd_pcm_ioplug_set_param_* functions are supposed to restrict
> which hw_params settings a client may set. This seems to work for at
> least access and format. But I noticed that it's broken for
> buffer_bytes and periods at least.
> 
> Attached is two source files demonstrating the problem. alsa-example.c
> is a dead-simple ioplug module. It sets some hw_params restrictions,
> and does basically nothing else. The important thing to notice is that
> it sets /only one/ allowable period size, period count, and buffer
> size, all of which are compatible with each other (256 byte period *
> 8 periods == 2048 byte buffer size).
> 
> There is also test client, alsa-test.c, which attempts to violate
> those hw_params restrictions, and succeeds! The eventual call to the
> ioplug's example_hw_params() callback gets an invalid buffer_size
> parameter (it happens to be 512).

Why is it wrong?  512 frames = 512 * 2 channel * 2 bytes-per-sample
 = 2048 bytes.

I guess you are confused about the units passed to
snd_pcm_hw_params_set_buffer_size_near().  It's in frames, not in
bytes.


Takashi

> 
> Here is the output when run on my Arch Linux with alsa-lib 1.0.25:
> 
> [aeikum at aeikum ioplug_buffer_bytes]$ ./alsa-test 
> ALSA-example: Opening
> bytes: 512
> ALSA-example: hw_params: 512
> TEST FAILURE: buffer_size != 8812: 512
> ALSA-example: stop
> ALSA-example: close
> [aeikum at aeikum ioplug_buffer_bytes]$ 
> 
> Am I misunderstanding the snd_pcm_ioplug_set_param_* methods, or is
> this a bug? I am seeing this behavior in the wild with a pulseaudio
> client that ignores my hw_params restrictions.
> 
> Thanks for any insight,
> Andrew
> [2 alsa-example.c <text/x-csrc; utf-8 (7bit)>]
> /* build with:
>  *   $ gcc -fPIC -DPIC -c -Wall -Wno-unused-variable -Wno-unused-function alsa-example.c
>  *   $ gcc -Wl,--no-undefined -shared -fPIC -lasound -o libasound_module_pcm_example.so alsa-example.o -lc
>  *   # cp libasound_module_pcm_example.so /usr/lib/alsa-lib/
>  * create an 'example' PCM in asoundrc of type 'example'
>  * test with attached alsa-test.c
>  */
> 
> #include <alsa/asoundlib.h>
> #include <alsa/pcm_external.h>
> 
> #include <stdio.h>
> #include <errno.h>
> #include <sys/types.h>
> #include <sys/stat.h>
> #include <fcntl.h>
> #include <stdint.h>
> #include <sys/time.h>
> 
> struct example_pcm {
>     snd_pcm_ioplug_t io;
> };
> 
> static int example_start(snd_pcm_ioplug_t *io)
> {
>     struct example_pcm *sp_pcm = io->private_data;
>     fprintf(stderr, "ALSA-example: start\n");
>     return 0;
> }
> 
> static int example_stop(snd_pcm_ioplug_t *io)
> {
>     struct example_pcm *sp_pcm = io->private_data;
>     fprintf(stderr, "ALSA-example: stop\n");
>     return 0;
> }
> 
> static snd_pcm_sframes_t example_pointer(snd_pcm_ioplug_t *io)
> {
>     struct example_pcm *sp_pcm = io->private_data;
>     fprintf(stderr, "ALSA-example: pointer\n");
>     return 0;
> }
> 
> static snd_pcm_sframes_t example_transfer(snd_pcm_ioplug_t *io,
>         const snd_pcm_channel_area_t *areas,
>         snd_pcm_uframes_t offset, snd_pcm_uframes_t size)
> {
>     struct example_pcm *sp_pcm = io->private_data;
>     fprintf(stderr, "ALSA-example: transfer\n");
>     return size;
> }
> 
> static int example_close(snd_pcm_ioplug_t *io)
> {
>     struct example_pcm *sp_pcm = io->private_data;
>     fprintf(stderr, "ALSA-example: close\n");
>     return 0;
> }
> 
> static int example_hw_params(snd_pcm_ioplug_t *io, snd_pcm_hw_params_t *params)
> {
>     struct example_pcm *sp_pcm = io->private_data;
> 
>     fprintf(stderr, "ALSA-example: hw_params\n");
> 
>     if(io->buffer_size != 8812)
>         fprintf(stderr, "TEST FAILURE: buffer_size != 8812: %lu\n", io->buffer_size);
>     else
>         fprintf(stderr, "TEST SUCCESS: buffer_size == 8812: %lu\n", io->buffer_size);
> 
>     return 0;
> }
> 
> static const snd_pcm_ioplug_callback_t example_playback_callback = {
>     example_start, /* start, required */
>     example_stop, /* stop, required */
>     example_pointer, /* pointer, required */
>     example_transfer, /* transfer, optional */
>     example_close, /* close, optional */
>     example_hw_params, /* hw_params, optional */
>     NULL, /* hw_free, optional */
>     NULL, /* sw_params, optional */
>     NULL, /* prepare, optional */
>     NULL, /* drain, optional */
>     NULL, /* pause, optional */
>     NULL, /* resume, optional */
>     NULL, /* poll_descriptors_count, optional */
>     NULL, /* poll_descriptors, optional */
>     NULL, /* poll_revents, optional */
>     NULL, /* dump, optional */
>     NULL/* delay, optional */
> };
> 
> static int example_set_hw_params(snd_pcm_ioplug_t *io)
> {
>     static const snd_pcm_access_t access_list[] = {
>         SND_PCM_ACCESS_RW_INTERLEAVED
>     };
> 
>     static const unsigned int format_list[] = {
>         SND_PCM_FORMAT_S16
>     };
> 
>     int err;
> 
>     if((err = snd_pcm_ioplug_set_param_list(io, SND_PCM_IOPLUG_HW_ACCESS,
>                     sizeof(access_list) / sizeof(*access_list), access_list)) < 0)
>         return err;
> 
>     if((err = snd_pcm_ioplug_set_param_list(io, SND_PCM_IOPLUG_HW_FORMAT,
>                     sizeof(format_list) / sizeof(*format_list), format_list)) < 0)
>         return err;
> 
>     if((err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_RATE,
>                     44100, 44100)) < 0)
>         return err;
> 
>     if((err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_CHANNELS,
>                     2, 2)) < 0)
>         return err;
> 
>     /* 8 * 256 = 2048 */
>     if((err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_PERIOD_BYTES,
>                     256,256)) < 0)
>         return err;
> 
>     if((err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_PERIODS,
>                     8, 8)) < 0)
>         return err;
> 
>     if((err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_BUFFER_BYTES,
>                     2048, 2048)) < 0)
>         return err;
> 
>     return 0;
> }
> 
> int _snd_pcm_example_open(snd_pcm_t **pcmp, const char *name, snd_config_t *root,
>         snd_config_t *conf, snd_pcm_stream_t stream, int mode)
> {
>     struct example_pcm *sp_pcm;
>     int err;
> 
>     fprintf(stderr, "ALSA-example: Opening\n");
> 
>     sp_pcm = malloc(sizeof(*sp_pcm));
>     if(!sp_pcm)
>         return -ENOMEM;
>     memset(sp_pcm, 0, sizeof(*sp_pcm));
> 
>     sp_pcm->io.version = SND_PCM_IOPLUG_VERSION;
>     sp_pcm->io.name = "example Protocol Plugin";
>     sp_pcm->io.callback = &example_playback_callback;
>     sp_pcm->io.private_data = sp_pcm;
> 
>     err = snd_pcm_ioplug_create(&sp_pcm->io, name, stream, mode);
>     if(err){
>         free(sp_pcm);
>         return err > 0 ? -err : err;
>     }
> 
>     err = example_set_hw_params(&sp_pcm->io);
>     if(err){
>         snd_pcm_ioplug_delete(&sp_pcm->io);
>         free(sp_pcm);
>         return err;
>     }
> 
>     *pcmp = sp_pcm->io.pcm;
> 
>     return 0;
> }
> 
> SND_PCM_PLUGIN_SYMBOL(example);
> [3 alsa-test.c <text/x-csrc; utf-8 (7bit)>]
> /* to build:
>  *   $ gcc -Wall -o alsa-test alsa-test.c -lasound
>  */
> 
> #include <alsa/asoundlib.h>
> 
> int main(int argc, char **argv)
> {
>     snd_pcm_t *pcm;
>     snd_pcm_hw_params_t *hw_params;
>     unsigned int rate;
>     snd_pcm_uframes_t bytes;
>     int err;
> 
>     if((err = snd_pcm_open(&pcm, "example", SND_PCM_STREAM_PLAYBACK, 0) < 0)){
>         fprintf(stderr, "snd_pcm_open: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     if((err = snd_pcm_hw_params_malloc(&hw_params)) < 0){
>         fprintf(stderr, "snd_pcm_hw_params_malloc: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     if((err = snd_pcm_hw_params_any(pcm, hw_params) < 0)){
>         fprintf(stderr, "snd_pcm_hw_params_any: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     if((err = snd_pcm_hw_params_set_access(pcm, hw_params,
>                     SND_PCM_ACCESS_RW_INTERLEAVED) < 0)){
>         fprintf(stderr, "snd_pcm_hw_params_set_access: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     if((err = snd_pcm_hw_params_set_format(pcm, hw_params,
>                     SND_PCM_FORMAT_S16_LE) < 0)){
>         fprintf(stderr, "snd_pcm_hw_params_set_format: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     rate = 44100;
>     if((err = snd_pcm_hw_params_set_rate_near(pcm, hw_params, &rate, NULL)) < 0){
>         fprintf(stderr, "snd_pcm_hw_params_set_rate_near: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     if((err = snd_pcm_hw_params_set_channels(pcm, hw_params, 2)) < 0){
>         fprintf(stderr, "snd_pcm_hw_params_set_channels: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     bytes = 512;
>     if((err = snd_pcm_hw_params_set_buffer_size_near(pcm, hw_params, &bytes)) < 0){
>         fprintf(stderr, "snd_pcm_hw_params_set_buffer_size_near: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
>     fprintf(stderr, "bytes: %lu\n", bytes);
> 
>     if((err = snd_pcm_hw_params(pcm, hw_params)) < 0){
>         fprintf(stderr, "snd_pcm_hw_params: %d (%s)\n", err, snd_strerror(err));
>         return 1;
>     }
> 
>     snd_pcm_hw_params_free(hw_params);
>     snd_pcm_close(pcm);
> 
>     return 0;
> }
> [4  <text/plain; us-ascii (7bit)>]
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


More information about the Alsa-devel mailing list