Smatch complains that we unlock this twice. It looks like an accidental to me. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> --- This was added in 525f31c1f3 ("dlm: fix race between remove and lookup") Untested. diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c index 06063a3..1c10bf4 100644 --- a/fs/dlm/lock.c +++ b/fs/dlm/lock.c @@ -1570,7 +1570,6 @@ static void wait_pending_remove(struct dlm_rsb *r_new) continue; if (rsb_cmp(r_new, r->res_name, r->res_length)) continue; - spin_unlock(&ls->ls_send_remove_spin); log_debug(ls, "delay lookup %p for remove %p dir %d %s", r_new, r, r_new->res_dir_nodeid, r_new->res_name); -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html