dlm: master - dlm_controld: don't skip unlinking checkpoint

David Teigland teigland at fedoraproject.org
Tue Apr 20 22:07:45 UTC 2010


Gitweb:        http://git.fedorahosted.org/git/dlm.git?p=dlm.git;a=commitdiff;h=fc714e99ee117288c3cc35e20cefb4d55df234ea
Commit:        fc714e99ee117288c3cc35e20cefb4d55df234ea
Parent:        ad80dd21d1f2f91989ba09e47277c939859dc5ca
Author:        David Teigland <teigland at redhat.com>
AuthorDate:    Wed Mar 31 15:06:30 2010 -0500
Committer:     David Teigland <teigland at redhat.com>
CommitterDate: Tue Apr 20 17:06:05 2010 -0500

dlm_controld: don't skip unlinking checkpoint

This reverts commit c1e139fb328408891d054e577b33d724230dde43
which was wrong.

bz 578628

Signed-off-by: David Teigland <teigland at redhat.com>
---
 group/dlm_controld/plock.c |    5 +----
 1 files changed, 1 insertions(+), 4 deletions(-)

diff --git a/group/dlm_controld/plock.c b/group/dlm_controld/plock.c
index 9766ec3..8b3eae2 100644
--- a/group/dlm_controld/plock.c
+++ b/group/dlm_controld/plock.c
@@ -1708,9 +1708,6 @@ static int _unlink_checkpoint(struct lockspace *ls, SaNameT *name)
 	h = (SaCkptCheckpointHandleT) ls->plock_ckpt_handle;
 	log_group(ls, "unlink ckpt %llx", (unsigned long long)h);
 
-	if (!h)
-		return ret;
-
  unlink_retry:
 	rv = saCkptCheckpointUnlink(system_ckpt_handle, name);
 	if (rv == SA_AIS_ERR_TRY_AGAIN) {
@@ -1732,7 +1729,7 @@ static int _unlink_checkpoint(struct lockspace *ls, SaNameT *name)
 		goto status_retry;
 	}
 	if (rv != SA_AIS_OK) {
-		log_error("unlink ckpt status error %d %s", rv, ls->name);
+		log_group(ls, "unlink ckpt status error %d %s", rv, ls->name);
 		goto out_close;
 	}
 


More information about the cluster-commits mailing list