diff options
| author | David Teigland <teigland@redhat.com> | 2006-08-23 23:05:44 +0200 |
|---|---|---|
| committer | Steven Whitehouse <swhiteho@redhat.com> | 2006-08-24 15:38:19 +0200 |
| commit | 233e515f4062f99569c24f00ad7429a860b23db4 (patch) | |
| tree | d2682d2a41f0f1e4ee3216dd661e1df3e276993a /fs/dlm | |
| parent | [DLM] sequence number missing in not_ready reply (diff) | |
| download | linux-233e515f4062f99569c24f00ad7429a860b23db4.tar.xz linux-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 528a2a0fa329..a5e6d184872e 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);); |