From: bobijam Date: Fri, 23 Oct 2009 06:09:35 +0000 (+0000) Subject: Branch HEAD X-Git-Tag: GIT_EPOCH_B_HD_KDMU~147 X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=f17556ad8337f187d169893f6df23d7ce1574c50;p=fs%2Flustre-release.git Branch HEAD b=20149 add more debug info. --- diff --git a/lustre/ldlm/ldlm_resource.c b/lustre/ldlm/ldlm_resource.c index ade6a92..af4ad55 100644 --- a/lustre/ldlm/ldlm_resource.c +++ b/lustre/ldlm/ldlm_resource.c @@ -504,7 +504,7 @@ int ldlm_namespace_cleanup(struct ldlm_namespace *ns, int flags) cleanup_resource(res, &res->lr_waiting, flags); spin_lock(&ns->ns_hash_lock); - tmp = tmp->next; + tmp = tmp->next; /* XXX: former stuff caused issues in case of race * between ldlm_namespace_cleanup() and lockd() when @@ -512,11 +512,16 @@ int ldlm_namespace_cleanup(struct ldlm_namespace *ns, int flags) * server. This is 1_4 branch solution, let's see how * will it behave. */ LDLM_RESOURCE_DELREF(res); - if (!ldlm_resource_putref_locked(res)) - CDEBUG(D_INFO, - "Namespace %s resource refcount nonzero " + if (!ldlm_resource_putref_locked(res)) { + CERROR("Namespace %s resource refcount nonzero " "(%d) after lock cleanup; forcing cleanup.\n", ns->ns_name, atomic_read(&res->lr_refcount)); + CERROR("Resource: %p ("LPU64"/"LPU64"/"LPU64"/" + LPU64") (rc: %d)\n", res, + res->lr_name.name[0], res->lr_name.name[1], + res->lr_name.name[2], res->lr_name.name[3], + atomic_read(&res->lr_refcount)); + } } spin_unlock(&ns->ns_hash_lock); }