X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Ftests%2Flnet-selftest.sh;h=0589a953eef7321791ac2b3a0a2b943e83520d29;hb=e240fb5099af8e62c532d314317095800ebb6864;hp=06d3b571fc19eae928e986d6119e255e3157612f;hpb=8c4f96f910786ff3d73474ef5f8d4a96a30a0bed;p=fs%2Flustre-release.git

diff --git a/lustre/tests/lnet-selftest.sh b/lustre/tests/lnet-selftest.sh
index 06d3b57..0589a95 100755
--- a/lustre/tests/lnet-selftest.sh
+++ b/lustre/tests/lnet-selftest.sh
@@ -1,14 +1,17 @@
-#!/bin/sh
+#!/bin/bash
+# -*- mode: Bash; tab-width: 4; indent-tabs-mode: t; -*-
+# vim:shiftwidth=4:softtabstop=4:tabstop=4:
 
 LUSTRE=${LUSTRE:-$(cd $(dirname $0)/..; echo $PWD)}
 . $LUSTRE/tests/test-framework.sh
 init_test_env $@
 . ${CONFIG:=$LUSTRE/tests/cfg/$NAME.sh}
+init_logging
 
 #
 ALWAYS_EXCEPT="$ALWAYS_EXCEPT $LNET_SELFTEST_EXCEPT"
 
-[ x$LST = x ] &&  { skip_env "$0 : lst not found LST=$LST" && exit 0; }
+[ x$LST = x ] && { skip_env "lst not found LST=$LST" && exit 0; }
 
 # FIXME: what is the reasonable value here?
 lst_LOOP=${lst_LOOP:-100000}
@@ -25,8 +28,28 @@ if [ "$SLOW" = no ]; then
     [ $smoke_DURATION -le 300 ] || smoke_DURATION=300
 fi
 
-lst_SERVERS=${lst_SERVERS:-$(comma_list $(osts_nodes) $(mdts_nodes))}
-lst_CLIENTS=${lst_CLIENTS:-${CLIENTS:-`hostname`}}
+nodes=$(comma_list "$(osts_nodes) $(mdts_nodes)")
+lst_SERVERS=${lst_SERVERS:-$(comma_list "$(host_nids_address $nodes $NETTYPE)")}
+lst_CLIENTS=${lst_CLIENTS:-$(comma_list "$(host_nids_address $CLIENTS $NETTYPE)")}
+interim_umount=false
+interim_umount1=false
+
+#
+# _restore_mount(): This function calls restore_mount function for "MOUNT" and
+# "MOUNT2" paths to mount clients if they were not mounted and were umounted
+# in this file earlier.
+# Parameter: None
+# Returns: None. Exit with error if client mount fails.
+#
+_restore_mount () {
+	if $interim_umount && ! is_mounted $MOUNT; then
+		restore_mount $MOUNT || error "Restore $MOUNT failed"
+	fi
+
+	if $interim_umount1 && ! is_mounted $MOUNT2; then
+		restore_mount $MOUNT2 || error "Restore $MOUNT2 failed"
+	fi
+}
 
 is_mounted () {
     local mntpt=$1
@@ -46,10 +69,14 @@ fi
 # 1) because lustre messages clutter logs - we needn't them for testing LNET
 # 2) it's theoretically possible that lst tests congest comm paths so tightly
 # that mounted lustre wouldn't able to perform some of its background activities
-if is_mounted $MOUNT || is_mounted $MOUNT2; then
-    local_mode && CLIENTONLY=yes
-    stopall
-    RESTORE_MOUNT=yes
+if is_mounted $MOUNT; then
+	cleanup_mount $MOUNT || error "Fail to unmount client $MOUNT"
+	interim_umount=true
+fi
+
+if is_mounted $MOUNT2; then
+	cleanup_mount $MOUNT2 || error "Fail to unmount client $MOUNT2"
+	interim_umount1=true
 fi
 
 build_test_filter
@@ -117,44 +144,43 @@ run_lst () {
 }
 
 check_lst_err () {
-   local log=$1
+	local log=$1
 
-   grep ^Total $log
+	grep ^Total $log
 
-   if awk '/^Total.*nodes/ {print $2}' $log | grep -vq '^0$'; then
-        error 'lst Error found'
-   fi
+	if awk '/^Total.*nodes/ {print $2}' $log | grep -vq '^0$'; then
+		_restore_mount
+		error 'lst Error found'
+	fi
 }
 
 test_smoke () {
-    lst_prepare
+	lst_prepare
 
-    local servers=$lst_SERVERS
-    local clients=$lst_CLIENTS
+	local servers=$lst_SERVERS
+	local clients=$lst_CLIENTS
 
-    local runlst=$TMP/smoke.sh
+	local runlst=$TMP/smoke.sh
 
-    local log=$TMP/$tfile.log
-    local rc=0
+	local log=$TMP/$tfile.log
+	local rc=0
 
-    test_smoke_sub $servers $clients 2>&1 > $runlst 
+	test_smoke_sub $servers $clients 2>&1 > $runlst
 
-    cat $runlst
+	cat $runlst
 
-    run_lst $runlst | tee $log
-    rc=${PIPESTATUS[0]}
-    [ $rc = 0 ] || error "$runlst failed: $rc"
-    
-    lst_end_session --verbose | tee -a $log
+	run_lst $runlst | tee $log
+	rc=${PIPESTATUS[0]}
+	[ $rc = 0 ] || { _restore_mount; error "$runlst failed: $rc"; }
+
+	lst_end_session --verbose | tee -a $log
 
-    # error counters in "lst show_error" should be checked
-    check_lst_err $log
-    lst_cleanup_all    
+	# error counters in "lst show_error" should be checked
+	check_lst_err $log
+	lst_cleanup_all
 }
 run_test smoke "lst regression test"
 
-complete $(basename $0) $SECONDS
-if [ "$RESTORE_MOUNT" = yes ]; then
-    setupall
-fi 
+complete $SECONDS
+_restore_mount
 exit_status