]> err.no Git - linux-2.6/blobdiff - sound/core/pcm.c
Merge branch 'linus'
[linux-2.6] / sound / core / pcm.c
index 2bc5f69ec2a850c879d795830e96305516c2d9d3..3da6a38c2d0fd6391da878e8bb6921f0e0b90274 100644 (file)
@@ -23,6 +23,7 @@
 #include <linux/init.h>
 #include <linux/slab.h>
 #include <linux/time.h>
+#include <linux/mutex.h>
 #include <sound/core.h>
 #include <sound/minors.h>
 #include <sound/pcm.h>
@@ -33,9 +34,9 @@ MODULE_AUTHOR("Jaroslav Kysela <perex@suse.cz>, Abramo Bagnara <abramo@alsa-proj
 MODULE_DESCRIPTION("Midlevel PCM code for ALSA.");
 MODULE_LICENSE("GPL");
 
-struct snd_pcm *snd_pcm_devices[SNDRV_CARDS * SNDRV_PCM_DEVICES];
+static LIST_HEAD(snd_pcm_devices);
 static LIST_HEAD(snd_pcm_notify_list);
-static DECLARE_MUTEX(register_mutex);
+static DEFINE_MUTEX(register_mutex);
 
 static int snd_pcm_free(struct snd_pcm *pcm);
 static int snd_pcm_dev_free(struct snd_device *device);
@@ -43,13 +44,23 @@ static int snd_pcm_dev_register(struct snd_device *device);
 static int snd_pcm_dev_disconnect(struct snd_device *device);
 static int snd_pcm_dev_unregister(struct snd_device *device);
 
+static struct snd_pcm *snd_pcm_search(struct snd_card *card, int device)
+{
+       struct list_head *p;
+       struct snd_pcm *pcm;
+
+       list_for_each(p, &snd_pcm_devices) {
+               pcm = list_entry(p, struct snd_pcm, list);
+               if (pcm->card == card && pcm->device == device)
+                       return pcm;
+       }
+       return NULL;
+}
+
 static int snd_pcm_control_ioctl(struct snd_card *card,
                                 struct snd_ctl_file *control,
                                 unsigned int cmd, unsigned long arg)
 {
-       unsigned int tmp;
-
-       tmp = card->number * SNDRV_PCM_DEVICES;
        switch (cmd) {
        case SNDRV_CTL_IOCTL_PCM_NEXT_DEVICE:
                {
@@ -57,14 +68,16 @@ static int snd_pcm_control_ioctl(struct snd_card *card,
 
                        if (get_user(device, (int __user *)arg))
                                return -EFAULT;
+                       mutex_lock(&register_mutex);
                        device = device < 0 ? 0 : device + 1;
                        while (device < SNDRV_PCM_DEVICES) {
-                               if (snd_pcm_devices[tmp + device])
+                               if (snd_pcm_search(card, device))
                                        break;
                                device++;
                        }
                        if (device == SNDRV_PCM_DEVICES)
                                device = -1;
+                       mutex_unlock(&register_mutex);
                        if (put_user(device, (int __user *)arg))
                                return -EFAULT;
                        return 0;
@@ -77,31 +90,44 @@ static int snd_pcm_control_ioctl(struct snd_card *card,
                        struct snd_pcm *pcm;
                        struct snd_pcm_str *pstr;
                        struct snd_pcm_substream *substream;
+                       int err;
+
                        info = (struct snd_pcm_info __user *)arg;
                        if (get_user(device, &info->device))
                                return -EFAULT;
-                       if (device >= SNDRV_PCM_DEVICES)
-                               return -ENXIO;
-                       pcm = snd_pcm_devices[tmp + device];
-                       if (pcm == NULL)
-                               return -ENXIO;
                        if (get_user(stream, &info->stream))
                                return -EFAULT;
                        if (stream < 0 || stream > 1)
                                return -EINVAL;
-                       pstr = &pcm->streams[stream];
-                       if (pstr->substream_count == 0)
-                               return -ENOENT;
                        if (get_user(subdevice, &info->subdevice))
                                return -EFAULT;
-                       if (subdevice >= pstr->substream_count)
-                               return -ENXIO;
-                       for (substream = pstr->substream; substream; substream = substream->next)
+                       mutex_lock(&register_mutex);
+                       pcm = snd_pcm_search(card, device);
+                       if (pcm == NULL) {
+                               err = -ENXIO;
+                               goto _error;
+                       }
+                       pstr = &pcm->streams[stream];
+                       if (pstr->substream_count == 0) {
+                               err = -ENOENT;
+                               goto _error;
+                       }
+                       if (subdevice >= pstr->substream_count) {
+                               err = -ENXIO;
+                               goto _error;
+                       }
+                       for (substream = pstr->substream; substream;
+                            substream = substream->next)
                                if (substream->number == (int)subdevice)
                                        break;
-                       if (substream == NULL)
-                               return -ENXIO;
-                       return snd_pcm_info_user(substream, info);
+                       if (substream == NULL) {
+                               err = -ENXIO;
+                               goto _error;
+                       }
+                       err = snd_pcm_info_user(substream, info);
+               _error:
+                       mutex_unlock(&register_mutex);
+                       return err;
                }
        case SNDRV_CTL_IOCTL_PCM_PREFER_SUBDEVICE:
                {
@@ -115,6 +141,9 @@ static int snd_pcm_control_ioctl(struct snd_card *card,
        }
        return -ENOIOCTLCMD;
 }
+
+#if defined(CONFIG_PROC_FS) && defined(CONFIG_SND_VERBOSE_PROCFS)
+
 #define STATE(v) [SNDRV_PCM_STATE_##v] = #v
 #define STREAM(v) [SNDRV_PCM_STREAM_##v] = #v
 #define READY(v) [SNDRV_PCM_READY_##v] = #v
@@ -126,30 +155,6 @@ static int snd_pcm_control_ioctl(struct snd_card *card,
 #define FORMAT(v) [SNDRV_PCM_FORMAT_##v] = #v
 #define SUBFORMAT(v) [SNDRV_PCM_SUBFORMAT_##v] = #v 
 
-static char *snd_pcm_stream_names[] = {
-       STREAM(PLAYBACK),
-       STREAM(CAPTURE),
-};
-
-static char *snd_pcm_state_names[] = {
-       STATE(OPEN),
-       STATE(SETUP),
-       STATE(PREPARED),
-       STATE(RUNNING),
-       STATE(XRUN),
-       STATE(DRAINING),
-       STATE(PAUSED),
-       STATE(SUSPENDED),
-};
-
-static char *snd_pcm_access_names[] = {
-       ACCESS(MMAP_INTERLEAVED), 
-       ACCESS(MMAP_NONINTERLEAVED),
-       ACCESS(MMAP_COMPLEX),
-       ACCESS(RW_INTERLEAVED),
-       ACCESS(RW_NONINTERLEAVED),
-};
-
 static char *snd_pcm_format_names[] = {
        FORMAT(S8),
        FORMAT(U8),
@@ -191,6 +196,35 @@ static char *snd_pcm_format_names[] = {
        FORMAT(U18_3BE),
 };
 
+const char *snd_pcm_format_name(snd_pcm_format_t format)
+{
+       return snd_pcm_format_names[format];
+}
+
+static char *snd_pcm_stream_names[] = {
+       STREAM(PLAYBACK),
+       STREAM(CAPTURE),
+};
+
+static char *snd_pcm_state_names[] = {
+       STATE(OPEN),
+       STATE(SETUP),
+       STATE(PREPARED),
+       STATE(RUNNING),
+       STATE(XRUN),
+       STATE(DRAINING),
+       STATE(PAUSED),
+       STATE(SUSPENDED),
+};
+
+static char *snd_pcm_access_names[] = {
+       ACCESS(MMAP_INTERLEAVED), 
+       ACCESS(MMAP_NONINTERLEAVED),
+       ACCESS(MMAP_COMPLEX),
+       ACCESS(RW_INTERLEAVED),
+       ACCESS(RW_NONINTERLEAVED),
+};
+
 static char *snd_pcm_subformat_names[] = {
        SUBFORMAT(STD), 
 };
@@ -211,11 +245,6 @@ static const char *snd_pcm_access_name(snd_pcm_access_t access)
        return snd_pcm_access_names[access];
 }
 
-const char *snd_pcm_format_name(snd_pcm_format_t format)
-{
-       return snd_pcm_format_names[format];
-}
-
 static const char *snd_pcm_subformat_name(snd_pcm_subformat_t subformat)
 {
        return snd_pcm_subformat_names[subformat];
@@ -234,6 +263,7 @@ static const char *snd_pcm_state_name(snd_pcm_state_t state)
 
 #if defined(CONFIG_SND_PCM_OSS) || defined(CONFIG_SND_PCM_OSS_MODULE)
 #include <linux/soundcard.h>
+
 static const char *snd_pcm_oss_format_name(int format)
 {
        switch (format) {
@@ -263,7 +293,6 @@ static const char *snd_pcm_oss_format_name(int format)
 }
 #endif
 
-#ifdef CONFIG_PROC_FS
 static void snd_pcm_proc_info_read(struct snd_pcm_substream *substream,
                                   struct snd_info_buffer *buffer)
 {
@@ -406,7 +435,6 @@ static void snd_pcm_substream_proc_status_read(struct snd_info_entry *entry,
        snd_iprintf(buffer, "hw_ptr      : %ld\n", runtime->status->hw_ptr);
        snd_iprintf(buffer, "appl_ptr    : %ld\n", runtime->control->appl_ptr);
 }
-#endif
 
 #ifdef CONFIG_SND_DEBUG
 static void snd_pcm_xrun_debug_read(struct snd_info_entry *entry,
@@ -571,6 +599,12 @@ static int snd_pcm_substream_proc_done(struct snd_pcm_substream *substream)
        }
        return 0;
 }
+#else /* !CONFIG_PROC_FS */
+static inline int snd_pcm_stream_proc_init(struct snd_pcm_str *pstr) { return 0; }
+static inline int snd_pcm_stream_proc_done(struct snd_pcm_str *pstr) { return 0; }
+static inline int snd_pcm_substream_proc_init(struct snd_pcm_substream *substream) { return 0; }
+static inline int snd_pcm_substream_proc_done(struct snd_pcm_substream *substream) { return 0; }
+#endif /* CONFIG_PROC_FS */
 
 /**
  * snd_pcm_new_stream - create a new PCM stream
@@ -592,7 +626,7 @@ int snd_pcm_new_stream(struct snd_pcm *pcm, int stream, int substream_count)
        struct snd_pcm_substream *substream, *prev;
 
 #if defined(CONFIG_SND_PCM_OSS) || defined(CONFIG_SND_PCM_OSS_MODULE)
-       init_MUTEX(&pstr->oss.setup_mutex);
+       mutex_init(&pstr->oss.setup_mutex);
 #endif
        pstr->stream = stream;
        pstr->pcm = pcm;
@@ -686,7 +720,7 @@ int snd_pcm_new(struct snd_card *card, char *id, int device,
                snd_pcm_free(pcm);
                return err;
        }
-       init_MUTEX(&pcm->open_mutex);
+       mutex_init(&pcm->open_mutex);
        init_waitqueue_head(&pcm->open_wait);
        if ((err = snd_device_new(card, SNDRV_DEV_PCM, pcm, &ops)) < 0) {
                snd_pcm_free(pcm);
@@ -865,21 +899,19 @@ void snd_pcm_release_substream(struct snd_pcm_substream *substream)
 
 static int snd_pcm_dev_register(struct snd_device *device)
 {
-       int idx, cidx, err;
-       unsigned short minor;
+       int cidx, err;
        struct snd_pcm_substream *substream;
        struct list_head *list;
        char str[16];
        struct snd_pcm *pcm = device->device_data;
 
        snd_assert(pcm != NULL && device != NULL, return -ENXIO);
-       down(&register_mutex);
-       idx = (pcm->card->number * SNDRV_PCM_DEVICES) + pcm->device;
-       if (snd_pcm_devices[idx]) {
-               up(&register_mutex);
+       mutex_lock(&register_mutex);
+       if (snd_pcm_search(pcm->card, pcm->device)) {
+               mutex_unlock(&register_mutex);
                return -EBUSY;
        }
-       snd_pcm_devices[idx] = pcm;
+       list_add_tail(&pcm->list, &snd_pcm_devices);
        for (cidx = 0; cidx < 2; cidx++) {
                int devtype = -1;
                if (pcm->streams[cidx].substream == NULL)
@@ -887,21 +919,20 @@ static int snd_pcm_dev_register(struct snd_device *device)
                switch (cidx) {
                case SNDRV_PCM_STREAM_PLAYBACK:
                        sprintf(str, "pcmC%iD%ip", pcm->card->number, pcm->device);
-                       minor = SNDRV_MINOR_PCM_PLAYBACK + idx;
                        devtype = SNDRV_DEVICE_TYPE_PCM_PLAYBACK;
                        break;
                case SNDRV_PCM_STREAM_CAPTURE:
                        sprintf(str, "pcmC%iD%ic", pcm->card->number, pcm->device);
-                       minor = SNDRV_MINOR_PCM_CAPTURE + idx;
                        devtype = SNDRV_DEVICE_TYPE_PCM_CAPTURE;
                        break;
                }
                if ((err = snd_register_device(devtype, pcm->card,
                                               pcm->device,
-                                              &snd_pcm_f_ops[cidx], str)) < 0)
+                                              &snd_pcm_f_ops[cidx],
+                                              pcm, str)) < 0)
                {
-                       snd_pcm_devices[idx] = NULL;
-                       up(&register_mutex);
+                       list_del(&pcm->list);
+                       mutex_unlock(&register_mutex);
                        return err;
                }
                for (substream = pcm->streams[cidx].substream; substream; substream = substream->next)
@@ -912,7 +943,7 @@ static int snd_pcm_dev_register(struct snd_device *device)
                notify = list_entry(list, struct snd_pcm_notify, list);
                notify->n_register(pcm);
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        return 0;
 }
 
@@ -921,11 +952,10 @@ static int snd_pcm_dev_disconnect(struct snd_device *device)
        struct snd_pcm *pcm = device->device_data;
        struct list_head *list;
        struct snd_pcm_substream *substream;
-       int idx, cidx;
+       int cidx;
 
-       down(&register_mutex);
-       idx = (pcm->card->number * SNDRV_PCM_DEVICES) + pcm->device;
-       snd_pcm_devices[idx] = NULL;
+       mutex_lock(&register_mutex);
+       list_del_init(&pcm->list);
        for (cidx = 0; cidx < 2; cidx++)
                for (substream = pcm->streams[cidx].substream; substream; substream = substream->next)
                        if (substream->runtime)
@@ -935,21 +965,20 @@ static int snd_pcm_dev_disconnect(struct snd_device *device)
                notify = list_entry(list, struct snd_pcm_notify, list);
                notify->n_disconnect(pcm);
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        return 0;
 }
 
 static int snd_pcm_dev_unregister(struct snd_device *device)
 {
-       int idx, cidx, devtype;
+       int cidx, devtype;
        struct snd_pcm_substream *substream;
        struct list_head *list;
        struct snd_pcm *pcm = device->device_data;
 
        snd_assert(pcm != NULL, return -ENXIO);
-       down(&register_mutex);
-       idx = (pcm->card->number * SNDRV_PCM_DEVICES) + pcm->device;
-       snd_pcm_devices[idx] = NULL;
+       mutex_lock(&register_mutex);
+       list_del(&pcm->list);
        for (cidx = 0; cidx < 2; cidx++) {
                devtype = -1;
                switch (cidx) {
@@ -969,35 +998,31 @@ static int snd_pcm_dev_unregister(struct snd_device *device)
                notify = list_entry(list, struct snd_pcm_notify, list);
                notify->n_unregister(pcm);
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        return snd_pcm_free(pcm);
 }
 
 int snd_pcm_notify(struct snd_pcm_notify *notify, int nfree)
 {
-       int idx;
+       struct list_head *p;
 
        snd_assert(notify != NULL && notify->n_register != NULL && notify->n_unregister != NULL, return -EINVAL);
-       down(&register_mutex);
+       mutex_lock(&register_mutex);
        if (nfree) {
                list_del(&notify->list);
-               for (idx = 0; idx < SNDRV_CARDS * SNDRV_PCM_DEVICES; idx++) {
-                       if (snd_pcm_devices[idx] == NULL)
-                               continue;
-                       notify->n_unregister(snd_pcm_devices[idx]);
-               }
+               list_for_each(p, &snd_pcm_devices)
+                       notify->n_unregister(list_entry(p,
+                                                       struct snd_pcm, list));
        } else {
                list_add_tail(&notify->list, &snd_pcm_notify_list);
-               for (idx = 0; idx < SNDRV_CARDS * SNDRV_PCM_DEVICES; idx++) {
-                       if (snd_pcm_devices[idx] == NULL)
-                               continue;
-                       notify->n_register(snd_pcm_devices[idx]);
-               }
+               list_for_each(p, &snd_pcm_devices)
+                       notify->n_register(list_entry(p, struct snd_pcm, list));
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
        return 0;
 }
 
+#ifdef CONFIG_PROC_FS
 /*
  *  Info interface
  */
@@ -1005,16 +1030,14 @@ int snd_pcm_notify(struct snd_pcm_notify *notify, int nfree)
 static void snd_pcm_proc_read(struct snd_info_entry *entry,
                              struct snd_info_buffer *buffer)
 {
-       int idx;
+       struct list_head *p;
        struct snd_pcm *pcm;
 
-       down(&register_mutex);
-       for (idx = 0; idx < SNDRV_CARDS * SNDRV_PCM_DEVICES; idx++) {
-               pcm = snd_pcm_devices[idx];
-               if (pcm == NULL)
-                       continue;
-               snd_iprintf(buffer, "%02i-%02i: %s : %s", idx / SNDRV_PCM_DEVICES,
-                           idx % SNDRV_PCM_DEVICES, pcm->id, pcm->name);
+       mutex_lock(&register_mutex);
+       list_for_each(p, &snd_pcm_devices) {
+               pcm = list_entry(p, struct snd_pcm, list);
+               snd_iprintf(buffer, "%02i-%02i: %s : %s",
+                           pcm->card->number, pcm->device, pcm->id, pcm->name);
                if (pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream)
                        snd_iprintf(buffer, " : playback %i",
                                    pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream_count);
@@ -1023,21 +1046,15 @@ static void snd_pcm_proc_read(struct snd_info_entry *entry,
                                    pcm->streams[SNDRV_PCM_STREAM_CAPTURE].substream_count);
                snd_iprintf(buffer, "\n");
        }
-       up(&register_mutex);
+       mutex_unlock(&register_mutex);
 }
 
-/*
- *  ENTRY functions
- */
-
 static struct snd_info_entry *snd_pcm_proc_entry = NULL;
 
-static int __init alsa_pcm_init(void)
+static void snd_pcm_proc_init(void)
 {
        struct snd_info_entry *entry;
 
-       snd_ctl_register_ioctl(snd_pcm_control_ioctl);
-       snd_ctl_register_ioctl_compat(snd_pcm_control_ioctl);
        if ((entry = snd_info_create_module_entry(THIS_MODULE, "pcm", NULL)) != NULL) {
                snd_info_set_text_ops(entry, NULL, SNDRV_CARDS * SNDRV_PCM_DEVICES * 128,
                                      snd_pcm_proc_read);
@@ -1047,6 +1064,29 @@ static int __init alsa_pcm_init(void)
                }
        }
        snd_pcm_proc_entry = entry;
+}
+
+static void snd_pcm_proc_done(void)
+{
+       if (snd_pcm_proc_entry)
+               snd_info_unregister(snd_pcm_proc_entry);
+}
+
+#else /* !CONFIG_PROC_FS */
+#define snd_pcm_proc_init()
+#define snd_pcm_proc_done()
+#endif /* CONFIG_PROC_FS */
+
+
+/*
+ *  ENTRY functions
+ */
+
+static int __init alsa_pcm_init(void)
+{
+       snd_ctl_register_ioctl(snd_pcm_control_ioctl);
+       snd_ctl_register_ioctl_compat(snd_pcm_control_ioctl);
+       snd_pcm_proc_init();
        return 0;
 }
 
@@ -1054,22 +1094,17 @@ static void __exit alsa_pcm_exit(void)
 {
        snd_ctl_unregister_ioctl(snd_pcm_control_ioctl);
        snd_ctl_unregister_ioctl_compat(snd_pcm_control_ioctl);
-       if (snd_pcm_proc_entry) {
-               snd_info_unregister(snd_pcm_proc_entry);
-               snd_pcm_proc_entry = NULL;
-       }
+       snd_pcm_proc_done();
 }
 
 module_init(alsa_pcm_init)
 module_exit(alsa_pcm_exit)
 
-EXPORT_SYMBOL(snd_pcm_devices);
 EXPORT_SYMBOL(snd_pcm_new);
 EXPORT_SYMBOL(snd_pcm_new_stream);
 EXPORT_SYMBOL(snd_pcm_notify);
 EXPORT_SYMBOL(snd_pcm_open_substream);
 EXPORT_SYMBOL(snd_pcm_release_substream);
-EXPORT_SYMBOL(snd_pcm_format_name);
   /* pcm_native.c */
 EXPORT_SYMBOL(snd_pcm_link_rwlock);
 #ifdef CONFIG_PM