X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fmodprobe.c;h=8d162857f6ae88eeec547857d96baf055db99b63;hp=bc2a86d7512a3321f20c20b36f828aa36f1fc164;hb=1d25334cf088d661bac0643ccb50d1acc6d02e05;hpb=90aa04a116a8be510b9d256b47de6bbb4144fb8b

diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-sessiond/modprobe.c
index bc2a86d75..8d162857f 100644
--- a/src/bin/lttng-sessiond/modprobe.c
+++ b/src/bin/lttng-sessiond/modprobe.c
@@ -514,14 +514,23 @@ static void modprobe_remove_lttng(const struct kern_modules_param *modules,
 		modprobe[sizeof(modprobe) - 1] = '\0';
 		ret = system(modprobe);
 		if (ret == -1) {
-			ERR("Unable to launch modprobe -r for module %s",
-					modules[i].name);
-		} else if (modules[i].load_policy == KERNEL_MODULE_PROPERTY_LOAD_POLICY_REQUIRED && WEXITSTATUS(ret) != 0) {
-			ERR("Unable to remove module %s",
-					modules[i].name);
+			if (modules[i].load_policy == KERNEL_MODULE_PROPERTY_LOAD_POLICY_REQUIRED) {
+				ERR("Unable to launch modprobe -r for required module %s",
+						modules[i].name);
+			} else {
+				DBG("Unable to launch modprobe -r for optional module %s",
+						modules[i].name);
+			}
+		} else if (WEXITSTATUS(ret) != 0) {
+			if (modules[i].load_policy == KERNEL_MODULE_PROPERTY_LOAD_POLICY_REQUIRED) {
+				ERR("Unable to remove required module %s",
+						modules[i].name);
+			} else {
+				DBG("Unable to remove optional module %s",
+						modules[i].name);
+			}
 		} else {
-			DBG("Modprobe removal successful %s",
-					modules[i].name);
+			DBG("Modprobe removal successful %s", modules[i].name);
 		}
 	}
 }