diff options
| author | David Teigland <teigland@redhat.com> | 2006-08-23 16:05:44 -0500 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2006-08-24 09:38:19 -0400 |
| commit | 233e515f4062f99569c24f00ad7429a860b23db4 (patch) | |
| tree | d2682d2a41f0f1e4ee3216dd661e1df3e276993a /fs/dlm | |
| parent | f5888750aad219bec42f3f28354eb230d1a47b89 (diff) | |
| download | linux-3.10-233e515f4062f99569c24f00ad7429a860b23db4.tar.gz linux-3.10-233e515f4062f99569c24f00ad7429a860b23db4.tar.bz2 linux-3.10-233e515f4062f99569c24f00ad7429a860b23db4.zip | |
[DLM] recover_locks not clearing NEW_MASTER flag
When there are no locks on a resource, the recover_locks() function fails
to clear the NEW_MASTER flag by going directly to out, missing the line
that clears the flag.
Signed-off-by: David Teigland <teigland@redhat.com>
Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'fs/dlm')
| -rw-r--r-- | fs/dlm/recover.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/fs/dlm/recover.c b/fs/dlm/recover.c index 528a2a0fa32..a5e6d184872 100644 --- a/fs/dlm/recover.c +++ b/fs/dlm/recover.c @@ -473,22 +473,11 @@ static int recover_locks_queue(struct dlm_rsb *r, struct list_head *head) return error; } -static int all_queues_empty(struct dlm_rsb *r) -{ - if (!list_empty(&r->res_grantqueue) || - !list_empty(&r->res_convertqueue) || - !list_empty(&r->res_waitqueue)) - return 0; - return 1; -} - static int recover_locks(struct dlm_rsb *r) { int error = 0; lock_rsb(r); - if (all_queues_empty(r)) - goto out; DLM_ASSERT(!r->res_recover_locks_count, dlm_dump_rsb(r);); |