Whamcloud - gitweb
add vmstat running in parallel
[fs/lustre-release.git] / lustre-iokit / obdfilter-survey / obdfilter-survey
index f700a88..ac967de 100755 (executable)
 ######################################################################
 # customize per survey
 
-# specify either the obdecho client names or the obdfilter names
-client_names=()
-ost_names=(ost{1,2,3,4,5,6,7,8})
-
-# result file prefix
-rslt=/tmp/obdfilter_survey
+# specify obd instances to exercise
+# these can be either...
+# obdfilter instances (set 'ost_names')
+# ...or...
+# echo_client instances (set 'client_names')
+# ... use 'host:name' for obd instances on other nodes.
+
+# allow these to be passed in via string...
+ost_names_str=${ost_names_str:-""}
+if [ -n "$ost_names_str" ]; then
+    declare -a ost_names
+    count=0
+    for name in $ost_names_str; do
+        ost_names[$count]=$name
+        count=$((count+1))
+    done
+else
+    ost_names=(ost{1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16})
+fi
 
-# lustre root (leave blank unless running with own source)
-lustre_root=
+#client_names=(ns8:ECHO_ns8 ns9:ECHO_ns9)
+client_names_str=${client_names_str:-""}
+if [ -n "$client_names_str" ]; then
+    # make sure we unset ost_names so that our client_names get noticed... 
+    unset ost_names
+    declare -a client_names
+    count=0
+    for name in $client_names_str; do
+        client_names[$count]=$name
+        count=$((count+1))
+    done
+fi
 
-# what to do (we always do an initial write)
-#tests="rewrite read reread rewrite_again"
-tests="rewrite read"
+# result file prefix (date/time + hostname makes unique)
+# NB ensure path to it exists
+rslt=${rslt:-"/home_nfs/eeb/obdfilter_survey_`date +%F@%R`_`uname -n`"}
+
+# lustre root (if running with own source tree)
+lustre_root=${lustre_root:-"/home_nfs/eeb/lustre"}
+
+# what tests to run (first must be write)
+tests_str=${tests_str:-""}
+if [ -n "$tests_str" ]; then
+    declare -a tests
+    count=0
+    for name in $tests_str; do
+        tests[$count]=$name
+        count=$((count+1))
+    done
+else
+    #tests=(write rewrite read reread rewrite_again)
+    tests=(write rewrite read)
+fi
 
-# total size (MBytes)
-# large enough to avoid cache effects
-size=8192
+# total size (MBytes) per obd instance
+# large enough to avoid cache effects 
+# and to make test startup/shutdown overhead insignificant
+size=${size:-16384}
 
 # record size (KBytes)
-rszlo=1024
-rszhi=1024
+rszlo=${rszlo:-1024}
+rszhi=${rszhi:-1024}
 
 # number of objects per OST
-nobjlo=1
-nobjhi=32
+nobjlo=${nobjlo:-1}
+nobjhi=${nobjhi:-512}
 
 # threads per OST (1024 max)
-thrlo=1
-thrhi=128
+thrlo=${thrlo:-1}
+thrhi=${thrhi:-64}
 
 # restart from here iff all are defined
 restart_rsz=
 restart_thr=1
 restart_nobj=1
 
-# machine's page size
-PAGE_SIZE=64
+# machine's page size (K)
+PAGE_SIZE=${PAGE_SIZE:-16}
 
 # max buffer_mem (total_threads * buffer size)
 # (to avoid lctl ENOMEM problems)
-max_buffer_mem=$((256*1024))
+max_buffer_mem=$((1024*1024))
+
+# how to run commands on other nodes
+# You need to make this work on your cluster if you have specified
+# non-local obd instances above
+custom_remote_shell () {
+    host=$1
+    shift
+    cmds="$*"
+    here=`pwd`
+    # Hop on to the remote node, chdir to 'here' and run the given
+    # commands. One of the following will probably work.
+    #ssh $host "cd $here; $cmds"
+    #rsh $host "cd $here; $cmds"
+    # we have to remove the leading `uname -n`: from pdsh output lines
+    pdsh -w $host "cd $here; $cmds" | sed 's/^[^:]*://'
+}
 
 #####################################################################
+# leave the rest of this alone unless you know what you're doing...
 
+# binaries
+lsmod="/sbin/lsmod"
+modprobe="/sbin/modprobe"
+insmod="/sbin/insmod"
+rmmod="/sbin/rmmod"
+
+# lctl::test_brw bandwidth snapshot interval (seconds)
 snap=1
+# check file contents?
 verify=1
 
-check_obdecho() {
-    lsmod | grep obdecho > /dev/null 2>&1
-}
+if [ ${#tests[@]} -eq 0 -o "${tests[0]}" != "write" ]; then
+    echo "tests: ${tests[@]}"
+    echo "First test must be 'write'" 1>&2
+    exit 1
+fi
+
+rsltf="${rslt}.summary"
+workf="${rslt}.detail"
+cmdsf="${rslt}.script"
+vmstatf="${rslt}.vmstat"
+echo -n > $rsltf
+echo -n > $workf
+
+declare -a vmstatpids
 
-check_obdecho
-load_obdecho=$(($? != 0))
+# hide a little trick to unset this from the command line
+if [ "$lustre_root" == " " ]; then
+    unset lustre_root
+fi
 
 if [ -z "$lustre_root" ]; then
-    lctl=lctl
-    if ((load_obdecho)); then
-       modprobe obdecho
-    fi
+    lctl=$(which lctl)
 else
-    lctl=${lustre_root}/lctl
-    if ((load_obdecho)); then
-       if [ -f ${lustre_root}/obdecho/obdecho.ko ]; then
-           insmod ${lustre_root}/obdecho/obdecho.ko
-       else
-           insmod ${lustre_root}/obdecho/obdecho.o
-       fi
-    fi
+    lctl=${lustre_root}/utils/lctl
 fi
 
-check_obdecho || (echo "Can't load obdecho"; exit 1)
+remote_shell () {
+    host=$1
+    shift
+    cmds="$*"
+    if [ "$host" = "localhost" -o "$host" = `uname -n` ]; then
+        eval "$cmds"
+    else
+        custom_remote_shell $host "$cmds"
+    fi
+}
+
+obdecho_loaded() {
+    local host=$1
+    remote_shell $host $lsmod | grep obdecho > /dev/null 2>&1
+}
+
+load_obdecho () {
+    local host=$1
+    if [ -z "$lustre_root" ]; then
+        remote_shell $host $modprobe obdecho
+    elif [ -f ${lustre_root}/obdecho/obdecho.ko ]; then
+        remote_shell $host $insmod ${lustre_root}/obdecho/obdecho.ko
+    else
+        remote_shell $host $insmod ${lustre_root}/obdecho/obdecho.o
+    fi
+}
+
+unload_obdecho () {
+    local host=$1
+    remote_shell $host $rmmod obdecho
+}
 
 get_devno () {
-    local type=$1
-    local name=$2
-    $lctl device_list | awk "{if (\$2 == \"UP\" && \$3 == \"$type\" && \$4 == \"$name\") {\
-                                print \$1; exit}}"
+    local host=$1
+    local type=$2
+    local name=$3
+    remote_shell $host $lctl device_list | \
+        awk "{if (\$2 == \"UP\" && \$3 == \"$type\" && \$4 == \"$name\") {\
+                  print \$1; exit}}"
 }
 
 get_ec_devno () {
-    local idx=$1
-    local client_name=${client_names[idx]}
-    local ost_name=${ost_names[idx]}
+    local host=$1
+    local client_name="$2"
+    local ost_name="$3"
     if [ -z "$client_name" ]; then
-       if [ -z "$ost_name" ]; then
-           echo "client and ost name both null" 1>&2
-           return
-       fi
-       client_name=${ost_name}_echo_client
+        if [ -z "$ost_name" ]; then
+            echo "client and ost name both null" 1>&2
+            return
+        fi
+        client_name=${ost_name}_echo_client
     fi
-    ec=`get_devno echo_client $client_name`
+    ec=`get_devno $host echo_client $client_name`
     if [ -n "$ec" ]; then
-       echo $ec $client_name 
-       return
+        echo $ec $client_name 0
+        return
     fi
     if [ -z "$ost_name" ]; then
-       echo "no echo client and ost_name not set" 1>&2
-       return
+        echo "no echo client and ost_name not set" 1>&2
+        return
     fi
-    ost=`get_devno obdfilter $ost_name`
+    ost=`get_devno $host obdfilter $ost_name`
     if [ -z "$ost" ]; then
-       echo "OST $ost_name not setup" 1>&2
-       return
+        echo "OST $ost_name not setup" 1>&2
+        return
     fi
-    $lctl <<EOF
+    remote_shell $host "$lctl <<EOF
         attach echo_client $client_name ${client_name}_UUID
-       setup $ost_name
-EOF
-    ec=`get_devno echo_client $client_name`
+setup $ost_name
+EOF"
+    ec=`get_devno $host echo_client $client_name`
     if [ -z "$ec" ]; then
-       echo "Can't setup echo client" 1>&2
-       return
+        echo "Can't setup echo client" 1>&2
+        return
     fi
     echo $ec $client_name 1
 }
 
 teardown_ec_devno () {
-    local idx=$1
-    local client_name=${client_names[$idx]}
-    if ((do_teardown_ec[$idx])); then
-       $lctl <<EOF
-       cfg $client_name
-       cleanup
-       detach
-EOF
-   fi
+    local host=$1
+    local client_name=$2
+    remote_shell $host "$lctl <<EOF
+cfg $client_name
+cleanup
+detach
+EOF"
 }
 
 create_objects () {
     # create a set of objects, check there are 'n' contiguous ones and
     # return the first or 'ERROR'
-    local devno=$1
-    local nobj=$2
-    local rfile=$3
-    $lctl --device $devno create $nobj > $rfile 2>&1
-    n=(`awk < $rfile \
-       '/is object id/ {obj=strtonum($6);\
-                        first=!not_first; not_first=1;\
-                        if (first) first_obj=obj;
-                        else if (obj != prev + 1) exit;\
-                        prev=obj; n++}\
-            END {printf "%d %d\n", first_obj, n}'`)
-    if ((n[1] != nobj)); then
-       echo "ERROR"
-    else
-       echo ${n[0]}
+    local host=$1
+    local devno=$2
+    local nobj=$3
+    local rfile=$4
+    remote_shell $host $lctl --device $devno create $nobj > $rfile 2>&1
+    first=0
+    prev=0
+    count=0
+    error=0
+    while read line; do
+        echo "$line" | grep -q 'is object id'
+        if [ $?  -ne 0 ]; then
+            continue
+        fi
+        if [ $first -eq 0 ]; then
+            first=$(echo $line | awk '{print $6}')
+            first=$(printf "%d" $first)
+            prev=$first
+            count=1
+        else
+            obj=$(echo $line | awk '{print $6}') 
+            obj=$(printf "%d" $obj) 
+            diff=$((obj - (prev+1))) 
+            if [ $diff -ne 0 ]; then 
+                error=1 
+            fi 
+            prev=$obj 
+            count=$((count+1)) 
+        fi 
+    done < $rfile 
+    if [ $nobj -ne $count ]; then 
+        echo "ERROR: $nobj != $count" >&2 
+        cat $rfile >&2 
+        echo "ERROR" 
+    elif [ $error -ne 0 ]; then 
+        echo "ERROR: non contiguous objs found" >&2 
+        echo "ERROR" 
+    else 
+        echo $first 
     fi
 }
 
 destroy_objects () {
-    local devno=$1
-    local obj0=$2
-    local nobj=$3
-    local rfile=$4
-    $lctl --device $devno destroy $obj0 $nobj > $rfile 2>&1
+    local host=$1
+    local devno=$2
+    local obj0=$3
+    local nobj=$4
+    local rfile=$5
+    remote_shell $host $lctl --device $devno destroy $obj0 $nobj > $rfile 2>&1
 }
 
 get_stats () {
@@ -187,22 +311,16 @@ get_global_stats () {
 testname2type () {
     # 'x' disables data check
     if ((verify)); then
-       x=""
+        x=""
     else
-       x="x"
+        x="x"
     fi
     case $1 in
-       *write*)  echo "w$x";;
-       *)        echo "r$x";;
+        *write*)  echo "w$x";;
+        *)        echo "r$x";;
     esac
 }
 
-start=`date +%F@%R`
-rsltf="${rslt}_${start}.summary"
-echo -n > $rsltf
-workf="${rslt}_${start}.detail"
-echo -n > $workf
-
 print_summary () {
     if [ "$1" = "-n" ]; then
        minusn=$1; shift
@@ -213,130 +331,250 @@ print_summary () {
     echo $minusn "$*"
 }
 
+unique () {
+    echo "$@" | xargs -n1 echo | sort -u
+}
+
+split_hostname () {
+    local name=$1
+    case $name in
+    *:*) host=`echo $name | sed 's/:.*$//'`
+        name=`echo $name | sed 's/[^:]*://'`
+        ;;
+    *)   host=localhost
+        ;;
+    esac
+    echo "$host $name"
+}
+
+# split out hostnames from client/ost names
 ndevs=${#client_names[@]}
-if ((ndevs < ${#ost_names[@]} )); then
+if ((ndevs != 0)); then
+    if ((${#ost_names[@]} != 0)); then
+        echo "Please specify client_names or ost_names, but not both" 1>&2
+       exit 1
+    fi
+    for ((i=0; i<ndevs;i++)); do
+       str=(`split_hostname ${client_names[$i]}`)
+       host_names[$i]=${str[0]}
+       client_names[$i]=${str[1]}
+    done
+else
     ndevs=${#ost_names[@]}
+    if ((ndevs == 0)); then
+        echo "Please specify either client_names or ost_names" 1>&2
+       exit 1
+    fi
+    for ((i=0; i<ndevs;i++)); do
+       str=(`split_hostname ${ost_names[$i]}`)
+       host_names[$i]=${str[0]}
+       ost_names[$i]=${str[1]}
+    done
 fi
 
-for ((idx = 0; idx < ndevs; idx++)); do
-    devno=(`get_ec_devno $idx`)
-    if ((${#devno[@]} < 2)); then
-       exit 1
+# get vmstat started
+# disable portals debug and get obdecho loaded on all relevant hosts
+unique_hosts=(`unique ${host_names[@]}`)
+pidcount=0
+for host in ${unique_hosts[@]}; do
+    remote_shell $host "echo 0 > /proc/sys/portals/debug"
+    host_vmstatf=${vmstatf}_${host}
+    echo -n > $host_vmstatf
+    remote_shell $host "vmstat 5 >> $host_vmstatf" &
+    pid=$!
+    vmstatpids[$pidcount]=$pid
+    pidcount=$((pidcount+1))
+    do_unload_obdecho[$host]=0
+    if obdecho_loaded $host; then
+        continue
     fi
-    devnos[$idx]=${devno[0]}
-    client_names[$idx]=${devno[1]}
-    do_teardown_ec[$idx]=$((${#devno[@]} > 2))
+    load_obdecho $host
+    if obdecho_loaded $host; then
+        do_unload_obdecho[$host]=1
+        continue
+        fi
+    echo "Can't load obdecho on $host" 1>&2
+    exit 1
 done
 
-echo 0 > /proc/sys/portals/debug
+# get all the echo_client device numbers and names
+for ((i=0; i<ndevs; i++)); do
+    host=${host_names[$i]}
+    devno=(`get_ec_devno $host "${client_names[$i]}" "${ost_names[$i]}"`)
+    if ((${#devno[@]} != 3)); then
+        exit 1
+    fi
+    devnos[$i]=${devno[0]}
+    client_names[$i]=${devno[1]}
+    do_teardown_ec[$i]=${devno[2]}
+done
 
 for ((rsz=$rszlo;rsz<=$rszhi;rsz*=2)); do
     for ((nobj=$nobjlo;nobj<=$nobjhi;nobj*=2)); do 
-       for ((thr=$thrlo;thr<=$thrhi;thr*=2)); do
-           if ((thr < nobj)); then
-               continue
-           fi
-           # restart?
-           if [ -n "$restart_rsz" -a\
-                -n "$restart_nobj" -a\
-                -n "$restart_thr" ]; then
-               if ((rsz < restart_rsz ||\
-                    (rsz == restart_rsz &&\
-                     (nobj < restart_nobj ||\
-                      (nobj == restart_nobj &&\
-                       thr < restart_thr))))); then
-                   continue;
-               fi
-           fi
-           # compute parameters
-           total_thr=$((ndevs*thr))
-           total_nobj=$((ndevs*nobj))
-           pages=$((rsz/PAGE_SIZE))
-           actual_rsz=$((pages*PAGE_SIZE))
-           count=$((size*1024/(actual_rsz*thr)))
-           actual_size=$((actual_rsz*count*thr))
-            total_size=$((actual_size*ndevs))
-           # show computed parameters
-           str=`printf 'ost %2d sz %8dK rsz %4d obj %4d thr %4d ' \
-                    $ndevs $total_size $actual_rsz $total_nobj $total_thr`
-           echo "=======================> $str" >> $workf
-           print_summary -n "$str"
-           if ((total_thr * actual_rsz > max_buffer_mem)); then
-               print_summary "Too much buffer space"
-               continue
-           fi
-           # create the objects
-           tmpf="${workf}_tmp"
-           for ((idx=0; idx < ndevs; idx++)); do
-               devno=${devnos[$idx]}
-               first_obj=`create_objects $devno $nobj $tmpf`
-               echo "========> Create [$idx]" >> $workf
-               cat $tmpf >> $workf
-               rm $tmpf
-               if [ $first_obj = "ERROR" ]; then
-                   print_summary "created object #s [$idx] not contiguous"
-                   exit 1
-               fi
-               first_objs[$idx]=$first_obj
-           done
-           for test in write $tests; do
-               print_summary -n "$test "
-               t0=`date +%s.%N`
-               for ((idx=0; idx < ndevs; idx++)); do
-                   devno=${devnos[$idx]}
-                   tmpfi="${tmpf}_$idx"
-                   first_obj=${first_objs[$idx]}
-                   $lctl > $tmpfi 2>&1 \
-                       --threads $thr -$snap $devno \
-                       test_brw $count `testname2type $test` q $pages ${thr}t${first_obj} &
-               done
-               wait
-               t1=`date +%s.%N`
-               str=`awk "BEGIN {printf \"%7.2f \",\
-                        $total_size / (( $t1 - $t0 ) * 1024)}"`
-               print_summary -n "$str"
-               echo -n > $tmpf
-               for ((idx=0; idx < ndevs; idx++)); do
-                   tmpfi="${tmpf}_$idx"
-                   echo "========> $test [$idx]" >> $workf
-                   cat $tmpfi >> $workf
-                   get_stats $tmpfi >> $tmpf
-                   rm $tmpfi
-               done
-               echo "========> $test [$idx] global" >> $workf
-               cat $tmpf >> $workf
-               stats=(`get_global_stats $tmpf`)
-               rm $tmpf
-               if ((stats[0] <= 0)); then
-                   if ((stats[0] < 0)); then
-                       str=`printf "%15s " ERROR`
-                   else
-                       str=`printf "%15s " SHORT`
-                   fi
-               else
-                   str=`awk "BEGIN {printf \"[%6.2f,%6.2f] \",\
-                            (${stats[1]} * $actual_rsz)/1024,\
-                            (${stats[2]} * $actual_rsz)/1024; exit}"`
-               fi
-               print_summary -n "$str"
-           done
-           print_summary ""
-           for ((idx=0; idx < ndevs; idx++)); do
-               devno=${devnos[$idx]}
-               first_obj=${first_objs[$idx]}
-               destroy_objects $devno $first_obj $nobj $tmpf
-               echo "========> Destroy [$idx]" >> $workf
-               cat $tmpf >> $workf
-               rm $tmpf
-           done
-       done
+        for ((thr=$thrlo;thr<=$thrhi;thr*=2)); do
+            if ((thr < nobj)); then
+                continue
+            fi
+            # restart?
+            if [ -n "$restart_rsz" -a\
+             -n "$restart_nobj" -a\
+             -n "$restart_thr" ]; then
+                if ((rsz < restart_rsz ||\
+                     (rsz == restart_rsz &&\
+                      (nobj < restart_nobj ||\
+                       (nobj == restart_nobj &&\
+                    thr < restart_thr))))); then
+                    continue;
+                fi
+            fi
+            # compute parameters
+            total_thr=$((ndevs*thr))
+            total_nobj=$((ndevs*nobj))
+            pages=$((rsz/PAGE_SIZE))
+            actual_rsz=$((pages*PAGE_SIZE))
+            count=$((size*1024/(actual_rsz*thr)))
+            actual_size=$((actual_rsz*count*thr))
+                total_size=$((actual_size*ndevs))
+            # show computed parameters
+            str=`printf 'ost %2d sz %8dK rsz %4d obj %4d thr %4d ' \
+                 $ndevs $total_size $actual_rsz $total_nobj $total_thr`
+            echo "=======================> $str" >> $workf
+            print_summary -n "$str"
+            if ((total_thr * actual_rsz > max_buffer_mem)); then
+                print_summary "Too much buffer space"
+                continue
+            fi
+            # create the objects
+            tmpf="${workf}_tmp"
+            for ((idx=0; idx < ndevs; idx++)); do
+                host=${host_names[$idx]}
+                devno=${devnos[$idx]}
+                client_name="${host}:${client_names[$idx]}"
+                echo "=============> Create $nobj on $client_name" >> $workf
+                first_obj=`create_objects $host $devno $nobj $tmpf`
+                cat $tmpf >> $workf
+                rm $tmpf
+                if [ $first_obj = "ERROR" ]; then
+                    print_summary "created object #s on $client_name not contiguous"
+                    exit 1
+                fi
+                first_objs[$idx]=$first_obj
+            done
+            # run tests
+            for test in ${tests[@]}; do
+                declare -a pidarray
+                for host in ${unique_hosts[@]}; do
+                    echo "starting run for test: $test rsz: $rsz threads: $thr objects: $nobj" >> ${vmstatf}_${host}
+                done
+                print_summary -n "$test "
+                # create per-host script files
+                for host in ${unique_hosts[@]}; do
+                    echo -n > ${cmdsf}_${host}
+                done
+                for ((idx=0; idx < ndevs; idx++)); do
+                    host=${host_names[$idx]}
+                    devno=${devnos[$idx]}
+                    tmpfi="${tmpf}_$idx"
+                    first_obj=${first_objs[$idx]}
+                    echo >> ${cmdsf}_${host} \
+                        "$lctl > $tmpfi 2>&1 \\
+                                 --threads $thr -$snap $devno \\
+                     test_brw $count `testname2type $test` q $pages ${thr}t${first_obj} &"
+                    
+                done
+                pidcount=0
+                for host in ${unique_hosts[@]}; do
+                    echo "wait" >> ${cmdsf}_${host}
+                    pidarray[$pidcount]=0
+                    pidcount=$((pidcount+1))
+                done
+                # timed run of all the per-host script files
+                t0=`date +%s.%N`
+                pidcount=0
+                for host in ${unique_hosts[@]}; do
+                    remote_shell $host bash ${cmdsf}_${host} &
+                    pidarray[$pidcount]=$!
+                    pidcount=$((pidcount+1))
+                done
+                pidcount=0
+                for host in ${unique_hosts[@]}; do
+                    wait ${pidarray[$pidcount]}
+                    pidcount=$((pidcount+1))
+                done
+                #wait
+                t1=`date +%s.%N`
+                # clean up per-host script files
+                for host in ${unique_hosts[@]}; do
+                    rm ${cmdsf}_${host}
+                done
+                # compute bandwidth from total data / elapsed time
+                str=`awk "BEGIN {printf \"%7.2f \",\
+                         $total_size / (( $t1 - $t0 ) * 1024)}"`
+                print_summary -n "$str"
+                # collect/check individual OST stats
+                echo -n > $tmpf
+                for ((idx=0; idx < ndevs; idx++)); do
+                    client_name="${host_names[$idx]}:${client_names[$idx]}"
+                    tmpfi="${tmpf}_$idx"
+                    echo "=============> $test $client_name" >> $workf
+                    cat $tmpfi >> $workf
+                    get_stats $tmpfi >> $tmpf
+                    rm $tmpfi
+                done
+                # compute/display global min/max stats
+                echo "=============> $test global" >> $workf
+                cat $tmpf >> $workf
+                stats=(`get_global_stats $tmpf`)
+                rm $tmpf
+                if ((stats[0] <= 0)); then
+                    if ((stats[0] < 0)); then
+                    str=`printf "%17s " ERROR`
+                    else
+                    str=`printf "%17s " SHORT`
+                    fi
+                else
+                    str=`awk "BEGIN {printf \"[%7.2f,%7.2f] \",\
+                         (${stats[1]} * $actual_rsz)/1024,\
+                         (${stats[2]} * $actual_rsz)/1024; exit}"`
+                fi
+                print_summary -n "$str"
+            done
+            print_summary ""
+            # destroy objects we created
+            for ((idx=0; idx < ndevs; idx++)); do
+                host=${host_names[$idx]}
+                devno=${devnos[$idx]}
+                client_name="${host}:${client_names[$idx]}"
+                first_obj=${first_objs[$idx]}
+                echo "=============> Destroy $nobj on $client_name" >> $workf
+                destroy_objects $host $devno $first_obj $nobj $tmpf
+                cat $tmpf >> $workf
+                rm $tmpf
+            done
+        done
     done
 done
 
-for ((idx=0; idx < ndevs; idx++)); do
-    teardown_ec_devno $idx
+# tear down any echo clients we created
+for ((i=0; i<ndevs; i++)); do
+    host=${host_names[$i]}
+    if ((${do_teardown_ec[$i]})); then
+        teardown_ec_devno $host ${client_names[$i]}
+    fi
 done
 
-if ((load_obdecho)); then
-    rmmod obdecho
-fi
+# unload any obdecho modules we loaded
+pidcount=0
+for host in ${unique_hosts[@]}; do
+    remote_shell $host "killall vmstat" &
+    pid=$!
+    kill -term ${vmstatpids[$pidcount]}
+    kill -kill ${vmstatpids[$pidcount]} 2>/dev/null
+    wait $pid
+    pidcount=$((pidcount+1))
+    if ((${do_unload_obdecho[$host]})); then
+        unload_obdecho $host
+    fi
+done
+
+exit 0