[lttng-dev] [RFC PATCH lttng-tools 1/5] Drop optionnal control modules load and unload

[Michael Jeanson]

The optionnal control modules are loaded and unloaded trought modules
dependencies by modprobe or libkmod. Trying to load them manually leads
to spurious log messages and slows down the sessiond startup.

Signed-off-by: Michael Jeanson <mjeanson at efficios.com>
---
 src/bin/lttng-sessiond/modprobe.c | 18 ------------------
 1 file changed, 18 deletions(-)

diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-sessiond/modprobe.c
index 2c527dc..81ce810 100644
--- a/src/bin/lttng-sessiond/modprobe.c
+++ b/src/bin/lttng-sessiond/modprobe.c
@@ -33,8 +33,6 @@
 
 /* LTTng kernel tracer mandatory core modules list */
 struct kern_modules_param kern_modules_control_core[] = {
-	{ "lttng-tracer" },	/* MUST be loaded first so keep at top */
-	{ "lttng-lib-ring-buffer" },
 	{ "lttng-ring-buffer-client-discard" },
 	{ "lttng-ring-buffer-client-overwrite" },
 	{ "lttng-ring-buffer-metadata-client" },
@@ -43,14 +41,6 @@ struct kern_modules_param kern_modules_control_core[] = {
 	{ "lttng-ring-buffer-metadata-mmap-client" },
 };
 
-/* LTTng kernel tracer optional base modules list */
-struct kern_modules_param kern_modules_control_opt[] = {
-	{ "lttng-types" },
-	{ "lttng-ftrace" },
-	{ "lttng-kprobes" },
-	{ "lttng-kretprobes" },
-};
-
 /* LTTng kernel tracer probe modules list */
 struct kern_modules_param kern_modules_probes_default[] = {
 	{ "lttng-probe-asoc" },
@@ -134,9 +124,6 @@ static void modprobe_remove_lttng(const struct kern_modules_param *modules,
  */
 void modprobe_remove_lttng_control(void)
 {
-	modprobe_remove_lttng(kern_modules_control_opt,
-			      ARRAY_SIZE(kern_modules_control_opt),
-			      LTTNG_MOD_OPTIONAL);
 	modprobe_remove_lttng(kern_modules_control_core,
 			      ARRAY_SIZE(kern_modules_control_core),
 			      LTTNG_MOD_REQUIRED);
@@ -303,11 +290,6 @@ int modprobe_lttng_control(void)
 	ret = modprobe_lttng(kern_modules_control_core,
 			     ARRAY_SIZE(kern_modules_control_core),
 			     LTTNG_MOD_REQUIRED);
-	if (ret != 0)
-		return ret;
-	ret = modprobe_lttng(kern_modules_control_opt,
-			     ARRAY_SIZE(kern_modules_control_opt),
-			     LTTNG_MOD_OPTIONAL);
 	return ret;
 }
 
-- 
2.7.0