From b7081601d1afd41d00de36af681ff9eae1c3de9c Mon Sep 17 00:00:00 2001 From: grev <grev> Date: Tue, 12 Feb 2008 22:10:19 +0000 Subject: [PATCH] b=14620 re-vert att. 14680 --- lustre/tests/test-framework.sh | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh index c6f7e02665..c657e323a6 100644 --- a/lustre/tests/test-framework.sh +++ b/lustre/tests/test-framework.sh @@ -150,10 +150,6 @@ init_test_env() { } -have_modules () { - lsmod | grep -q lnet -} - load_module() { EXT=".ko" module=$1 @@ -178,8 +174,11 @@ load_modules() { # use modprobe return 0 fi - # we already loaded ? - have_modules && return 0 + if [ "$HAVE_MODULES" = true ]; then + # we already loaded + return 0 + fi + HAVE_MODULES=true echo Loading modules from $LUSTRE load_module ../lnet/libcfs/libcfs @@ -282,6 +281,7 @@ unload_modules() { wait_for_lnet || return 3 fi fi + HAVE_MODULES=false LEAK_LUSTRE=$(dmesg | tail -n 30 | grep "obd mem.*leaked" || true) LEAK_PORTALS=$(dmesg | tail -n 20 | grep "Portals memory leaked" || true) @@ -1275,9 +1275,7 @@ equals_msg() { log() { echo "$*" - - local HAVE_MODULES="" - lsmod | grep lnet > /dev/null || { load_modules && HAVE_MODULES="yes"; } + lsmod | grep lnet > /dev/null || load_modules local MSG="$*" # Get rif of ' @@ -1292,8 +1290,6 @@ log() { for NODE in $NODES; do do_node $NODE $LCTL mark "$MSG" 2> /dev/null || true done - - [ -z "$HAVE_MODULES" ] || unload_modules } trace() { -- GitLab