]> git.ipfire.org Git - thirdparty/kmod.git/blobdiff - tools/modprobe.c
modprobe: Move -R to "Query options"
[thirdparty/kmod.git] / tools / modprobe.c
index 9387537f7bc2446ac6015136cea4fea57237fd31..caaf87f443dbafcede18186868d2154026f36f8f 100644 (file)
@@ -54,7 +54,7 @@ static int use_blacklist = 0;
 static int force = 0;
 static int strip_modversion = 0;
 static int strip_vermagic = 0;
-static int remove_dependencies = 0;
+static int remove_holders = 0;
 static int quiet_inuse = 0;
 
 static const char cmdopts_s[] = "arRibfDcnC:d:S:sqvVh";
@@ -62,6 +62,7 @@ static const struct option cmdopts[] = {
        {"all", no_argument, 0, 'a'},
        {"remove", no_argument, 0, 'r'},
        {"remove-dependencies", no_argument, 0, 5},
+       {"remove-holders", no_argument, 0, 5},
        {"resolve-alias", no_argument, 0, 'R'},
        {"first-time", no_argument, 0, 3},
        {"ignore-install", no_argument, 0, 'i'},
@@ -107,8 +108,8 @@ static void help(void)
                "\t                            be a module name to be inserted\n"
                "\t                            or removed (-r)\n"
                "\t-r, --remove                Remove modules instead of inserting\n"
-               "\t    --remove-dependencies   Also remove modules depending on it\n"
-               "\t-R, --resolve-alias         Only lookup and print alias and exit\n"
+               "\t    --remove-dependencies   Deprecated: use --remove-holders\n"
+               "\t    --remove-holders        Also remove module holders (use together with -r)\n"
                "\t    --first-time            Fail if module already inserted or removed\n"
                "\t-i, --ignore-install        Ignore install commands\n"
                "\t-i, --ignore-remove         Ignore remove commands\n"
@@ -120,6 +121,7 @@ static void help(void)
                "\t    --force-vermagic        Ignore module's version magic\n"
                "\n"
                "Query Options:\n"
+               "\t-R, --resolve-alias         Only lookup and print alias and exit\n"
                "\t-D, --show-depends          Only print module dependencies and exit\n"
                "\t-c, --showconfig            Print out known configuration and exit\n"
                "\t-c, --show-config           Same as --showconfig\n"
@@ -320,10 +322,9 @@ end:
 static int rmmod_do_remove_module(struct kmod_module *mod)
 {
        const char *modname = kmod_module_get_name(mod);
-       struct kmod_list *deps, *itr;
        int flags = 0, err;
 
-       SHOW("rmmod %s\n", kmod_module_get_name(mod));
+       SHOW("rmmod %s\n", modname);
 
        if (dry_run)
                return 0;
@@ -339,25 +340,15 @@ static int rmmod_do_remove_module(struct kmod_module *mod)
                        LOG("Module %s is not in kernel.\n", modname);
        }
 
-       deps = kmod_module_get_dependencies(mod);
-       if (deps != NULL) {
-               kmod_list_foreach(itr, deps) {
-                       struct kmod_module *dep = kmod_module_get_module(itr);
-                       if (kmod_module_get_refcnt(dep) == 0)
-                               rmmod_do_remove_module(dep);
-                       kmod_module_unref(dep);
-               }
-               kmod_module_unref_list(deps);
-       }
-
        return err;
 }
 
-#define RMMOD_FLAG_DO_DEPENDENCIES     0x1
+#define RMMOD_FLAG_REMOVE_HOLDERS      0x1
 #define RMMOD_FLAG_IGNORE_BUILTIN      0x2
 static int rmmod_do_module(struct kmod_module *mod, int flags);
 
-static int rmmod_do_deps_list(struct kmod_list *list, bool stop_on_errors)
+/* Remove modules in reverse order */
+static int rmmod_do_modlist(struct kmod_list *list, bool stop_on_errors)
 {
        struct kmod_list *l;
 
@@ -391,7 +382,8 @@ static int rmmod_do_module(struct kmod_module *mod, int flags)
                cmd = kmod_module_get_remove_commands(mod);
        }
 
-       if (cmd == NULL && !ignore_loaded) {
+       /* Quick check if module is loaded, otherwise there's nothing to do */
+       if (!cmd && !ignore_loaded) {
                int state = kmod_module_get_initstate(mod);
 
                if (state < 0) {
@@ -413,17 +405,20 @@ static int rmmod_do_module(struct kmod_module *mod, int flags)
                }
        }
 
-       rmmod_do_deps_list(post, false);
+       /* 1. @mod's post-softdeps in reverse order */
+       rmmod_do_modlist(post, false);
 
-       if ((flags & RMMOD_FLAG_DO_DEPENDENCIES) && remove_dependencies) {
-               struct kmod_list *deps = kmod_module_get_dependencies(mod);
+       /* 2. Other modules holding @mod */
+       if (flags & RMMOD_FLAG_REMOVE_HOLDERS) {
+               struct kmod_list *holders = kmod_module_get_holders(mod);
 
-               err = rmmod_do_deps_list(deps, true);
+               err = rmmod_do_modlist(holders, true);
                if (err < 0)
                        goto error;
        }
 
-       if (!ignore_loaded && !cmd) {
+       /* 3. @mod itself, but check for refcnt first */
+       if (!cmd && !ignore_loaded) {
                int usage = kmod_module_get_refcnt(mod);
 
                if (usage > 0) {
@@ -435,7 +430,7 @@ static int rmmod_do_module(struct kmod_module *mod, int flags)
                }
        }
 
-       if (cmd == NULL)
+       if (!cmd)
                err = rmmod_do_remove_module(mod);
        else
                err = command_do(mod, "remove", cmd, NULL);
@@ -443,7 +438,22 @@ static int rmmod_do_module(struct kmod_module *mod, int flags)
        if (err < 0)
                goto error;
 
-       rmmod_do_deps_list(pre, false);
+       /* 4. Other modules that became unused: errors are non-fatal */
+       if (!cmd) {
+               struct kmod_list *deps, *itr;
+
+               deps = kmod_module_get_dependencies(mod);
+               kmod_list_foreach(itr, deps) {
+                       struct kmod_module *dep = kmod_module_get_module(itr);
+                       if (kmod_module_get_refcnt(dep) == 0)
+                               rmmod_do_remove_module(dep);
+                       kmod_module_unref(dep);
+               }
+               kmod_module_unref_list(deps);
+       }
+
+       /* 5. @mod's pre-softdeps in reverse order: errors are non-fatal */
+       rmmod_do_modlist(pre, false);
 
 error:
        kmod_module_unref_list(pre);
@@ -468,7 +478,9 @@ static int rmmod(struct kmod_ctx *ctx, const char *alias)
 
        kmod_list_foreach(l, list) {
                struct kmod_module *mod = kmod_module_get_module(l);
-               err = rmmod_do_module(mod, RMMOD_FLAG_DO_DEPENDENCIES);
+               int flags = remove_holders ? RMMOD_FLAG_REMOVE_HOLDERS : 0;
+
+               err = rmmod_do_module(mod, flags);
                kmod_module_unref(mod);
                if (err < 0)
                        break;
@@ -683,7 +695,7 @@ static int options_from_array(char **args, int nargs, char **output)
 static char **prepend_options_from_env(int *p_argc, char **orig_argv)
 {
        const char *p, *env = getenv("MODPROBE_OPTIONS");
-       char **new_argv, *str_start, *str_end, *str, *s, *quote;
+       char **new_argv, *str_end, *str, *s, *quote;
        int i, argc = *p_argc;
        size_t envlen, space_count = 0;
 
@@ -701,10 +713,10 @@ static char **prepend_options_from_env(int *p_argc, char **orig_argv)
                return NULL;
 
        new_argv[0] = orig_argv[0];
-       str_start = str = (char *) (new_argv + argc + space_count + 3);
+       str = (char *) (new_argv + argc + space_count + 3);
        memcpy(str, env, envlen + 1);
 
-       str_end = str_start + envlen;
+       str_end = str + envlen;
 
        quote = NULL;
        for (i = 1, s = str; *s != '\0'; s++) {
@@ -743,7 +755,7 @@ static char **prepend_options_from_env(int *p_argc, char **orig_argv)
        }
 
        memcpy(new_argv + i, orig_argv + 1, sizeof(char *) * (argc - 1));
-       new_argv[i + argc] = NULL;
+       new_argv[i + argc - 1] = NULL;
        *p_argc = i + argc - 1;
 
        return new_argv;
@@ -786,10 +798,7 @@ static int do_modprobe(int argc, char **orig_argv)
                        do_remove = 1;
                        break;
                case 5:
-                       remove_dependencies = 1;
-                       break;
-               case 'R':
-                       lookup_only = 1;
+                       remove_holders = 1;
                        break;
                case 3:
                        first_time = 1;
@@ -814,6 +823,9 @@ static int do_modprobe(int argc, char **orig_argv)
                        dry_run = 1;
                        do_show = 1;
                        break;
+               case 'R':
+                       lookup_only = 1;
+                       break;
                case 'c':
                        do_show_config = 1;
                        break;