diff options
author | Kay Sievers <kay.sievers@vrfy.org> | 2010-11-23 11:43:19 -0500 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2010-11-23 23:53:25 -0500 |
commit | 03cfe6f57dc5c13ccdd235c23c80e3fa170f03d1 (patch) | |
tree | babee2b37a52ec6b0c5f62f8e4783c937fd265d3 /sound/core | |
parent | 109fef9edcc100952eec980acbc2e1295627fbab (diff) |
ALSA: support module on-demand loading for seq and timer
If CONFIG_SND_DYNAMIC_MINORS is used, assign /dev/snd/seq and
/dev/snd/timer the usual static minors, and export specific
module aliases to generate udev module on-demand loading
instructions:
$ cat /lib/modules/2.6.33.4-smp/modules.devname
# Device nodes to trigger on-demand module loading.
microcode cpu/microcode c10:184
fuse fuse c10:229
ppp_generic ppp c108:0
tun net/tun c10:200
uinput uinput c10:223
dm_mod mapper/control c10:236
snd_timer snd/timer c116:33
snd_seq snd/seq c116:1
The last two lines instruct udev to create device nodes, even
when the modules are not loaded at that time.
As soon as userspace accesses any of these nodes, the in-kernel
module-loader will load the module, and the device can be used.
The header file minor calculation needed to be simplified to
make __stringify() (supports only two indirections) in
the MODULE_ALIAS macro work.
This is part of systemd's effort to get rid of unconditional
module load instructions and needless init scripts.
Cc: Lennart Poettering <lennart@poettering.net>
Signed-off-by: Kay Sievers <kay.sievers@vrfy.org>
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/core')
-rw-r--r-- | sound/core/seq/seq.c | 4 | ||||
-rw-r--r-- | sound/core/sound.c | 18 | ||||
-rw-r--r-- | sound/core/timer.c | 3 |
3 files changed, 20 insertions, 5 deletions
diff --git a/sound/core/seq/seq.c b/sound/core/seq/seq.c index bf09a5ad1865..119fddb6fc99 100644 --- a/sound/core/seq/seq.c +++ b/sound/core/seq/seq.c | |||
@@ -32,6 +32,7 @@ | |||
32 | #include "seq_timer.h" | 32 | #include "seq_timer.h" |
33 | #include "seq_system.h" | 33 | #include "seq_system.h" |
34 | #include "seq_info.h" | 34 | #include "seq_info.h" |
35 | #include <sound/minors.h> | ||
35 | #include <sound/seq_device.h> | 36 | #include <sound/seq_device.h> |
36 | 37 | ||
37 | #if defined(CONFIG_SND_SEQ_DUMMY_MODULE) | 38 | #if defined(CONFIG_SND_SEQ_DUMMY_MODULE) |
@@ -73,6 +74,9 @@ MODULE_PARM_DESC(seq_default_timer_subdevice, "The default timer subdevice numbe | |||
73 | module_param(seq_default_timer_resolution, int, 0644); | 74 | module_param(seq_default_timer_resolution, int, 0644); |
74 | MODULE_PARM_DESC(seq_default_timer_resolution, "The default timer resolution in Hz."); | 75 | MODULE_PARM_DESC(seq_default_timer_resolution, "The default timer resolution in Hz."); |
75 | 76 | ||
77 | MODULE_ALIAS_CHARDEV(CONFIG_SND_MAJOR, SNDRV_MINOR_SEQUENCER); | ||
78 | MODULE_ALIAS("devname:snd/seq"); | ||
79 | |||
76 | /* | 80 | /* |
77 | * INIT PART | 81 | * INIT PART |
78 | */ | 82 | */ |
diff --git a/sound/core/sound.c b/sound/core/sound.c index 62a093efb453..345caea2d749 100644 --- a/sound/core/sound.c +++ b/sound/core/sound.c | |||
@@ -189,14 +189,22 @@ static const struct file_operations snd_fops = | |||
189 | }; | 189 | }; |
190 | 190 | ||
191 | #ifdef CONFIG_SND_DYNAMIC_MINORS | 191 | #ifdef CONFIG_SND_DYNAMIC_MINORS |
192 | static int snd_find_free_minor(void) | 192 | static int snd_find_free_minor(int type) |
193 | { | 193 | { |
194 | int minor; | 194 | int minor; |
195 | 195 | ||
196 | /* static minors for module auto loading */ | ||
197 | if (type == SNDRV_DEVICE_TYPE_SEQUENCER) | ||
198 | return SNDRV_MINOR_SEQUENCER; | ||
199 | if (type == SNDRV_DEVICE_TYPE_TIMER) | ||
200 | return SNDRV_MINOR_TIMER; | ||
201 | |||
196 | for (minor = 0; minor < ARRAY_SIZE(snd_minors); ++minor) { | 202 | for (minor = 0; minor < ARRAY_SIZE(snd_minors); ++minor) { |
197 | /* skip minors still used statically for autoloading devices */ | 203 | /* skip static minors still used for module auto loading */ |
198 | if (SNDRV_MINOR_DEVICE(minor) == SNDRV_MINOR_CONTROL || | 204 | if (SNDRV_MINOR_DEVICE(minor) == SNDRV_MINOR_CONTROL) |
199 | minor == SNDRV_MINOR_SEQUENCER) | 205 | continue; |
206 | if (minor == SNDRV_MINOR_SEQUENCER || | ||
207 | minor == SNDRV_MINOR_TIMER) | ||
200 | continue; | 208 | continue; |
201 | if (!snd_minors[minor]) | 209 | if (!snd_minors[minor]) |
202 | return minor; | 210 | return minor; |
@@ -270,7 +278,7 @@ int snd_register_device_for_dev(int type, struct snd_card *card, int dev, | |||
270 | preg->private_data = private_data; | 278 | preg->private_data = private_data; |
271 | mutex_lock(&sound_mutex); | 279 | mutex_lock(&sound_mutex); |
272 | #ifdef CONFIG_SND_DYNAMIC_MINORS | 280 | #ifdef CONFIG_SND_DYNAMIC_MINORS |
273 | minor = snd_find_free_minor(); | 281 | minor = snd_find_free_minor(type); |
274 | #else | 282 | #else |
275 | minor = snd_kernel_minor(type, card, dev); | 283 | minor = snd_kernel_minor(type, card, dev); |
276 | if (minor >= 0 && snd_minors[minor]) | 284 | if (minor >= 0 && snd_minors[minor]) |
diff --git a/sound/core/timer.c b/sound/core/timer.c index b3aaa600dcc2..ed016329e911 100644 --- a/sound/core/timer.c +++ b/sound/core/timer.c | |||
@@ -52,6 +52,9 @@ MODULE_PARM_DESC(timer_limit, "Maximum global timers in system."); | |||
52 | module_param(timer_tstamp_monotonic, int, 0444); | 52 | module_param(timer_tstamp_monotonic, int, 0444); |
53 | MODULE_PARM_DESC(timer_tstamp_monotonic, "Use posix monotonic clock source for timestamps (default)."); | 53 | MODULE_PARM_DESC(timer_tstamp_monotonic, "Use posix monotonic clock source for timestamps (default)."); |
54 | 54 | ||
55 | MODULE_ALIAS_CHARDEV(CONFIG_SND_MAJOR, SNDRV_MINOR_TIMER); | ||
56 | MODULE_ALIAS("devname:snd/timer"); | ||
57 | |||
55 | struct snd_timer_user { | 58 | struct snd_timer_user { |
56 | struct snd_timer_instance *timeri; | 59 | struct snd_timer_instance *timeri; |
57 | int tread; /* enhanced read with timestamps and events */ | 60 | int tread; /* enhanced read with timestamps and events */ |