OSDN Git Service

dshare and dsnoop plugins are working now
authorJaroslav Kysela <perex@perex.cz>
Wed, 19 Mar 2003 16:35:35 +0000 (16:35 +0000)
committerJaroslav Kysela <perex@perex.cz>
Wed, 19 Mar 2003 16:35:35 +0000 (16:35 +0000)
src/pcm/pcm_direct.c
src/pcm/pcm_direct.h
src/pcm/pcm_dmix.c
src/pcm/pcm_dshare.c
src/pcm/pcm_dsnoop.c

index 2d74d15..5a0f146 100644 (file)
@@ -521,6 +521,10 @@ int snd_pcm_direct_initialize_slave(snd_pcm_direct_t *dmix, snd_pcm_t *spcm, str
                SNDERR("unable to set stop threshold");
                return ret;
        }
+
+       if (dmix->type != SND_PCM_TYPE_DMIX)
+               goto __skip_silencing;
+
        ret = snd_pcm_sw_params_set_silence_threshold(spcm, sw_params, 0);
        if (ret < 0) {
                SNDERR("unable to set silence threshold");
@@ -532,11 +536,19 @@ int snd_pcm_direct_initialize_slave(snd_pcm_direct_t *dmix, snd_pcm_t *spcm, str
                return ret;
        }
 
+      __skip_silencing:
+
        ret = snd_pcm_sw_params(spcm, sw_params);
        if (ret < 0) {
                SNDERR("unable to install sw params (please upgrade to 0.9.0rc8+ driver)");
                return ret;
        }
+
+       if (dmix->type == SND_PCM_TYPE_DSHARE) {
+               const snd_pcm_channel_area_t *dst_areas;
+               dst_areas = snd_pcm_mmap_areas(spcm);
+               snd_pcm_areas_silence(dst_areas, 0, spcm->channels, spcm->buffer_size, spcm->format);
+       }
        
        ret = snd_pcm_start(spcm);
        if (ret < 0) {
@@ -575,6 +587,7 @@ int snd_pcm_direct_initialize_poll_fd(snd_pcm_direct_t *dmix)
        snd_timer_params_t *params;
        char name[128];
        struct pollfd fd;
+       int capture = dmix->type == SND_PCM_TYPE_DSNOOP ? 1 : 0;
        
        snd_pcm_info_alloca(&info);
        snd_timer_params_alloca(&params);
@@ -587,7 +600,7 @@ int snd_pcm_direct_initialize_poll_fd(snd_pcm_direct_t *dmix)
                                (int)SND_TIMER_CLASS_PCM, 
                                snd_pcm_info_get_card(info),
                                snd_pcm_info_get_device(info),
-                               snd_pcm_info_get_subdevice(info) * 2);  /* it's a bit trick to distict playback and capture */
+                               snd_pcm_info_get_subdevice(info) * 2 + capture);
        ret = snd_timer_open(&dmix->timer, name, SND_TIMER_OPEN_NONBLOCK);
        if (ret < 0) {
                SNDERR("unable to open timer '%s'", name);
@@ -619,7 +632,9 @@ int snd_pcm_direct_check_interleave(snd_pcm_direct_t *dmix, snd_pcm_t *pcm)
        const snd_pcm_channel_area_t *dst_areas;
        const snd_pcm_channel_area_t *src_areas;
 
-       channels = dmix->u.dmix.channels;
+       if ((snd_pcm_format_physical_width(dmix->type) % 8) != 0)
+               interleaved = 0;
+       channels = dmix->channels;
        dst_areas = snd_pcm_mmap_areas(dmix->spcm);
        src_areas = snd_pcm_mmap_areas(pcm);
        for (chn = 1; chn < channels; chn++) {
@@ -633,7 +648,7 @@ int snd_pcm_direct_check_interleave(snd_pcm_direct_t *dmix, snd_pcm_t *pcm)
                }
        }
        for (chn = 0; chn < channels; chn++) {
-               if (dmix->u.dmix.bindings && dmix->u.dmix.bindings[chn] != chn) {
+               if (dmix->bindings && dmix->bindings[chn] != chn) {
                        interleaved = 0;
                        break;
                }
@@ -650,3 +665,74 @@ int snd_pcm_direct_check_interleave(snd_pcm_direct_t *dmix, snd_pcm_t *pcm)
        }
        return dmix->interleaved = interleaved;
 }
+
+/*
+ * parse the channel map
+ * id == client channel
+ * value == slave's channel
+ */
+int snd_pcm_direct_parse_bindings(snd_pcm_direct_t *dmix, snd_config_t *cfg)
+{
+       snd_config_iterator_t i, next;
+       unsigned int chn, chn1, count = 0;
+       int err;
+
+       dmix->channels = UINT_MAX;
+       if (cfg == NULL)
+               return 0;
+       if (snd_config_get_type(cfg) != SND_CONFIG_TYPE_COMPOUND) {
+               SNDERR("invalid type for bindings");
+               return -EINVAL;
+       }
+       snd_config_for_each(i, next, cfg) {
+               snd_config_t *n = snd_config_iterator_entry(i);
+               const char *id;
+               long cchannel;
+               if (snd_config_get_id(n, &id) < 0)
+                       continue;
+               err = safe_strtol(id, &cchannel);
+               if (err < 0 || cchannel < 0) {
+                       SNDERR("invalid client channel in binding: %s\n", id);
+                       return -EINVAL;
+               }
+               if ((unsigned)cchannel >= count)
+                       count = cchannel + 1;
+       }
+       if (count == 0)
+               return 0;
+       if (count > 1024) {
+               SNDERR("client channel out of range");
+               return -EINVAL;
+       }
+       dmix->bindings = malloc(count * sizeof(unsigned int));
+       for (chn = 0; chn < count; chn++)
+               dmix->bindings[chn] = UINT_MAX;         /* don't route */
+       snd_config_for_each(i, next, cfg) {
+               snd_config_t *n = snd_config_iterator_entry(i);
+               const char *id;
+               long cchannel, schannel;
+               if (snd_config_get_id(n, &id) < 0)
+                       continue;
+               safe_strtol(id, &cchannel);
+               if (snd_config_get_integer(n, &schannel) < 0) {
+                       SNDERR("unable to get slave channel (should be integer type) in binding: %s\n", id);
+                       return -EINVAL;
+               }
+               dmix->bindings[cchannel] = schannel;
+       }
+       if (dmix->type == SND_PCM_TYPE_DSNOOP)
+               goto __skip_same_dst;
+       for (chn = 0; chn < count; chn++) {
+               for (chn1 = 0; chn1 < count; chn1++) {
+                       if (chn == chn1)
+                               continue;
+                       if (dmix->bindings[chn] == dmix->bindings[chn1]) {
+                               SNDERR("unable to route channels %d,%d to same destination %d", chn, chn1, dmix->bindings[chn]);
+                               return -EINVAL;
+                       }
+               }
+       }
+      __skip_same_dst:
+       dmix->channels = count;
+       return 0;
+}
index 4f339d0..fd9e1c1 100644 (file)
@@ -65,9 +65,15 @@ typedef struct {
                unsigned int rate;
                snd_pcm_format_t format;
        } s;
+       union {
+               struct {
+                       unsigned long long chn_mask;
+               } dshare;
+       } u;
 } snd_pcm_direct_share_t;
 
 typedef struct {
+       snd_pcm_type_t type;            /* type (dmix, dsnoop, dshare) */
        key_t ipc_key;                  /* IPC key for semaphore and memory */
        int semid;                      /* IPC global semaphore identification */
        int shmid;                      /* IPC global shared memory identification */
@@ -88,18 +94,19 @@ typedef struct {
        pid_t server_pid;
        snd_timer_t *timer;             /* timer used as poll_fd */
        int interleaved;                /* we have interleaved buffer */
+       unsigned int channels;          /* client's channels */
+       unsigned int *bindings;
        union {
                struct {
                        int shmid_sum;                  /* IPC global sum ring buffer memory identification */
                        signed int *sum_buffer;         /* shared sum buffer */
                        mix_areas1_t *mix_areas1;
                        mix_areas2_t *mix_areas2;
-                       unsigned int channels;  /* client's channels */
-                       unsigned int *bindings;
                } dmix;
                struct {
                } dsnoop;
                struct {
+                       unsigned long long chn_mask;
                } dshare;
        } u;
 } snd_pcm_direct_t;
@@ -117,6 +124,7 @@ int snd_pcm_direct_client_discard(snd_pcm_direct_t *dmix);
 int snd_pcm_direct_initialize_slave(snd_pcm_direct_t *dmix, snd_pcm_t *spcm, struct slave_params *params);
 int snd_pcm_direct_initialize_poll_fd(snd_pcm_direct_t *dmix);
 int snd_pcm_direct_check_interleave(snd_pcm_direct_t *dmix, snd_pcm_t *pcm);
+int snd_pcm_direct_parse_bindings(snd_pcm_direct_t *dmix, snd_config_t *cfg);
 
 int snd_timer_async(snd_timer_t *timer, int sig, pid_t pid);
 struct timespec snd_pcm_hw_fast_tstamp(snd_pcm_t *pcm);
index b38849b..db5084c 100644 (file)
@@ -236,7 +236,7 @@ static void mix_areas(snd_pcm_direct_t *dmix,
        unsigned int src_step, dst_step;
        unsigned int chn, dchn, channels;
        
-       channels = dmix->u.dmix.channels;
+       channels = dmix->channels;
        if (dmix->shmptr->s.format == SND_PCM_FORMAT_S16) {
                signed short *src;
                volatile signed short *dst;
@@ -255,7 +255,7 @@ static void mix_areas(snd_pcm_direct_t *dmix,
                        return;
                }
                for (chn = 0; chn < channels; chn++) {
-                       dchn = dmix->u.dmix.bindings ? dmix->u.dmix.bindings[chn] : chn;
+                       dchn = dmix->bindings ? dmix->bindings[chn] : chn;
                        if (dchn >= dmix->shmptr->s.channels)
                                continue;
                        src_step = src_areas[chn].step / 8;
@@ -283,7 +283,7 @@ static void mix_areas(snd_pcm_direct_t *dmix,
                        return;
                }
                for (chn = 0; chn < channels; chn++) {
-                       dchn = dmix->u.dmix.bindings ? dmix->u.dmix.bindings[chn] : chn;
+                       dchn = dmix->bindings ? dmix->bindings[chn] : chn;
                        if (dchn >= dmix->shmptr->s.channels)
                                continue;
                        src_step = src_areas[chn].step / 8;
@@ -487,7 +487,7 @@ static int snd_pcm_dmix_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
                        SNDERR("dmix channels mask empty?");
                        return -EINVAL;
                }
-               err = snd_interval_refine_set(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS), dmix->u.dmix.channels);
+               err = snd_interval_refine_set(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS), dmix->channels);
                if (err < 0)
                        return err;
        }
@@ -764,8 +764,8 @@ static int snd_pcm_dmix_close(snd_pcm_t *pcm)
        } else {
                snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
        }
-       if (dmix->u.dmix.bindings)
-               free(dmix->u.dmix.bindings);
+       if (dmix->bindings)
+               free(dmix->bindings);
        pcm->private_data = NULL;
        free(dmix);
        return 0;
@@ -854,74 +854,6 @@ static snd_pcm_fast_ops_t snd_pcm_dmix_fast_ops = {
        mmap_commit: snd_pcm_dmix_mmap_commit,
 };
 
-/*
- * parse the channel map
- * id == client channel
- * value == slave's channel
- */
-static int parse_bindings(snd_pcm_direct_t *dmix, snd_config_t *cfg)
-{
-       snd_config_iterator_t i, next;
-       unsigned int chn, chn1, count = 0;
-       int err;
-
-       dmix->u.dmix.channels = UINT_MAX;
-       if (cfg == NULL)
-               return 0;
-       if (snd_config_get_type(cfg) != SND_CONFIG_TYPE_COMPOUND) {
-               SNDERR("invalid type for bindings");
-               return -EINVAL;
-       }
-       snd_config_for_each(i, next, cfg) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               long cchannel;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               err = safe_strtol(id, &cchannel);
-               if (err < 0 || cchannel < 0) {
-                       SNDERR("invalid client channel in binding: %s\n", id);
-                       return -EINVAL;
-               }
-               if ((unsigned)cchannel > count)
-                       count = cchannel + 1;
-       }
-       if (count == 0)
-               return 0;
-       if (count > 1024) {
-               SNDERR("client channel out of range");
-               return -EINVAL;
-       }
-       dmix->u.dmix.bindings = malloc(count * sizeof(unsigned int));
-       for (chn = 0; chn < count; chn++)
-               dmix->u.dmix.bindings[chn] = UINT_MAX;          /* don't route */
-       snd_config_for_each(i, next, cfg) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               long cchannel, schannel;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               safe_strtol(id, &cchannel);
-               if (snd_config_get_integer(n, &schannel) < 0) {
-                       SNDERR("unable to get slave channel (should be integer type) in binding: %s\n", id);
-                       return -EINVAL;
-               }
-               dmix->u.dmix.bindings[cchannel] = schannel;
-       }
-       for (chn = 0; chn < count; chn++) {
-               for (chn1 = 0; chn1 < count; chn1++) {
-                       if (chn == chn1)
-                               continue;
-                       if (dmix->u.dmix.bindings[chn] == dmix->u.dmix.bindings[chn1]) {
-                               SNDERR("unable to route channels %d,%d to same destination %d", chn, chn1, dmix->u.dmix.bindings[chn]);
-                               return -EINVAL;
-                       }
-               }
-       }
-       dmix->u.dmix.channels = count;
-       return 0;
-}
-
 /**
  * \brief Creates a new dmix PCM
  * \param pcmp Returns created PCM handle
@@ -960,7 +892,7 @@ int snd_pcm_dmix_open(snd_pcm_t **pcmp, const char *name,
                goto _err;
        }
        
-       ret = parse_bindings(dmix, bindings);
+       ret = snd_pcm_direct_parse_bindings(dmix, bindings);
        if (ret < 0)
                goto _err;
        
@@ -968,7 +900,7 @@ int snd_pcm_dmix_open(snd_pcm_t **pcmp, const char *name,
        dmix->semid = -1;
        dmix->shmid = -1;
 
-       ret = snd_pcm_new(&pcm, SND_PCM_TYPE_DMIX, name, stream, mode);
+       ret = snd_pcm_new(&pcm, dmix->type = SND_PCM_TYPE_DMIX, name, stream, mode);
        if (ret < 0)
                goto _err;
 
@@ -1071,8 +1003,8 @@ int snd_pcm_dmix_open(snd_pcm_t **pcmp, const char *name,
        snd_pcm_set_hw_ptr(pcm, &dmix->hw_ptr, -1, 0);
        snd_pcm_set_appl_ptr(pcm, &dmix->appl_ptr, -1, 0);
        
-       if (dmix->u.dmix.channels == UINT_MAX)
-               dmix->u.dmix.channels = dmix->shmptr->s.channels;
+       if (dmix->channels == UINT_MAX)
+               dmix->channels = dmix->shmptr->s.channels;
        
        snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
 
@@ -1099,8 +1031,8 @@ int snd_pcm_dmix_open(snd_pcm_t **pcmp, const char *name,
                                }
                        }
                }
-               if (dmix->u.dmix.bindings)
-                       free(dmix->u.dmix.bindings);
+               if (dmix->bindings)
+                       free(dmix->bindings);
                free(dmix);
        }
        if (pcm)
@@ -1247,14 +1179,6 @@ int _snd_pcm_dmix_open(snd_pcm_t **pcmp, const char *name,
        if (err < 0)
                return err;
 
-       /* sorry, limited features */
-       if (params.format != SND_PCM_FORMAT_S16 &&
-           params.format != SND_PCM_FORMAT_S32) {
-               SNDERR("invalid format, specify s16 or s32");
-               snd_config_delete(sconf);
-               return -EINVAL;
-       }
-
        params.period_size = psize;
        params.buffer_size = bsize;
        err = snd_pcm_dmix_open(pcmp, name, ipc_key, &params, bindings, root, sconf, stream, mode);
index d94d98f..0dfa303 100644 (file)
 const char *_snd_module_pcm_dshare = "";
 #endif
 
-static void share_areas(snd_pcm_direct_t *dmix,
+static void do_silence(snd_pcm_t *pcm)
+{
+       snd_pcm_direct_t *dshare = pcm->private_data;
+       const snd_pcm_channel_area_t *dst_areas;
+       unsigned int chn, dchn, channels;
+       snd_pcm_format_t format;
+
+       dst_areas = snd_pcm_mmap_areas(dshare->spcm);
+       channels = dshare->channels;
+       format = dshare->shmptr->s.format;
+       for (chn = 0; chn < channels; chn++) {
+               dchn = dshare->bindings ? dshare->bindings[chn] : chn;
+               snd_pcm_area_silence(&dst_areas[dchn], 0, dshare->shmptr->s.buffer_size, format);
+       }
+}
+
+static void share_areas(snd_pcm_direct_t *dshare,
                      const snd_pcm_channel_area_t *src_areas,
                      const snd_pcm_channel_area_t *dst_areas,
                      snd_pcm_uframes_t src_ofs,
                      snd_pcm_uframes_t dst_ofs,
                      snd_pcm_uframes_t size)
 {
-       if (dmix->interleaved) {
+       unsigned int chn, dchn, channels;
+       snd_pcm_format_t format;
+
+       channels = dshare->channels;
+       format = dshare->shmptr->s.format;
+       if (dshare->interleaved) {
+               unsigned int fbytes = snd_pcm_format_physical_width(format) / 8;
+               memcpy(((char *)dst_areas[0].addr) + (dst_ofs * channels * fbytes),
+                      ((char *)src_areas[0].addr) + (src_ofs * channels * fbytes),
+                      size * channels * fbytes);
        } else {
+               for (chn = 0; chn < channels; chn++) {
+                       dchn = dshare->bindings ? dshare->bindings[chn] : chn;
+                       snd_pcm_area_copy(&dst_areas[dchn], dst_ofs, &src_areas[chn], src_ofs, size, format);
+
+               }
        }
 }
 
@@ -65,28 +95,28 @@ static void share_areas(snd_pcm_direct_t *dmix,
  */
 static void snd_pcm_dshare_sync_area(snd_pcm_t *pcm, snd_pcm_uframes_t size)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        snd_pcm_uframes_t appl_ptr, slave_appl_ptr, transfer;
        const snd_pcm_channel_area_t *src_areas, *dst_areas;
        
        /* get the start of update area */
-       appl_ptr = dmix->appl_ptr - size;
+       appl_ptr = dshare->appl_ptr - size;
        if (appl_ptr > pcm->boundary)
                appl_ptr += pcm->boundary;
        appl_ptr %= pcm->buffer_size;
        /* add sample areas here */
        src_areas = snd_pcm_mmap_areas(pcm);
-       dst_areas = snd_pcm_mmap_areas(dmix->spcm);
-       slave_appl_ptr = dmix->slave_appl_ptr % dmix->shmptr->s.buffer_size;
-       dmix->slave_appl_ptr += size;
-       dmix->slave_appl_ptr %= dmix->shmptr->s.boundary;
+       dst_areas = snd_pcm_mmap_areas(dshare->spcm);
+       slave_appl_ptr = dshare->slave_appl_ptr % dshare->shmptr->s.buffer_size;
+       dshare->slave_appl_ptr += size;
+       dshare->slave_appl_ptr %= dshare->shmptr->s.boundary;
        while (size > 0) {
                transfer = appl_ptr + size > pcm->buffer_size ? pcm->buffer_size - appl_ptr : size;
-               transfer = slave_appl_ptr + transfer > dmix->shmptr->s.buffer_size ? dmix->shmptr->s.buffer_size - slave_appl_ptr : transfer;
+               transfer = slave_appl_ptr + transfer > dshare->shmptr->s.buffer_size ? dshare->shmptr->s.buffer_size - slave_appl_ptr : transfer;
                size -= transfer;
-               share_areas(dmix, src_areas, dst_areas, appl_ptr, slave_appl_ptr, transfer);
+               share_areas(dshare, src_areas, dst_areas, appl_ptr, slave_appl_ptr, transfer);
                slave_appl_ptr += transfer;
-               slave_appl_ptr %= dmix->shmptr->s.buffer_size;
+               slave_appl_ptr %= dshare->shmptr->s.buffer_size;
                appl_ptr += transfer;
                appl_ptr %= pcm->buffer_size;
        }
@@ -97,35 +127,35 @@ static void snd_pcm_dshare_sync_area(snd_pcm_t *pcm, snd_pcm_uframes_t size)
  */
 static int snd_pcm_dshare_sync_ptr(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        snd_pcm_uframes_t slave_hw_ptr, old_slave_hw_ptr, avail;
        snd_pcm_sframes_t diff;
        
-       old_slave_hw_ptr = dmix->slave_hw_ptr;
-       slave_hw_ptr = dmix->slave_hw_ptr = *dmix->spcm->hw.ptr;
+       old_slave_hw_ptr = dshare->slave_hw_ptr;
+       slave_hw_ptr = dshare->slave_hw_ptr = *dshare->spcm->hw.ptr;
        diff = slave_hw_ptr - old_slave_hw_ptr;
        if (diff == 0)          /* fast path */
                return 0;
        if (diff < 0) {
-               slave_hw_ptr += dmix->shmptr->s.boundary;
+               slave_hw_ptr += dshare->shmptr->s.boundary;
                diff = slave_hw_ptr - old_slave_hw_ptr;
        }
-       dmix->hw_ptr += diff;
-       dmix->hw_ptr %= pcm->boundary;
+       dshare->hw_ptr += diff;
+       dshare->hw_ptr %= pcm->boundary;
        // printf("sync ptr diff = %li\n", diff);
        if (pcm->stop_threshold >= pcm->boundary)       /* don't care */
                return 0;
        if ((avail = snd_pcm_mmap_playback_avail(pcm)) >= pcm->stop_threshold) {
                struct timeval tv;
                gettimeofday(&tv, 0);
-               dmix->trigger_tstamp.tv_sec = tv.tv_sec;
-               dmix->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
-               dmix->state = SND_PCM_STATE_XRUN;
-               dmix->avail_max = avail;
+               dshare->trigger_tstamp.tv_sec = tv.tv_sec;
+               dshare->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
+               dshare->state = SND_PCM_STATE_XRUN;
+               dshare->avail_max = avail;
                return -EPIPE;
        }
-       if (avail > dmix->avail_max)
-               dmix->avail_max = avail;
+       if (avail > dshare->avail_max)
+               dshare->avail_max = avail;
        return 0;
 }
 
@@ -141,13 +171,13 @@ static int snd_pcm_dshare_nonblock(snd_pcm_t *pcm ATTRIBUTE_UNUSED, int nonblock
 
 static int snd_pcm_dshare_async(snd_pcm_t *pcm, int sig, pid_t pid)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       return snd_timer_async(dmix->timer, sig, pid);
+       snd_pcm_direct_t *dshare = pcm->private_data;
+       return snd_timer_async(dshare->timer, sig, pid);
 }
 
 static int snd_pcm_dshare_poll_revents(snd_pcm_t *pcm, struct pollfd *pfds, unsigned int nfds, unsigned short *revents)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        unsigned short events;
        static snd_timer_read_t rbuf[5];        /* can be overwriten by multiple plugins, we don't need the value */
 
@@ -157,7 +187,7 @@ static int snd_pcm_dshare_poll_revents(snd_pcm_t *pcm, struct pollfd *pfds, unsi
                events |= POLLOUT;
                events &= ~POLLIN;
                /* empty the timer read queue */
-               while (snd_timer_read(dmix->timer, &rbuf, sizeof(rbuf)) == sizeof(rbuf)) ;
+               while (snd_timer_read(dshare->timer, &rbuf, sizeof(rbuf)) == sizeof(rbuf)) ;
        }
        *revents = events;
        return 0;
@@ -165,7 +195,7 @@ static int snd_pcm_dshare_poll_revents(snd_pcm_t *pcm, struct pollfd *pfds, unsi
 
 static int snd_pcm_dshare_info(snd_pcm_t *pcm, snd_pcm_info_t * info)
 {
-       // snd_pcm_direct_t *dmix = pcm->private_data;
+       // snd_pcm_direct_t *dshare = pcm->private_data;
 
        memset(info, 0, sizeof(*info));
        info->stream = pcm->stream;
@@ -200,7 +230,7 @@ static int hw_param_interval_refine_one(snd_pcm_hw_params_t *params,
                return 0;
        i = hw_param_interval(params, var);
        if (snd_interval_empty(i)) {
-               SNDERR("dmix interval %i empty?", (int)var);
+               SNDERR("dshare interval %i empty?", (int)var);
                return -EINVAL;
        }
        if (snd_interval_refine(i, hw_param_interval(src, var)))
@@ -212,8 +242,8 @@ static int hw_param_interval_refine_one(snd_pcm_hw_params_t *params,
 
 static int snd_pcm_dshare_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       snd_pcm_hw_params_t *hw_params = &dmix->shmptr->hw_params;
+       snd_pcm_direct_t *dshare = pcm->private_data;
+       snd_pcm_hw_params_t *hw_params = &dshare->shmptr->hw_params;
        static snd_mask_t access = { .bits = { 
                                        (1<<SNDRV_PCM_ACCESS_MMAP_INTERLEAVED) |
                                        (1<<SNDRV_PCM_ACCESS_MMAP_NONINTERLEAVED) |
@@ -228,10 +258,9 @@ static int snd_pcm_dshare_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
        snd_output_puts(log, "DMIX REFINE (begin):\n");
        snd_pcm_hw_params_dump(params, log);
 #endif
-\
        if (params->rmask & (1<<SND_PCM_HW_PARAM_ACCESS)) {
                if (snd_mask_empty(hw_param_mask(params, SND_PCM_HW_PARAM_ACCESS))) {
-                       SNDERR("dmix access mask empty?");
+                       SNDERR("dshare access mask empty?");
                        return -EINVAL;
                }
                if (snd_mask_refine(hw_param_mask(params, SND_PCM_HW_PARAM_ACCESS), &access))
@@ -239,7 +268,7 @@ static int snd_pcm_dshare_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
        }
        if (params->rmask & (1<<SND_PCM_HW_PARAM_FORMAT)) {
                if (snd_mask_empty(hw_param_mask(params, SND_PCM_HW_PARAM_FORMAT))) {
-                       SNDERR("dmix format mask empty?");
+                       SNDERR("dshare format mask empty?");
                        return -EINVAL;
                }
                if (snd_mask_refine_set(hw_param_mask(params, SND_PCM_HW_PARAM_FORMAT),
@@ -249,10 +278,10 @@ static int snd_pcm_dshare_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
        //snd_mask_none(hw_param_mask(params, SND_PCM_HW_PARAM_SUBFORMAT));
        if (params->rmask & (1<<SND_PCM_HW_PARAM_CHANNELS)) {
                if (snd_interval_empty(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS))) {
-                       SNDERR("dmix channels mask empty?");
+                       SNDERR("dshare channels mask empty?");
                        return -EINVAL;
                }
-               err = snd_interval_refine_set(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS), dmix->u.dmix.channels);
+               err = snd_interval_refine_set(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS), dshare->channels);
                if (err < 0)
                        return err;
        }
@@ -308,30 +337,30 @@ static int snd_pcm_dshare_channel_info(snd_pcm_t *pcm, snd_pcm_channel_info_t *
 
 static int snd_pcm_dshare_status(snd_pcm_t *pcm, snd_pcm_status_t * status)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
 
        memset(status, 0, sizeof(*status));
-       status->state = dmix->state;
-       status->trigger_tstamp = dmix->trigger_tstamp;
-       status->tstamp = snd_pcm_hw_fast_tstamp(dmix->spcm);
+       status->state = dshare->state;
+       status->trigger_tstamp = dshare->trigger_tstamp;
+       status->tstamp = snd_pcm_hw_fast_tstamp(dshare->spcm);
        status->avail = snd_pcm_mmap_playback_avail(pcm);
-       status->avail_max = status->avail > dmix->avail_max ? status->avail : dmix->avail_max;
-       dmix->avail_max = 0;
+       status->avail_max = status->avail > dshare->avail_max ? status->avail : dshare->avail_max;
+       dshare->avail_max = 0;
        return 0;
 }
 
 static snd_pcm_state_t snd_pcm_dshare_state(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       return dmix->state;
+       snd_pcm_direct_t *dshare = pcm->private_data;
+       return dshare->state;
 }
 
 static int snd_pcm_dshare_delay(snd_pcm_t *pcm, snd_pcm_sframes_t *delayp)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        int err;
        
-       switch(dmix->state) {
+       switch(dshare->state) {
        case SNDRV_PCM_STATE_DRAINING:
        case SNDRV_PCM_STATE_RUNNING:
                err = snd_pcm_dshare_sync_ptr(pcm);
@@ -350,9 +379,9 @@ static int snd_pcm_dshare_delay(snd_pcm_t *pcm, snd_pcm_sframes_t *delayp)
 
 static int snd_pcm_dshare_hwsync(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
 
-       switch(dmix->state) {
+       switch(dshare->state) {
        case SNDRV_PCM_STATE_DRAINING:
        case SNDRV_PCM_STATE_RUNNING:
                return snd_pcm_dshare_sync_ptr(pcm);
@@ -368,39 +397,39 @@ static int snd_pcm_dshare_hwsync(snd_pcm_t *pcm)
 
 static int snd_pcm_dshare_prepare(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
 
-       snd_pcm_direct_check_interleave(dmix, pcm);
-       // assert(pcm->boundary == dmix->shmptr->s.boundary);   /* for sure */
-       dmix->state = SND_PCM_STATE_PREPARED;
-       dmix->appl_ptr = 0;
-       dmix->hw_ptr = 0;
+       snd_pcm_direct_check_interleave(dshare, pcm);
+       // assert(pcm->boundary == dshare->shmptr->s.boundary); /* for sure */
+       dshare->state = SND_PCM_STATE_PREPARED;
+       dshare->appl_ptr = 0;
+       dshare->hw_ptr = 0;
        return 0;
 }
 
 static int snd_pcm_dshare_reset(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       dmix->hw_ptr %= pcm->period_size;
-       dmix->appl_ptr = dmix->hw_ptr;
-       dmix->slave_appl_ptr = dmix->slave_hw_ptr = *dmix->spcm->hw.ptr;
+       snd_pcm_direct_t *dshare = pcm->private_data;
+       dshare->hw_ptr %= pcm->period_size;
+       dshare->appl_ptr = dshare->hw_ptr;
+       dshare->slave_appl_ptr = dshare->slave_hw_ptr = *dshare->spcm->hw.ptr;
        return 0;
 }
 
 static int snd_pcm_dshare_start(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        snd_pcm_sframes_t avail;
        struct timeval tv;
        int err;
        
-       if (dmix->state != SND_PCM_STATE_PREPARED)
+       if (dshare->state != SND_PCM_STATE_PREPARED)
                return -EBADFD;
-       err = snd_timer_start(dmix->timer);
+       err = snd_timer_start(dshare->timer);
        if (err < 0)
                return err;
-       dmix->state = SND_PCM_STATE_RUNNING;
-       dmix->slave_appl_ptr = dmix->slave_hw_ptr = *dmix->spcm->hw.ptr;
+       dshare->state = SND_PCM_STATE_RUNNING;
+       dshare->slave_appl_ptr = dshare->slave_hw_ptr = *dshare->spcm->hw.ptr;
        avail = snd_pcm_mmap_playback_hw_avail(pcm);
        if (avail < 0)
                return 0;
@@ -408,33 +437,34 @@ static int snd_pcm_dshare_start(snd_pcm_t *pcm)
                avail = pcm->buffer_size;
        snd_pcm_dshare_sync_area(pcm, avail);
        gettimeofday(&tv, 0);
-       dmix->trigger_tstamp.tv_sec = tv.tv_sec;
-       dmix->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
+       dshare->trigger_tstamp.tv_sec = tv.tv_sec;
+       dshare->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
        return 0;
 }
 
 static int snd_pcm_dshare_drop(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       if (dmix->state == SND_PCM_STATE_OPEN)
+       snd_pcm_direct_t *dshare = pcm->private_data;
+       if (dshare->state == SND_PCM_STATE_OPEN)
                return -EBADFD;
-       snd_timer_stop(dmix->timer);
-       dmix->state = SND_PCM_STATE_SETUP;
+       snd_timer_stop(dshare->timer);
+       do_silence(pcm);
+       dshare->state = SND_PCM_STATE_SETUP;
        return 0;
 }
 
 static int snd_pcm_dshare_drain(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        snd_pcm_uframes_t stop_threshold;
        int err;
 
-       if (dmix->state == SND_PCM_STATE_OPEN)
+       if (dshare->state == SND_PCM_STATE_OPEN)
                return -EBADFD;
        stop_threshold = pcm->stop_threshold;
        if (pcm->stop_threshold > pcm->buffer_size)
                pcm->stop_threshold = pcm->buffer_size;
-       while (dmix->state == SND_PCM_STATE_RUNNING) {
+       while (dshare->state == SND_PCM_STATE_RUNNING) {
                err = snd_pcm_dshare_sync_ptr(pcm);
                if (err < 0)
                        break;
@@ -448,17 +478,18 @@ static int snd_pcm_dshare_drain(snd_pcm_t *pcm)
 
 static int snd_pcm_dshare_pause(snd_pcm_t *pcm, int enable)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
         if (enable) {
-               if (dmix->state != SND_PCM_STATE_RUNNING)
+               if (dshare->state != SND_PCM_STATE_RUNNING)
                        return -EBADFD;
-               dmix->state = SND_PCM_STATE_PAUSED;
-               snd_timer_stop(dmix->timer);
+               dshare->state = SND_PCM_STATE_PAUSED;
+               snd_timer_stop(dshare->timer);
+               do_silence(pcm);
        } else {
-               if (dmix->state != SND_PCM_STATE_PAUSED)
+               if (dshare->state != SND_PCM_STATE_PAUSED)
                        return -EBADFD;
-                dmix->state = SND_PCM_STATE_RUNNING;
-                snd_timer_start(dmix->timer);
+                dshare->state = SND_PCM_STATE_RUNNING;
+                snd_timer_start(dshare->timer);
        }
        return 0;
 }
@@ -485,7 +516,7 @@ static snd_pcm_sframes_t snd_pcm_dshare_forward(snd_pcm_t *pcm, snd_pcm_uframes_
 
 static int snd_pcm_dshare_resume(snd_pcm_t *pcm ATTRIBUTE_UNUSED)
 {
-       // snd_pcm_direct_t *dmix = pcm->private_data;
+       // snd_pcm_direct_t *dshare = pcm->private_data;
        // FIXME
        return 0;
 }
@@ -512,26 +543,28 @@ static int snd_pcm_dshare_munmap(snd_pcm_t *pcm ATTRIBUTE_UNUSED)
 
 static int snd_pcm_dshare_close(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-
-       if (dmix->timer)
-               snd_timer_close(dmix->timer);
-       snd_pcm_direct_semaphore_down(dmix, DIRECT_IPC_SEM_CLIENT);
-       snd_pcm_close(dmix->spcm);
-       if (dmix->server)
-               snd_pcm_direct_server_discard(dmix);
-       if (dmix->client)
-               snd_pcm_direct_client_discard(dmix);
-       if (snd_pcm_direct_shm_discard(dmix) > 0) {
-               if (snd_pcm_direct_semaphore_discard(dmix) < 0)
-                       snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+       snd_pcm_direct_t *dshare = pcm->private_data;
+
+       if (dshare->timer)
+               snd_timer_close(dshare->timer);
+       do_silence(pcm);
+       snd_pcm_direct_semaphore_down(dshare, DIRECT_IPC_SEM_CLIENT);
+       dshare->shmptr->u.dshare.chn_mask &= ~dshare->u.dshare.chn_mask;
+       snd_pcm_close(dshare->spcm);
+       if (dshare->server)
+               snd_pcm_direct_server_discard(dshare);
+       if (dshare->client)
+               snd_pcm_direct_client_discard(dshare);
+       if (snd_pcm_direct_shm_discard(dshare) > 0) {
+               if (snd_pcm_direct_semaphore_discard(dshare) < 0)
+                       snd_pcm_direct_semaphore_up(dshare, DIRECT_IPC_SEM_CLIENT);
        } else {
-               snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+               snd_pcm_direct_semaphore_up(dshare, DIRECT_IPC_SEM_CLIENT);
        }
-       if (dmix->u.dmix.bindings)
-               free(dmix->u.dmix.bindings);
+       if (dshare->bindings)
+               free(dshare->bindings);
        pcm->private_data = NULL;
-       free(dmix);
+       free(dshare);
        return 0;
 }
 
@@ -539,11 +572,11 @@ static snd_pcm_sframes_t snd_pcm_dshare_mmap_commit(snd_pcm_t *pcm,
                                                  snd_pcm_uframes_t offset ATTRIBUTE_UNUSED,
                                                  snd_pcm_uframes_t size)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        int err;
 
        snd_pcm_mmap_appl_forward(pcm, size);
-       if (dmix->state == SND_PCM_STATE_RUNNING) {
+       if (dshare->state == SND_PCM_STATE_RUNNING) {
                err = snd_pcm_dshare_sync_ptr(pcm);
                if (err < 0)
                        return err;
@@ -556,10 +589,10 @@ static snd_pcm_sframes_t snd_pcm_dshare_mmap_commit(snd_pcm_t *pcm,
 
 static snd_pcm_sframes_t snd_pcm_dshare_avail_update(snd_pcm_t *pcm ATTRIBUTE_UNUSED)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
        int err;
        
-       if (dmix->state == SND_PCM_STATE_RUNNING) {
+       if (dshare->state == SND_PCM_STATE_RUNNING) {
                err = snd_pcm_dshare_sync_ptr(pcm);
                if (err < 0)
                        return err;
@@ -569,15 +602,15 @@ static snd_pcm_sframes_t snd_pcm_dshare_avail_update(snd_pcm_t *pcm ATTRIBUTE_UN
 
 static void snd_pcm_dshare_dump(snd_pcm_t *pcm, snd_output_t *out)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dshare = pcm->private_data;
 
        snd_output_printf(out, "Direct Stream Mixing PCM\n");
        if (pcm->setup) {
                snd_output_printf(out, "\nIts setup is:\n");
                snd_pcm_dump_setup(pcm, out);
        }
-       if (dmix->spcm)
-               snd_pcm_dump(dmix->spcm, out);
+       if (dshare->spcm)
+               snd_pcm_dump(dshare->spcm, out);
 }
 
 static snd_pcm_ops_t snd_pcm_dshare_ops = {
@@ -618,76 +651,8 @@ static snd_pcm_fast_ops_t snd_pcm_dshare_fast_ops = {
        mmap_commit: snd_pcm_dshare_mmap_commit,
 };
 
-/*
- * parse the channel map
- * id == client channel
- * value == slave's channel
- */
-static int parse_bindings(snd_pcm_direct_t *dmix, snd_config_t *cfg)
-{
-       snd_config_iterator_t i, next;
-       unsigned int chn, chn1, count = 0;
-       int err;
-
-       dmix->u.dmix.channels = UINT_MAX;
-       if (cfg == NULL)
-               return 0;
-       if (snd_config_get_type(cfg) != SND_CONFIG_TYPE_COMPOUND) {
-               SNDERR("invalid type for bindings");
-               return -EINVAL;
-       }
-       snd_config_for_each(i, next, cfg) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               long cchannel;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               err = safe_strtol(id, &cchannel);
-               if (err < 0 || cchannel < 0) {
-                       SNDERR("invalid client channel in binding: %s\n", id);
-                       return -EINVAL;
-               }
-               if ((unsigned)cchannel > count)
-                       count = cchannel + 1;
-       }
-       if (count == 0)
-               return 0;
-       if (count > 1024) {
-               SNDERR("client channel out of range");
-               return -EINVAL;
-       }
-       dmix->u.dmix.bindings = malloc(count * sizeof(unsigned int));
-       for (chn = 0; chn < count; chn++)
-               dmix->u.dmix.bindings[chn] = UINT_MAX;          /* don't route */
-       snd_config_for_each(i, next, cfg) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               long cchannel, schannel;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               safe_strtol(id, &cchannel);
-               if (snd_config_get_integer(n, &schannel) < 0) {
-                       SNDERR("unable to get slave channel (should be integer type) in binding: %s\n", id);
-                       return -EINVAL;
-               }
-               dmix->u.dmix.bindings[cchannel] = schannel;
-       }
-       for (chn = 0; chn < count; chn++) {
-               for (chn1 = 0; chn1 < count; chn1++) {
-                       if (chn == chn1)
-                               continue;
-                       if (dmix->u.dmix.bindings[chn] == dmix->u.dmix.bindings[chn1]) {
-                               SNDERR("unable to route channels %d,%d to same destination %d", chn, chn1, dmix->u.dmix.bindings[chn]);
-                               return -EINVAL;
-                       }
-               }
-       }
-       dmix->u.dmix.channels = count;
-       return 0;
-}
-
 /**
- * \brief Creates a new dmix PCM
+ * \brief Creates a new dshare PCM
  * \param pcmp Returns created PCM handle
  * \param name Name of PCM
  * \param ipc_key IPC key for semaphore and shared memory
@@ -708,47 +673,54 @@ int snd_pcm_dshare_open(snd_pcm_t **pcmp, const char *name,
                      snd_pcm_stream_t stream, int mode)
 {
        snd_pcm_t *pcm = NULL, *spcm = NULL;
-       snd_pcm_direct_t *dmix = NULL;
+       snd_pcm_direct_t *dshare = NULL;
        int ret, first_instance;
+       unsigned int chn;
 
        assert(pcmp);
 
        if (stream != SND_PCM_STREAM_PLAYBACK) {
-               SNDERR("The dmix plugin supports only playback stream");
+               SNDERR("The dshare plugin supports only playback stream");
                return -EINVAL;
        }
 
-       dmix = calloc(1, sizeof(snd_pcm_direct_t));
-       if (!dmix) {
+       dshare = calloc(1, sizeof(snd_pcm_direct_t));
+       if (!dshare) {
                ret = -ENOMEM;
                goto _err;
        }
        
-       ret = parse_bindings(dmix, bindings);
+       ret = snd_pcm_direct_parse_bindings(dshare, bindings);
        if (ret < 0)
                goto _err;
+               
+       if (!dshare->bindings) {
+               SNDERR("dshare: specify bindings!!!");
+               ret = -EINVAL;
+               goto _err;
+       }
        
-       dmix->ipc_key = ipc_key;
-       dmix->semid = -1;
-       dmix->shmid = -1;
+       dshare->ipc_key = ipc_key;
+       dshare->semid = -1;
+       dshare->shmid = -1;
 
-       ret = snd_pcm_new(&pcm, SND_PCM_TYPE_DSHARE, name, stream, mode);
+       ret = snd_pcm_new(&pcm, dshare->type = SND_PCM_TYPE_DSHARE, name, stream, mode);
        if (ret < 0)
                goto _err;
 
-       ret = snd_pcm_direct_semaphore_create_or_connect(dmix);
+       ret = snd_pcm_direct_semaphore_create_or_connect(dshare);
        if (ret < 0) {
                SNDERR("unable to create IPC semaphore");
                goto _err;
        }
        
-       ret = snd_pcm_direct_semaphore_down(dmix, DIRECT_IPC_SEM_CLIENT);
+       ret = snd_pcm_direct_semaphore_down(dshare, DIRECT_IPC_SEM_CLIENT);
        if (ret < 0) {
-               snd_pcm_direct_semaphore_discard(dmix);
+               snd_pcm_direct_semaphore_discard(dshare);
                goto _err;
        }
-               
-       first_instance = ret = snd_pcm_direct_shm_create_or_connect(dmix);
+
+       first_instance = ret = snd_pcm_direct_shm_create_or_connect(dshare);
        if (ret < 0) {
                SNDERR("unable to create IPC shm instance");
                goto _err;
@@ -756,8 +728,8 @@ int snd_pcm_dshare_open(snd_pcm_t **pcmp, const char *name,
                
        pcm->ops = &snd_pcm_dshare_ops;
        pcm->fast_ops = &snd_pcm_dshare_fast_ops;
-       pcm->private_data = dmix;
-       dmix->state = SND_PCM_STATE_OPEN;
+       pcm->private_data = dshare;
+       dshare->state = SND_PCM_STATE_OPEN;
 
        if (first_instance) {
                ret = snd_pcm_open_slave(&spcm, root, sconf, stream, mode);
@@ -767,33 +739,33 @@ int snd_pcm_dshare_open(snd_pcm_t **pcmp, const char *name,
                }
        
                if (snd_pcm_type(spcm) != SND_PCM_TYPE_HW) {
-                       SNDERR("dmix plugin can be only connected to hw plugin");
+                       SNDERR("dshare plugin can be only connected to hw plugin");
                        goto _err;
                }
                
-               ret = snd_pcm_direct_initialize_slave(dmix, spcm, params);
+               ret = snd_pcm_direct_initialize_slave(dshare, spcm, params);
                if (ret < 0) {
                        SNDERR("unable to initialize slave");
                        goto _err;
                }
 
-               dmix->spcm = spcm;
+               dshare->spcm = spcm;
                
-               ret = snd_pcm_direct_server_create(dmix);
+               ret = snd_pcm_direct_server_create(dshare);
                if (ret < 0) {
                        SNDERR("unable to create server");
                        goto _err;
                }
 
-               dmix->shmptr->type = spcm->type;
+               dshare->shmptr->type = spcm->type;
        } else {
-               ret = snd_pcm_direct_client_connect(dmix);
+               ret = snd_pcm_direct_client_connect(dshare);
                if (ret < 0) {
                        SNDERR("unable to connect client");
                        return ret;
                }
                        
-               ret = snd_pcm_hw_open_fd(&spcm, "dmix_client", dmix->hw_fd, 0);
+               ret = snd_pcm_hw_open_fd(&spcm, "dshare_client", dshare->hw_fd, 0);
                if (ret < 0) {
                        SNDERR("unable to open hardware");
                        goto _err;
@@ -801,61 +773,70 @@ int snd_pcm_dshare_open(snd_pcm_t **pcmp, const char *name,
                
                spcm->donot_close = 1;
                spcm->setup = 1;
-               spcm->buffer_size = dmix->shmptr->s.buffer_size;
-               spcm->sample_bits = dmix->shmptr->s.sample_bits;
-               spcm->channels = dmix->shmptr->s.channels;
-               spcm->format = dmix->shmptr->s.format;
-               spcm->boundary = dmix->shmptr->s.boundary;
+               spcm->buffer_size = dshare->shmptr->s.buffer_size;
+               spcm->sample_bits = dshare->shmptr->s.sample_bits;
+               spcm->channels = dshare->shmptr->s.channels;
+               spcm->format = dshare->shmptr->s.format;
+               spcm->boundary = dshare->shmptr->s.boundary;
                ret = snd_pcm_mmap(spcm);
                if (ret < 0) {
                        SNDERR("unable to mmap channels");
                        goto _err;
                }
-               dmix->spcm = spcm;
+               dshare->spcm = spcm;
        }
 
-       ret = snd_pcm_direct_initialize_poll_fd(dmix);
+       for (chn = 0; chn < dshare->channels; chn++)
+               dshare->u.dshare.chn_mask |= (1ULL<<dshare->bindings[chn]);
+       if (dshare->shmptr->u.dshare.chn_mask & dshare->u.dshare.chn_mask) {
+               SNDERR("destination channel specified in bindings is already used");
+               dshare->u.dshare.chn_mask = 0;
+               ret = -EINVAL;
+               goto _err;
+       }
+       dshare->shmptr->u.dshare.chn_mask |= dshare->u.dshare.chn_mask;
+               
+       ret = snd_pcm_direct_initialize_poll_fd(dshare);
        if (ret < 0) {
                SNDERR("unable to initialize poll_fd");
                goto _err;
        }
 
-       pcm->poll_fd = dmix->poll_fd;
+       pcm->poll_fd = dshare->poll_fd;
        pcm->poll_events = POLLIN;      /* it's different than other plugins */
                
        pcm->mmap_rw = 1;
-       snd_pcm_set_hw_ptr(pcm, &dmix->hw_ptr, -1, 0);
-       snd_pcm_set_appl_ptr(pcm, &dmix->appl_ptr, -1, 0);
-       
-       if (dmix->u.dmix.channels == UINT_MAX)
-               dmix->u.dmix.channels = dmix->shmptr->s.channels;
+       snd_pcm_set_hw_ptr(pcm, &dshare->hw_ptr, -1, 0);
+       snd_pcm_set_appl_ptr(pcm, &dshare->appl_ptr, -1, 0);
        
-       snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+       snd_pcm_direct_semaphore_up(dshare, DIRECT_IPC_SEM_CLIENT);
 
        *pcmp = pcm;
        return 0;
        
  _err:
-       if (dmix) {
-               if (dmix->timer)
-                       snd_timer_close(dmix->timer);
-               if (dmix->server)
-                       snd_pcm_direct_server_discard(dmix);
-               if (dmix->client)
-                       snd_pcm_direct_client_discard(dmix);
+       if (dshare) {
+               if (dshare->shmptr)
+                       dshare->shmptr->u.dshare.chn_mask &= ~dshare->u.dshare.chn_mask;
+               if (dshare->timer)
+                       snd_timer_close(dshare->timer);
+               if (dshare->server)
+                       snd_pcm_direct_server_discard(dshare);
+               if (dshare->client)
+                       snd_pcm_direct_client_discard(dshare);
                if (spcm)
                        snd_pcm_close(spcm);
-               if (dmix->shmid >= 0) {
-                       if (snd_pcm_direct_shm_discard(dmix) > 0) {
-                               if (dmix->semid >= 0) {
-                                       if (snd_pcm_direct_semaphore_discard(dmix) < 0)
-                                               snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+               if (dshare->shmid >= 0) {
+                       if (snd_pcm_direct_shm_discard(dshare) > 0) {
+                               if (dshare->semid >= 0) {
+                                       if (snd_pcm_direct_semaphore_discard(dshare) < 0)
+                                               snd_pcm_direct_semaphore_up(dshare, DIRECT_IPC_SEM_CLIENT);
                                }
                        }
                }
-               if (dmix->u.dmix.bindings)
-                       free(dmix->u.dmix.bindings);
-               free(dmix);
+               if (dshare->bindings)
+                       free(dshare->bindings);
+               free(dshare);
        }
        if (pcm)
                snd_pcm_free(pcm);
@@ -906,11 +887,11 @@ pcm.name {
 */
 
 /**
- * \brief Creates a new dmix PCM
+ * \brief Creates a new dshare PCM
  * \param pcmp Returns created PCM handle
  * \param name Name of PCM
  * \param root Root configuration node
- * \param conf Configuration node with dmix PCM description
+ * \param conf Configuration node with dshare PCM description
  * \param stream PCM Stream
  * \param mode PCM Mode
  * \warning Using of this function might be dangerous in the sense
@@ -1001,14 +982,6 @@ int _snd_pcm_dshare_open(snd_pcm_t **pcmp, const char *name,
        if (err < 0)
                return err;
 
-       /* sorry, limited features */
-       if (params.format != SND_PCM_FORMAT_S16 &&
-           params.format != SND_PCM_FORMAT_S32) {
-               SNDERR("invalid format, specify s16 or s32");
-               snd_config_delete(sconf);
-               return -EINVAL;
-       }
-
        params.period_size = psize;
        params.buffer_size = bsize;
        err = snd_pcm_dshare_open(pcmp, name, ipc_key, &params, bindings, root, sconf, stream, mode);
index b3d711d..22590f2 100644 (file)
@@ -52,85 +52,94 @@ const char *_snd_module_pcm_dsnoop = "";
  *
  */
 
-static void snoop_areas(snd_pcm_direct_t *dmix,
+static void snoop_areas(snd_pcm_direct_t *dsnoop,
                        const snd_pcm_channel_area_t *src_areas,
                        const snd_pcm_channel_area_t *dst_areas,
                        snd_pcm_uframes_t src_ofs,
                        snd_pcm_uframes_t dst_ofs,
                        snd_pcm_uframes_t size)
 {
-       if (dmix->interleaved) {
+       unsigned int chn, schn, channels;
+       snd_pcm_format_t format;
+
+       channels = dsnoop->channels;
+       format = dsnoop->shmptr->s.format;
+       if (dsnoop->interleaved) {
+               unsigned int fbytes = snd_pcm_format_physical_width(format) / 8;
+               memcpy(((char *)dst_areas[0].addr) + (dst_ofs * channels * fbytes),
+                      ((char *)src_areas[0].addr) + (src_ofs * channels * fbytes),
+                      size * channels * fbytes);
        } else {
+               for (chn = 0; chn < channels; chn++) {
+                       schn = dsnoop->bindings ? dsnoop->bindings[chn] : chn;
+                       snd_pcm_area_copy(&dst_areas[chn], dst_ofs, &src_areas[schn], src_ofs, size, format);
+               }
        }
 }
 
 /*
  *  synchronize shm ring buffer with hardware
  */
-static void snd_pcm_dsnoop_sync_area(snd_pcm_t *pcm, snd_pcm_uframes_t size)
+static void snd_pcm_dsnoop_sync_area(snd_pcm_t *pcm, snd_pcm_uframes_t slave_hw_ptr, snd_pcm_uframes_t size)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       snd_pcm_uframes_t appl_ptr, slave_appl_ptr, transfer;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+       snd_pcm_uframes_t hw_ptr = dsnoop->hw_ptr;
+       snd_pcm_uframes_t transfer;
        const snd_pcm_channel_area_t *src_areas, *dst_areas;
        
-       /* get the start of update area */
-       appl_ptr = dmix->appl_ptr - size;
-       if (appl_ptr > pcm->boundary)
-               appl_ptr += pcm->boundary;
-       appl_ptr %= pcm->buffer_size;
        /* add sample areas here */
-       src_areas = snd_pcm_mmap_areas(pcm);
-       dst_areas = snd_pcm_mmap_areas(dmix->spcm);
-       slave_appl_ptr = dmix->slave_appl_ptr % dmix->shmptr->s.buffer_size;
-       dmix->slave_appl_ptr += size;
-       dmix->slave_appl_ptr %= dmix->shmptr->s.boundary;
+       dst_areas = snd_pcm_mmap_areas(pcm);
+       src_areas = snd_pcm_mmap_areas(dsnoop->spcm);
+       hw_ptr %= pcm->buffer_size;
+       slave_hw_ptr %= dsnoop->shmptr->s.buffer_size;
        while (size > 0) {
-               transfer = appl_ptr + size > pcm->buffer_size ? pcm->buffer_size - appl_ptr : size;
-               transfer = slave_appl_ptr + transfer > dmix->shmptr->s.buffer_size ? dmix->shmptr->s.buffer_size - slave_appl_ptr : transfer;
+               transfer = hw_ptr + size > pcm->buffer_size ? pcm->buffer_size - hw_ptr : size;
+               transfer = slave_hw_ptr + transfer > dsnoop->shmptr->s.buffer_size ? dsnoop->shmptr->s.buffer_size - slave_hw_ptr : transfer;
                size -= transfer;
-               snoop_areas(dmix, src_areas, dst_areas, appl_ptr, slave_appl_ptr, transfer);
-               slave_appl_ptr += transfer;
-               slave_appl_ptr %= dmix->shmptr->s.buffer_size;
-               appl_ptr += transfer;
-               appl_ptr %= pcm->buffer_size;
+               snoop_areas(dsnoop, src_areas, dst_areas, slave_hw_ptr, hw_ptr, transfer);
+               slave_hw_ptr += transfer;
+               slave_hw_ptr %= dsnoop->shmptr->s.buffer_size;
+               hw_ptr += transfer;
+               hw_ptr %= pcm->buffer_size;
        }
 }
 
 /*
  *  synchronize hardware pointer (hw_ptr) with ours
  */
-static int snd_pcm_dsnoop_sync_ptr(snd_pcm_t *pcm)
+static snd_pcm_sframes_t snd_pcm_dsnoop_sync_ptr(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        snd_pcm_uframes_t slave_hw_ptr, old_slave_hw_ptr, avail;
        snd_pcm_sframes_t diff;
        
-       old_slave_hw_ptr = dmix->slave_hw_ptr;
-       slave_hw_ptr = dmix->slave_hw_ptr = *dmix->spcm->hw.ptr;
+       old_slave_hw_ptr = dsnoop->slave_hw_ptr;
+       slave_hw_ptr = dsnoop->slave_hw_ptr = *dsnoop->spcm->hw.ptr;
        diff = slave_hw_ptr - old_slave_hw_ptr;
        if (diff == 0)          /* fast path */
                return 0;
        if (diff < 0) {
-               slave_hw_ptr += dmix->shmptr->s.boundary;
+               slave_hw_ptr += dsnoop->shmptr->s.boundary;
                diff = slave_hw_ptr - old_slave_hw_ptr;
        }
-       dmix->hw_ptr += diff;
-       dmix->hw_ptr %= pcm->boundary;
+       snd_pcm_dsnoop_sync_area(pcm, old_slave_hw_ptr, diff);
+       dsnoop->hw_ptr += diff;
+       dsnoop->hw_ptr %= pcm->boundary;
        // printf("sync ptr diff = %li\n", diff);
        if (pcm->stop_threshold >= pcm->boundary)       /* don't care */
                return 0;
-       if ((avail = snd_pcm_mmap_playback_avail(pcm)) >= pcm->stop_threshold) {
+       if ((avail = snd_pcm_mmap_capture_hw_avail(pcm)) >= pcm->stop_threshold) {
                struct timeval tv;
                gettimeofday(&tv, 0);
-               dmix->trigger_tstamp.tv_sec = tv.tv_sec;
-               dmix->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
-               dmix->state = SND_PCM_STATE_XRUN;
-               dmix->avail_max = avail;
+               dsnoop->trigger_tstamp.tv_sec = tv.tv_sec;
+               dsnoop->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
+               dsnoop->state = SND_PCM_STATE_XRUN;
+               dsnoop->avail_max = avail;
                return -EPIPE;
        }
-       if (avail > dmix->avail_max)
-               dmix->avail_max = avail;
-       return 0;
+       if (avail > dsnoop->avail_max)
+               dsnoop->avail_max = avail;
+       return diff;
 }
 
 /*
@@ -145,13 +154,13 @@ static int snd_pcm_dsnoop_nonblock(snd_pcm_t *pcm ATTRIBUTE_UNUSED, int nonblock
 
 static int snd_pcm_dsnoop_async(snd_pcm_t *pcm, int sig, pid_t pid)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       return snd_timer_async(dmix->timer, sig, pid);
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+       return snd_timer_async(dsnoop->timer, sig, pid);
 }
 
 static int snd_pcm_dsnoop_poll_revents(snd_pcm_t *pcm, struct pollfd *pfds, unsigned int nfds, unsigned short *revents)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        unsigned short events;
        static snd_timer_read_t rbuf[5];        /* can be overwriten by multiple plugins, we don't need the value */
 
@@ -161,7 +170,7 @@ static int snd_pcm_dsnoop_poll_revents(snd_pcm_t *pcm, struct pollfd *pfds, unsi
                events |= POLLOUT;
                events &= ~POLLIN;
                /* empty the timer read queue */
-               while (snd_timer_read(dmix->timer, &rbuf, sizeof(rbuf)) == sizeof(rbuf)) ;
+               while (snd_timer_read(dsnoop->timer, &rbuf, sizeof(rbuf)) == sizeof(rbuf)) ;
        }
        *revents = events;
        return 0;
@@ -169,7 +178,7 @@ static int snd_pcm_dsnoop_poll_revents(snd_pcm_t *pcm, struct pollfd *pfds, unsi
 
 static int snd_pcm_dsnoop_info(snd_pcm_t *pcm, snd_pcm_info_t * info)
 {
-       // snd_pcm_direct_t *dmix = pcm->private_data;
+       // snd_pcm_direct_t *dsnoop = pcm->private_data;
 
        memset(info, 0, sizeof(*info));
        info->stream = pcm->stream;
@@ -204,7 +213,7 @@ static int hw_param_interval_refine_one(snd_pcm_hw_params_t *params,
                return 0;
        i = hw_param_interval(params, var);
        if (snd_interval_empty(i)) {
-               SNDERR("dmix interval %i empty?", (int)var);
+               SNDERR("dsnoop interval %i empty?", (int)var);
                return -EINVAL;
        }
        if (snd_interval_refine(i, hw_param_interval(src, var)))
@@ -216,8 +225,8 @@ static int hw_param_interval_refine_one(snd_pcm_hw_params_t *params,
 
 static int snd_pcm_dsnoop_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       snd_pcm_hw_params_t *hw_params = &dmix->shmptr->hw_params;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+       snd_pcm_hw_params_t *hw_params = &dsnoop->shmptr->hw_params;
        static snd_mask_t access = { .bits = { 
                                        (1<<SNDRV_PCM_ACCESS_MMAP_INTERLEAVED) |
                                        (1<<SNDRV_PCM_ACCESS_MMAP_NONINTERLEAVED) |
@@ -234,7 +243,7 @@ static int snd_pcm_dsnoop_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
 #endif
        if (params->rmask & (1<<SND_PCM_HW_PARAM_ACCESS)) {
                if (snd_mask_empty(hw_param_mask(params, SND_PCM_HW_PARAM_ACCESS))) {
-                       SNDERR("dmix access mask empty?");
+                       SNDERR("dsnoop access mask empty?");
                        return -EINVAL;
                }
                if (snd_mask_refine(hw_param_mask(params, SND_PCM_HW_PARAM_ACCESS), &access))
@@ -242,7 +251,7 @@ static int snd_pcm_dsnoop_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
        }
        if (params->rmask & (1<<SND_PCM_HW_PARAM_FORMAT)) {
                if (snd_mask_empty(hw_param_mask(params, SND_PCM_HW_PARAM_FORMAT))) {
-                       SNDERR("dmix format mask empty?");
+                       SNDERR("dsnoop format mask empty?");
                        return -EINVAL;
                }
                if (snd_mask_refine_set(hw_param_mask(params, SND_PCM_HW_PARAM_FORMAT),
@@ -252,10 +261,10 @@ static int snd_pcm_dsnoop_hw_refine(snd_pcm_t *pcm, snd_pcm_hw_params_t *params)
        //snd_mask_none(hw_param_mask(params, SND_PCM_HW_PARAM_SUBFORMAT));
        if (params->rmask & (1<<SND_PCM_HW_PARAM_CHANNELS)) {
                if (snd_interval_empty(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS))) {
-                       SNDERR("dmix channels mask empty?");
+                       SNDERR("dsnoop channels mask empty?");
                        return -EINVAL;
                }
-               err = snd_interval_refine_set(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS), dmix->u.dmix.channels);
+               err = snd_interval_refine_set(hw_param_interval(params, SND_PCM_HW_PARAM_CHANNELS), dsnoop->channels);
                if (err < 0)
                        return err;
        }
@@ -311,30 +320,30 @@ static int snd_pcm_dsnoop_channel_info(snd_pcm_t *pcm, snd_pcm_channel_info_t *
 
 static int snd_pcm_dsnoop_status(snd_pcm_t *pcm, snd_pcm_status_t * status)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
 
        memset(status, 0, sizeof(*status));
-       status->state = dmix->state;
-       status->trigger_tstamp = dmix->trigger_tstamp;
-       status->tstamp = snd_pcm_hw_fast_tstamp(dmix->spcm);
-       status->avail = snd_pcm_mmap_playback_avail(pcm);
-       status->avail_max = status->avail > dmix->avail_max ? status->avail : dmix->avail_max;
-       dmix->avail_max = 0;
+       status->state = dsnoop->state;
+       status->trigger_tstamp = dsnoop->trigger_tstamp;
+       status->tstamp = snd_pcm_hw_fast_tstamp(dsnoop->spcm);
+       status->avail = snd_pcm_mmap_capture_avail(pcm);
+       status->avail_max = status->avail > dsnoop->avail_max ? status->avail : dsnoop->avail_max;
+       dsnoop->avail_max = 0;
        return 0;
 }
 
 static snd_pcm_state_t snd_pcm_dsnoop_state(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       return dmix->state;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+       return dsnoop->state;
 }
 
 static int snd_pcm_dsnoop_delay(snd_pcm_t *pcm, snd_pcm_sframes_t *delayp)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        int err;
        
-       switch(dmix->state) {
+       switch(dsnoop->state) {
        case SNDRV_PCM_STATE_DRAINING:
        case SNDRV_PCM_STATE_RUNNING:
                err = snd_pcm_dsnoop_sync_ptr(pcm);
@@ -342,7 +351,7 @@ static int snd_pcm_dsnoop_delay(snd_pcm_t *pcm, snd_pcm_sframes_t *delayp)
                        return err;
        case SNDRV_PCM_STATE_PREPARED:
        case SNDRV_PCM_STATE_SUSPENDED:
-               *delayp = snd_pcm_mmap_playback_hw_avail(pcm);
+               *delayp = snd_pcm_mmap_capture_hw_avail(pcm);
                return 0;
        case SNDRV_PCM_STATE_XRUN:
                return -EPIPE;
@@ -353,9 +362,9 @@ static int snd_pcm_dsnoop_delay(snd_pcm_t *pcm, snd_pcm_sframes_t *delayp)
 
 static int snd_pcm_dsnoop_hwsync(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
 
-       switch(dmix->state) {
+       switch(dsnoop->state) {
        case SNDRV_PCM_STATE_DRAINING:
        case SNDRV_PCM_STATE_RUNNING:
                return snd_pcm_dsnoop_sync_ptr(pcm);
@@ -371,73 +380,66 @@ static int snd_pcm_dsnoop_hwsync(snd_pcm_t *pcm)
 
 static int snd_pcm_dsnoop_prepare(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
 
-       snd_pcm_direct_check_interleave(dmix, pcm);
-       // assert(pcm->boundary == dmix->shmptr->s.boundary);   /* for sure */
-       dmix->state = SND_PCM_STATE_PREPARED;
-       dmix->appl_ptr = 0;
-       dmix->hw_ptr = 0;
+       snd_pcm_direct_check_interleave(dsnoop, pcm);
+       // assert(pcm->boundary == dsnoop->shmptr->s.boundary); /* for sure */
+       dsnoop->state = SND_PCM_STATE_PREPARED;
+       dsnoop->appl_ptr = 0;
+       dsnoop->hw_ptr = 0;
        return 0;
 }
 
 static int snd_pcm_dsnoop_reset(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       dmix->hw_ptr %= pcm->period_size;
-       dmix->appl_ptr = dmix->hw_ptr;
-       dmix->slave_appl_ptr = dmix->slave_hw_ptr = *dmix->spcm->hw.ptr;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+       dsnoop->hw_ptr %= pcm->period_size;
+       dsnoop->appl_ptr = dsnoop->hw_ptr;
+       dsnoop->slave_appl_ptr = dsnoop->slave_hw_ptr = *dsnoop->spcm->hw.ptr;
        return 0;
 }
 
 static int snd_pcm_dsnoop_start(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       snd_pcm_sframes_t avail;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        struct timeval tv;
        int err;
        
-       if (dmix->state != SND_PCM_STATE_PREPARED)
+       if (dsnoop->state != SND_PCM_STATE_PREPARED)
                return -EBADFD;
-       err = snd_timer_start(dmix->timer);
+       err = snd_timer_start(dsnoop->timer);
        if (err < 0)
                return err;
-       dmix->state = SND_PCM_STATE_RUNNING;
-       dmix->slave_appl_ptr = dmix->slave_hw_ptr = *dmix->spcm->hw.ptr;
-       avail = snd_pcm_mmap_playback_hw_avail(pcm);
-       if (avail < 0)
-               return 0;
-       if (avail > (snd_pcm_sframes_t)pcm->buffer_size)
-               avail = pcm->buffer_size;
-       snd_pcm_dsnoop_sync_area(pcm, avail);
+       dsnoop->state = SND_PCM_STATE_RUNNING;
+       dsnoop->slave_appl_ptr = dsnoop->slave_hw_ptr = *dsnoop->spcm->hw.ptr;
        gettimeofday(&tv, 0);
-       dmix->trigger_tstamp.tv_sec = tv.tv_sec;
-       dmix->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
+       dsnoop->trigger_tstamp.tv_sec = tv.tv_sec;
+       dsnoop->trigger_tstamp.tv_nsec = tv.tv_usec * 1000L;
        return 0;
 }
 
 static int snd_pcm_dsnoop_drop(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-       if (dmix->state == SND_PCM_STATE_OPEN)
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+       if (dsnoop->state == SND_PCM_STATE_OPEN)
                return -EBADFD;
-       snd_timer_stop(dmix->timer);
-       dmix->state = SND_PCM_STATE_SETUP;
+       snd_timer_stop(dsnoop->timer);
+       dsnoop->state = SND_PCM_STATE_SETUP;
        return 0;
 }
 
 static int snd_pcm_dsnoop_drain(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        snd_pcm_uframes_t stop_threshold;
        int err;
 
-       if (dmix->state == SND_PCM_STATE_OPEN)
+       if (dsnoop->state == SND_PCM_STATE_OPEN)
                return -EBADFD;
        stop_threshold = pcm->stop_threshold;
        if (pcm->stop_threshold > pcm->buffer_size)
                pcm->stop_threshold = pcm->buffer_size;
-       while (dmix->state == SND_PCM_STATE_RUNNING) {
+       while (dsnoop->state == SND_PCM_STATE_RUNNING) {
                err = snd_pcm_dsnoop_sync_ptr(pcm);
                if (err < 0)
                        break;
@@ -451,24 +453,23 @@ static int snd_pcm_dsnoop_drain(snd_pcm_t *pcm)
 
 static int snd_pcm_dsnoop_pause(snd_pcm_t *pcm, int enable)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
         if (enable) {
-               if (dmix->state != SND_PCM_STATE_RUNNING)
+               if (dsnoop->state != SND_PCM_STATE_RUNNING)
                        return -EBADFD;
-               dmix->state = SND_PCM_STATE_PAUSED;
-               snd_timer_stop(dmix->timer);
+               dsnoop->state = SND_PCM_STATE_PAUSED;
+               snd_timer_stop(dsnoop->timer);
        } else {
-               if (dmix->state != SND_PCM_STATE_PAUSED)
+               if (dsnoop->state != SND_PCM_STATE_PAUSED)
                        return -EBADFD;
-                dmix->state = SND_PCM_STATE_RUNNING;
-                snd_timer_start(dmix->timer);
+                dsnoop->state = SND_PCM_STATE_RUNNING;
+                snd_timer_start(dsnoop->timer);
        }
        return 0;
 }
 
 static snd_pcm_sframes_t snd_pcm_dsnoop_rewind(snd_pcm_t *pcm, snd_pcm_uframes_t frames)
 {
-       /* FIXME: substract samples from the mix ring buffer, too? */
        snd_pcm_mmap_appl_backward(pcm, frames);
        return frames;
 }
@@ -477,7 +478,7 @@ static snd_pcm_sframes_t snd_pcm_dsnoop_forward(snd_pcm_t *pcm, snd_pcm_uframes_
 {
        snd_pcm_sframes_t avail;
 
-       avail = snd_pcm_mmap_avail(pcm);
+       avail = snd_pcm_mmap_capture_hw_avail(pcm);
        if (avail < 0)
                return 0;
        if (frames > (snd_pcm_uframes_t)avail)
@@ -488,17 +489,17 @@ static snd_pcm_sframes_t snd_pcm_dsnoop_forward(snd_pcm_t *pcm, snd_pcm_uframes_
 
 static int snd_pcm_dsnoop_resume(snd_pcm_t *pcm ATTRIBUTE_UNUSED)
 {
-       // snd_pcm_direct_t *dmix = pcm->private_data;
+       // snd_pcm_direct_t *dsnoop = pcm->private_data;
        // FIXME
        return 0;
 }
 
-static snd_pcm_sframes_t snd_pcm_dsnoop_readi(snd_pcm_t *pcm ATTRIBUTE_UNUSED, void *buffer ATTRIBUTE_UNUSED, snd_pcm_uframes_t size ATTRIBUTE_UNUSED)
+static snd_pcm_sframes_t snd_pcm_dsnoop_writei(snd_pcm_t *pcm ATTRIBUTE_UNUSED, const void *buffer ATTRIBUTE_UNUSED, snd_pcm_uframes_t size ATTRIBUTE_UNUSED)
 {
        return -ENODEV;
 }
 
-static snd_pcm_sframes_t snd_pcm_dsnoop_readn(snd_pcm_t *pcm ATTRIBUTE_UNUSED, void **bufs ATTRIBUTE_UNUSED, snd_pcm_uframes_t size ATTRIBUTE_UNUSED)
+static snd_pcm_sframes_t snd_pcm_dsnoop_writen(snd_pcm_t *pcm ATTRIBUTE_UNUSED, void **bufs ATTRIBUTE_UNUSED, snd_pcm_uframes_t size ATTRIBUTE_UNUSED)
 {
        return -ENODEV;
 }
@@ -515,72 +516,69 @@ static int snd_pcm_dsnoop_munmap(snd_pcm_t *pcm ATTRIBUTE_UNUSED)
 
 static int snd_pcm_dsnoop_close(snd_pcm_t *pcm)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
-
-       if (dmix->timer)
-               snd_timer_close(dmix->timer);
-       snd_pcm_direct_semaphore_down(dmix, DIRECT_IPC_SEM_CLIENT);
-       snd_pcm_close(dmix->spcm);
-       if (dmix->server)
-               snd_pcm_direct_server_discard(dmix);
-       if (dmix->client)
-               snd_pcm_direct_client_discard(dmix);
-       if (snd_pcm_direct_shm_discard(dmix) > 0) {
-               if (snd_pcm_direct_semaphore_discard(dmix) < 0)
-                       snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
+
+       if (dsnoop->timer)
+               snd_timer_close(dsnoop->timer);
+       snd_pcm_direct_semaphore_down(dsnoop, DIRECT_IPC_SEM_CLIENT);
+       snd_pcm_close(dsnoop->spcm);
+       if (dsnoop->server)
+               snd_pcm_direct_server_discard(dsnoop);
+       if (dsnoop->client)
+               snd_pcm_direct_client_discard(dsnoop);
+       if (snd_pcm_direct_shm_discard(dsnoop) > 0) {
+               if (snd_pcm_direct_semaphore_discard(dsnoop) < 0)
+                       snd_pcm_direct_semaphore_up(dsnoop, DIRECT_IPC_SEM_CLIENT);
        } else {
-               snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+               snd_pcm_direct_semaphore_up(dsnoop, DIRECT_IPC_SEM_CLIENT);
        }
-       if (dmix->u.dmix.bindings)
-               free(dmix->u.dmix.bindings);
+       if (dsnoop->bindings)
+               free(dsnoop->bindings);
        pcm->private_data = NULL;
-       free(dmix);
+       free(dsnoop);
        return 0;
 }
 
 static snd_pcm_sframes_t snd_pcm_dsnoop_mmap_commit(snd_pcm_t *pcm,
-                                                 snd_pcm_uframes_t offset ATTRIBUTE_UNUSED,
-                                                 snd_pcm_uframes_t size)
+                                                   snd_pcm_uframes_t offset ATTRIBUTE_UNUSED,
+                                                   snd_pcm_uframes_t size)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        int err;
 
-       snd_pcm_mmap_appl_forward(pcm, size);
-       if (dmix->state == SND_PCM_STATE_RUNNING) {
+       if (dsnoop->state == SND_PCM_STATE_RUNNING) {
                err = snd_pcm_dsnoop_sync_ptr(pcm);
                if (err < 0)
                        return err;
-               /* ok, we commit the changes after the validation of area */
-               /* it's intended, although the result might be crappy */
-               snd_pcm_dsnoop_sync_area(pcm, size);
        }
+       snd_pcm_mmap_appl_forward(pcm, size);
        return size;
 }
 
 static snd_pcm_sframes_t snd_pcm_dsnoop_avail_update(snd_pcm_t *pcm ATTRIBUTE_UNUSED)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
        int err;
        
-       if (dmix->state == SND_PCM_STATE_RUNNING) {
+       if (dsnoop->state == SND_PCM_STATE_RUNNING) {
                err = snd_pcm_dsnoop_sync_ptr(pcm);
                if (err < 0)
                        return err;
        }
-       return snd_pcm_mmap_playback_avail(pcm);
+       return snd_pcm_mmap_capture_avail(pcm);
 }
 
 static void snd_pcm_dsnoop_dump(snd_pcm_t *pcm, snd_output_t *out)
 {
-       snd_pcm_direct_t *dmix = pcm->private_data;
+       snd_pcm_direct_t *dsnoop = pcm->private_data;
 
        snd_output_printf(out, "Direct Stream Mixing PCM\n");
        if (pcm->setup) {
                snd_output_printf(out, "\nIts setup is:\n");
                snd_pcm_dump_setup(pcm, out);
        }
-       if (dmix->spcm)
-               snd_pcm_dump(dmix->spcm, out);
+       if (dsnoop->spcm)
+               snd_pcm_dump(dsnoop->spcm, out);
 }
 
 static snd_pcm_ops_t snd_pcm_dsnoop_ops = {
@@ -613,84 +611,16 @@ static snd_pcm_fast_ops_t snd_pcm_dsnoop_fast_ops = {
        rewind: snd_pcm_dsnoop_rewind,
        forward: snd_pcm_dsnoop_forward,
        resume: snd_pcm_dsnoop_resume,
-       writei: snd_pcm_mmap_writei,
-       writen: snd_pcm_mmap_writen,
-       readi: snd_pcm_dsnoop_readi,
-       readn: snd_pcm_dsnoop_readn,
+       writei: snd_pcm_dsnoop_writei,
+       writen: snd_pcm_dsnoop_writen,
+       readi: snd_pcm_mmap_readi,
+       readn: snd_pcm_mmap_readn,
        avail_update: snd_pcm_dsnoop_avail_update,
        mmap_commit: snd_pcm_dsnoop_mmap_commit,
 };
 
-/*
- * parse the channel map
- * id == client channel
- * value == slave's channel
- */
-static int parse_bindings(snd_pcm_direct_t *dmix, snd_config_t *cfg)
-{
-       snd_config_iterator_t i, next;
-       unsigned int chn, chn1, count = 0;
-       int err;
-
-       dmix->u.dmix.channels = UINT_MAX;
-       if (cfg == NULL)
-               return 0;
-       if (snd_config_get_type(cfg) != SND_CONFIG_TYPE_COMPOUND) {
-               SNDERR("invalid type for bindings");
-               return -EINVAL;
-       }
-       snd_config_for_each(i, next, cfg) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               long cchannel;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               err = safe_strtol(id, &cchannel);
-               if (err < 0 || cchannel < 0) {
-                       SNDERR("invalid client channel in binding: %s\n", id);
-                       return -EINVAL;
-               }
-               if ((unsigned)cchannel > count)
-                       count = cchannel + 1;
-       }
-       if (count == 0)
-               return 0;
-       if (count > 1024) {
-               SNDERR("client channel out of range");
-               return -EINVAL;
-       }
-       dmix->u.dmix.bindings = malloc(count * sizeof(unsigned int));
-       for (chn = 0; chn < count; chn++)
-               dmix->u.dmix.bindings[chn] = UINT_MAX;          /* don't route */
-       snd_config_for_each(i, next, cfg) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               long cchannel, schannel;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               safe_strtol(id, &cchannel);
-               if (snd_config_get_integer(n, &schannel) < 0) {
-                       SNDERR("unable to get slave channel (should be integer type) in binding: %s\n", id);
-                       return -EINVAL;
-               }
-               dmix->u.dmix.bindings[cchannel] = schannel;
-       }
-       for (chn = 0; chn < count; chn++) {
-               for (chn1 = 0; chn1 < count; chn1++) {
-                       if (chn == chn1)
-                               continue;
-                       if (dmix->u.dmix.bindings[chn] == dmix->u.dmix.bindings[chn1]) {
-                               SNDERR("unable to route channels %d,%d to same destination %d", chn, chn1, dmix->u.dmix.bindings[chn]);
-                               return -EINVAL;
-                       }
-               }
-       }
-       dmix->u.dmix.channels = count;
-       return 0;
-}
-
 /**
- * \brief Creates a new dmix PCM
+ * \brief Creates a new dsnoop PCM
  * \param pcmp Returns created PCM handle
  * \param name Name of PCM
  * \param ipc_key IPC key for semaphore and shared memory
@@ -711,47 +641,47 @@ int snd_pcm_dsnoop_open(snd_pcm_t **pcmp, const char *name,
                      snd_pcm_stream_t stream, int mode)
 {
        snd_pcm_t *pcm = NULL, *spcm = NULL;
-       snd_pcm_direct_t *dmix = NULL;
+       snd_pcm_direct_t *dsnoop = NULL;
        int ret, first_instance;
 
        assert(pcmp);
 
-       if (stream != SND_PCM_STREAM_PLAYBACK) {
-               SNDERR("The dmix plugin supports only playback stream");
+       if (stream != SND_PCM_STREAM_CAPTURE) {
+               SNDERR("The dsnoop plugin supports only capture stream");
                return -EINVAL;
        }
 
-       dmix = calloc(1, sizeof(snd_pcm_direct_t));
-       if (!dmix) {
+       dsnoop = calloc(1, sizeof(snd_pcm_direct_t));
+       if (!dsnoop) {
                ret = -ENOMEM;
                goto _err;
        }
        
-       ret = parse_bindings(dmix, bindings);
+       ret = snd_pcm_direct_parse_bindings(dsnoop, bindings);
        if (ret < 0)
                goto _err;
        
-       dmix->ipc_key = ipc_key;
-       dmix->semid = -1;
-       dmix->shmid = -1;
+       dsnoop->ipc_key = ipc_key;
+       dsnoop->semid = -1;
+       dsnoop->shmid = -1;
 
-       ret = snd_pcm_new(&pcm, SND_PCM_TYPE_DSNOOP, name, stream, mode);
+       ret = snd_pcm_new(&pcm, dsnoop->type = SND_PCM_TYPE_DSNOOP, name, stream, mode);
        if (ret < 0)
                goto _err;
 
-       ret = snd_pcm_direct_semaphore_create_or_connect(dmix);
+       ret = snd_pcm_direct_semaphore_create_or_connect(dsnoop);
        if (ret < 0) {
                SNDERR("unable to create IPC semaphore");
                goto _err;
        }
        
-       ret = snd_pcm_direct_semaphore_down(dmix, DIRECT_IPC_SEM_CLIENT);
+       ret = snd_pcm_direct_semaphore_down(dsnoop, DIRECT_IPC_SEM_CLIENT);
        if (ret < 0) {
-               snd_pcm_direct_semaphore_discard(dmix);
+               snd_pcm_direct_semaphore_discard(dsnoop);
                goto _err;
        }
                
-       first_instance = ret = snd_pcm_direct_shm_create_or_connect(dmix);
+       first_instance = ret = snd_pcm_direct_shm_create_or_connect(dsnoop);
        if (ret < 0) {
                SNDERR("unable to create IPC shm instance");
                goto _err;
@@ -759,8 +689,8 @@ int snd_pcm_dsnoop_open(snd_pcm_t **pcmp, const char *name,
                
        pcm->ops = &snd_pcm_dsnoop_ops;
        pcm->fast_ops = &snd_pcm_dsnoop_fast_ops;
-       pcm->private_data = dmix;
-       dmix->state = SND_PCM_STATE_OPEN;
+       pcm->private_data = dsnoop;
+       dsnoop->state = SND_PCM_STATE_OPEN;
 
        if (first_instance) {
                ret = snd_pcm_open_slave(&spcm, root, sconf, stream, mode);
@@ -770,33 +700,33 @@ int snd_pcm_dsnoop_open(snd_pcm_t **pcmp, const char *name,
                }
        
                if (snd_pcm_type(spcm) != SND_PCM_TYPE_HW) {
-                       SNDERR("dmix plugin can be only connected to hw plugin");
+                       SNDERR("dsnoop plugin can be only connected to hw plugin");
                        goto _err;
                }
                
-               ret = snd_pcm_direct_initialize_slave(dmix, spcm, params);
+               ret = snd_pcm_direct_initialize_slave(dsnoop, spcm, params);
                if (ret < 0) {
                        SNDERR("unable to initialize slave");
                        goto _err;
                }
 
-               dmix->spcm = spcm;
+               dsnoop->spcm = spcm;
                
-               ret = snd_pcm_direct_server_create(dmix);
+               ret = snd_pcm_direct_server_create(dsnoop);
                if (ret < 0) {
                        SNDERR("unable to create server");
                        goto _err;
                }
 
-               dmix->shmptr->type = spcm->type;
+               dsnoop->shmptr->type = spcm->type;
        } else {
-               ret = snd_pcm_direct_client_connect(dmix);
+               ret = snd_pcm_direct_client_connect(dsnoop);
                if (ret < 0) {
                        SNDERR("unable to connect client");
                        return ret;
                }
                        
-               ret = snd_pcm_hw_open_fd(&spcm, "dmix_client", dmix->hw_fd, 0);
+               ret = snd_pcm_hw_open_fd(&spcm, "dsnoop_client", dsnoop->hw_fd, 0);
                if (ret < 0) {
                        SNDERR("unable to open hardware");
                        goto _err;
@@ -804,61 +734,61 @@ int snd_pcm_dsnoop_open(snd_pcm_t **pcmp, const char *name,
                
                spcm->donot_close = 1;
                spcm->setup = 1;
-               spcm->buffer_size = dmix->shmptr->s.buffer_size;
-               spcm->sample_bits = dmix->shmptr->s.sample_bits;
-               spcm->channels = dmix->shmptr->s.channels;
-               spcm->format = dmix->shmptr->s.format;
-               spcm->boundary = dmix->shmptr->s.boundary;
+               spcm->buffer_size = dsnoop->shmptr->s.buffer_size;
+               spcm->sample_bits = dsnoop->shmptr->s.sample_bits;
+               spcm->channels = dsnoop->shmptr->s.channels;
+               spcm->format = dsnoop->shmptr->s.format;
+               spcm->boundary = dsnoop->shmptr->s.boundary;
                ret = snd_pcm_mmap(spcm);
                if (ret < 0) {
                        SNDERR("unable to mmap channels");
                        goto _err;
                }
-               dmix->spcm = spcm;
+               dsnoop->spcm = spcm;
        }
 
-       ret = snd_pcm_direct_initialize_poll_fd(dmix);
+       ret = snd_pcm_direct_initialize_poll_fd(dsnoop);
        if (ret < 0) {
                SNDERR("unable to initialize poll_fd");
                goto _err;
        }
 
-       pcm->poll_fd = dmix->poll_fd;
+       pcm->poll_fd = dsnoop->poll_fd;
        pcm->poll_events = POLLIN;      /* it's different than other plugins */
                
        pcm->mmap_rw = 1;
-       snd_pcm_set_hw_ptr(pcm, &dmix->hw_ptr, -1, 0);
-       snd_pcm_set_appl_ptr(pcm, &dmix->appl_ptr, -1, 0);
+       snd_pcm_set_hw_ptr(pcm, &dsnoop->hw_ptr, -1, 0);
+       snd_pcm_set_appl_ptr(pcm, &dsnoop->appl_ptr, -1, 0);
        
-       if (dmix->u.dmix.channels == UINT_MAX)
-               dmix->u.dmix.channels = dmix->shmptr->s.channels;
+       if (dsnoop->channels == UINT_MAX)
+               dsnoop->channels = dsnoop->shmptr->s.channels;
        
-       snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+       snd_pcm_direct_semaphore_up(dsnoop, DIRECT_IPC_SEM_CLIENT);
 
        *pcmp = pcm;
        return 0;
        
  _err:
-       if (dmix) {
-               if (dmix->timer)
-                       snd_timer_close(dmix->timer);
-               if (dmix->server)
-                       snd_pcm_direct_server_discard(dmix);
-               if (dmix->client)
-                       snd_pcm_direct_client_discard(dmix);
+       if (dsnoop) {
+               if (dsnoop->timer)
+                       snd_timer_close(dsnoop->timer);
+               if (dsnoop->server)
+                       snd_pcm_direct_server_discard(dsnoop);
+               if (dsnoop->client)
+                       snd_pcm_direct_client_discard(dsnoop);
                if (spcm)
                        snd_pcm_close(spcm);
-               if (dmix->shmid >= 0) {
-                       if (snd_pcm_direct_shm_discard(dmix) > 0) {
-                               if (dmix->semid >= 0) {
-                                       if (snd_pcm_direct_semaphore_discard(dmix) < 0)
-                                               snd_pcm_direct_semaphore_up(dmix, DIRECT_IPC_SEM_CLIENT);
+               if (dsnoop->shmid >= 0) {
+                       if (snd_pcm_direct_shm_discard(dsnoop) > 0) {
+                               if (dsnoop->semid >= 0) {
+                                       if (snd_pcm_direct_semaphore_discard(dsnoop) < 0)
+                                               snd_pcm_direct_semaphore_up(dsnoop, DIRECT_IPC_SEM_CLIENT);
                                }
                        }
                }
-               if (dmix->u.dmix.bindings)
-                       free(dmix->u.dmix.bindings);
-               free(dmix);
+               if (dsnoop->bindings)
+                       free(dsnoop->bindings);
+               free(dsnoop);
        }
        if (pcm)
                snd_pcm_free(pcm);
@@ -909,11 +839,11 @@ pcm.name {
 */
 
 /**
- * \brief Creates a new dmix PCM
+ * \brief Creates a new dsnoop PCM
  * \param pcmp Returns created PCM handle
  * \param name Name of PCM
  * \param root Root configuration node
- * \param conf Configuration node with dmix PCM description
+ * \param conf Configuration node with dsnoop PCM description
  * \param stream PCM Stream
  * \param mode PCM Mode
  * \warning Using of this function might be dangerous in the sense