From: zhanghc <zhanghc>
Date: Wed, 25 Nov 2009 10:08:32 +0000 (+0000)
Subject: b=16818
X-Git-Tag: GIT_EPOCH_B_HD_KDMU~58
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=a09235e0c8169f12a3577feea69669504739e6a6;p=fs%2Flustre-release.git

b=16818

1, the test will be skipped if the MGS and OST1 are on the same node
2, the test will be interrputed if "zconf_mount $MOUNT2" failed

i=nathan.rutman@sun.com
i=robert.read@sun.com
i=grev@sun.com
---

diff --git a/lustre/ptlrpc/service.c b/lustre/ptlrpc/service.c
index 36fbdfb..f9d1b73 100644
--- a/lustre/ptlrpc/service.c
+++ b/lustre/ptlrpc/service.c
@@ -774,8 +774,8 @@ static void ptlrpc_update_export_timer(struct obd_export *exp, long extra_delay)
                         exp->exp_obd->obd_eviction_timer =
                                 cfs_time_current_sec() + 3 * PING_INTERVAL;
                         CDEBUG(D_HA, "%s: Think about evicting %s from "CFS_TIME_T"\n",
-                               exp->exp_obd->obd_name, obd_export_nid2str(exp),
-                               oldest_time);
+                               exp->exp_obd->obd_name, 
+                               obd_export_nid2str(oldest_exp), oldest_time);
                 }
         } else {
                 if (cfs_time_current_sec() >
diff --git a/lustre/tests/recovery-small.sh b/lustre/tests/recovery-small.sh
index f4745eb..f910d25 100755
--- a/lustre/tests/recovery-small.sh
+++ b/lustre/tests/recovery-small.sh
@@ -658,6 +658,11 @@ test_26a() {      # was test_26 bug 5921 - evict dead exports by pinger
 	remote_ost_nodsh && skip "remote OST with nodsh" && return 0
 	remote_mds || { skip "local MDS" && return 0; }
 
+        if [ $(facet_host mgs) = $(facet_host ost1) ]; then
+                skip "msg and ost1 are at the same node"
+                return 0
+        fi
+
 	check_timeout || return 1
 
 	local OST_NEXP=$(do_facet ost1 lctl get_param -n obdfilter.${ost1_svc}.num_exports | cut -d' ' -f2)
@@ -680,9 +685,15 @@ run_test 26a "evict dead exports"
 test_26b() {      # bug 10140 - evict dead exports by pinger
 	remote_ost_nodsh && skip "remote OST with nodsh" && return 0
 
+        if [ $(facet_host mgs) = $(facet_host ost1) ]; then
+                skip "msg and ost1 are at the same node"
+                return 0
+        fi
+
 	check_timeout || return 1
 	clients_up
-	zconf_mount `hostname` $MOUNT2 || error "Failed to mount $MOUNT2"
+	zconf_mount `hostname` $MOUNT2 ||
+                { error "Failed to mount $MOUNT2"; return 2; }
 	sleep 1 # wait connections being established
 
 	local MDS_NEXP=$(do_facet $SINGLEMDS lctl get_param -n mdt.${mds1_svc}.num_exports | cut -d' ' -f2)