kmodloader: modprobe: return 0 for loaded modules
authorYousong Zhou <yszhou4tech@gmail.com>
Fri, 3 Mar 2017 02:32:48 +0000 (10:32 +0800)
committerFelix Fietkau <nbd@nbd.name>
Fri, 3 Mar 2017 10:25:23 +0000 (11:25 +0100)
This is the default behaviour with modprobe from kmod package [1] unless
it is explicitly told that the module is to be loaded for --first-time

 [1] http://www.kernel.org/pub/linux/utils/kernel/kmod/

Fixes FS#433

Signed-off-by: Yousong Zhou <yszhou4tech@gmail.com>
kmodloader.c

index 892ddd8..ed8f833 100644 (file)
@@ -837,7 +837,7 @@ static int main_modprobe(int argc, char **argv)
        if (m && m->state == LOADED) {
                if (!quiet)
                        ULOG_ERR("%s is already loaded\n", name);
        if (m && m->state == LOADED) {
                if (!quiet)
                        ULOG_ERR("%s is already loaded\n", name);
-               return -1;
+               return 0;
        } else if (!m) {
                if (!quiet)
                        ULOG_ERR("failed to find a module named %s\n", name);
        } else if (!m) {
                if (!quiet)
                        ULOG_ERR("failed to find a module named %s\n", name);