diff options
author | Lennart Poettering <lennart@poettering.net> | 2013-05-07 17:48:25 +0200 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2013-05-07 17:52:23 +0200 |
commit | b4b879648c378e7cfaef940489c3c1ccc21b27e0 (patch) | |
tree | bc6166e9217fde5c22dce1325701043655799075 /src/core/kmod-setup.c | |
parent | systemd-sleep: it is not an error if the config file is missing (diff) | |
download | systemd-b4b879648c378e7cfaef940489c3c1ccc21b27e0.tar.xz systemd-b4b879648c378e7cfaef940489c3c1ccc21b27e0.zip |
kmod-setup: properly iterate through module table
Previously we skipped every second entry.
This also cleans up much of the code and removes some dead code.
Diffstat (limited to '')
-rw-r--r-- | src/core/kmod-setup.c | 72 |
1 files changed, 39 insertions, 33 deletions
diff --git a/src/core/kmod-setup.c b/src/core/kmod-setup.c index 20ab232374..e4885fb212 100644 --- a/src/core/kmod-setup.c +++ b/src/core/kmod-setup.c @@ -30,66 +30,72 @@ #include "kmod-setup.h" -typedef struct Kmodule { - const char *name; - const char *directory; - bool (*condition_fn)(void); -} KModule; - -static const KModule kmod_table[] = { - { "autofs4", "/sys/class/misc/autofs", NULL } , - { "ipv6", "/sys/module/ipv6", NULL }, - { "unix", "/proc/net/unix", NULL } , -}; - #pragma GCC diagnostic push #pragma GCC diagnostic ignored "-Wformat-nonliteral" -static void systemd_kmod_log(void *data, int priority, const char *file, int line, - const char *fn, const char *format, va_list args) -{ + +static void systemd_kmod_log( + void *data, + int priority, + const char *file, int line, + const char *fn, + const char *format, + va_list args) { + /* library logging is enabled at debug only */ log_metav(LOG_DEBUG, file, line, fn, format, args); } + #pragma GCC diagnostic pop int kmod_setup(void) { - unsigned i; + + static const char kmod_table[] = + /* This one we need to load explicitly, since + * auto-loading on use doesn't work before udev + * created the ghost device nodes, and we need it + * earlier than that. */ + "autofs4\0" "/sys/class/misc/autofs\0" + + /* This one we need to load explicitly, since + * auto-loading of IPv6 is not done when we try to + * configure ::1 on the loopback device. */ + "ipv6\0" "/sys/module/ipv6\0" + + "unix\0" "/proc/net/unix\0"; + struct kmod_ctx *ctx = NULL; - struct kmod_module *mod; - int err; + const char *name, *path; + int r; - for (i = 0; i < ELEMENTSOF(kmod_table); i += 2) { - if (kmod_table[i].condition_fn && !kmod_table[i].condition_fn()) - continue; + NULSTR_FOREACH_PAIR(name, path, kmod_table) { + struct kmod_module *mod; - if (access(kmod_table[i].directory, F_OK) >= 0) + if (access(path, F_OK) >= 0) continue; log_debug("Your kernel apparently lacks built-in %s support. Might be a good idea to compile it in. " "We'll now try to work around this by loading the module...", - kmod_table[i].name); + name); if (!ctx) { ctx = kmod_new(NULL, NULL); - if (!ctx) { - log_error("Failed to allocate memory for kmod"); - return -ENOMEM; - } + if (!ctx) + return log_oom(); kmod_set_log_fn(ctx, systemd_kmod_log, NULL); kmod_load_resources(ctx); } - err = kmod_module_new_from_name(ctx, kmod_table[i].name, &mod); - if (err < 0) { - log_error("Failed to lookup module '%s'", kmod_table[i].name); + r = kmod_module_new_from_name(ctx, name, &mod); + if (r < 0) { + log_error("Failed to lookup module '%s'", name); continue; } - err = kmod_module_probe_insert_module(mod, KMOD_PROBE_APPLY_BLACKLIST, NULL, NULL, NULL, NULL); - if (err == 0) + r = kmod_module_probe_insert_module(mod, KMOD_PROBE_APPLY_BLACKLIST, NULL, NULL, NULL, NULL); + if (r == 0) log_info("Inserted module '%s'", kmod_module_get_name(mod)); - else if (err == KMOD_PROBE_APPLY_BLACKLIST) + else if (r == KMOD_PROBE_APPLY_BLACKLIST) log_info("Module '%s' is blacklisted", kmod_module_get_name(mod)); else log_error("Failed to insert module '%s'", kmod_module_get_name(mod)); |