Low: Improve netfs unmount when server is unavailable [RHEL6]

Resolves: rhbz#678467

Signed-off-by Lon Hohberger<[email protected]>

---
  rgmanager/src/resources/fs.sh.in        |    3 ++-
  rgmanager/src/resources/netfs.sh        |    3 ++-
  rgmanager/src/resources/utils/fs-lib.sh |   12 ++++++++++--
  3 files changed, 14 insertions(+), 4 deletions(-)

rgmanager/src/resources/fs.sh.in +2 -1

index 806a5d6..3e1cc15 100644
@@ -445,7 +445,8 @@ do_force_unmount() {
  		export nfslock_reclaim=1
  	fi
 
-	return 0
+	# Proceed with fuser -kvm...
+	return 1
  }
 
 

rgmanager/src/resources/netfs.sh +2 -1

index f911ae2..52cefc4 100644
@@ -354,12 +354,13 @@ do_force_unmount() {
  	nfs|nfs4)
  		ocf_log warning "Calling 'umount -f $mp'"
  		umount -f "$OCF_RESKEY_mountpoint"
+		return $?
  		;;
  	*)
  		;;
  	esac
 
-	return 0	# Returning 0 lets stop_filesystem do add'l checks
+	return 1	# Returning 1 lets stop_filesystem do add'l checks
  }
 
 

rgmanager/src/resources/utils/fs-lib.sh +10 -2

index 38c242d..163cfb7 100644
@@ -543,8 +543,10 @@ do_post_unmount() {
 
 
  # Agent-specific force-unmount logic, if required
+# return = nonzero if successful, or 0 if unsuccessful
+# (unsuccessful = try harder)
  do_force_unmount() {
-	return 0
+	return 1
  }
 
 

@@ -811,6 +813,13 @@ stop: Could not match $OCF_RESKEY_device with a real device"
 
  		# Force unmount: try #1: send SIGTERM
  		if [ $try -eq 1 ]; then
+			# Try fs-specific force-unmount, if provided
+			do_force_unmount
+			if [ $? -eq 0 ]; then
+				# if this succeeds, we should be done
+				continue
+			fi
+
  			ocf_log warning "Sending SIGTERM to processes on $mp"
  			fuser -TERM -kvm "$mp"
  			continue

@@ -818,7 +827,6 @@ stop: Could not match $OCF_RESKEY_device with a real device"
  			ocf_log warning "Sending SIGKILL to processes on $mp"
  			fuser -kvm "$mp"
 
-			do_force_unmount
  			case $? in
  			0)
  				;;