From de1d63645679fa38b037a8fc03a87676f3ed3d98 Mon Sep 17 00:00:00 2001 From: Heng Guo Date: Thu, 12 May 2022 14:18:08 +0800 Subject: [PATCH] Tests: fix wrong sequence of unload modules Issue ===== Below modprobe FATAL is got in test_clock_override: ok 29 - Wait after kill session daemon modprobe: FATAL: Module lttng_clock_plugin_test is in use. ok 30 - Unique event timestamps with clock override: 1 expect 1 Cause ===== This issue was introduced by the following commit: commit d267104b87f12ea8f0bf73bff1af272f98dc069e Author: Francis Deslauriers Date: Tue Sep 15 12:10:18 2020 -0400 Cleanup: use `modprobe --remove` rather than `rmmod` Modprobe unload lttng-clock-plugin-test before lttng-test Solution ======== Change sequence to unload lttng-test module before lttng-clock-plugin-test Signed-off-by: Heng Guo --- tests/regression/kernel/test_clock_override | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/regression/kernel/test_clock_override b/tests/regression/kernel/test_clock_override index 8f471def8..0a44f5f28 100755 --- a/tests/regression/kernel/test_clock_override +++ b/tests/regression/kernel/test_clock_override @@ -149,7 +149,7 @@ function test_clock_override_timestamp() destroy_lttng_session_ok $SESSION_NAME stop_lttng_sessiond - modprobe --remove lttng-clock-plugin-test lttng-clock lttng-test + modprobe --remove lttng-test lttng-clock-plugin-test lttng-clock # Use Babeltrace with "-n all" to give a comma separated list for # easy extraction of timestamps. -- 2.35.2