diff options
author | David Teigland <teigland@redhat.com> | 2011-10-14 13:34:58 -0400 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2012-01-04 09:53:27 -0500 |
commit | f95a34c66554235b70a681fcd9feebc195f7ec0e (patch) | |
tree | fab03b65c76767e69ac0d58c1a7bae3bfae7459a /fs/dlm/member.c | |
parent | 9beb3bf5a92bb8fc6503f844bf0772df29f14a02 (diff) |
dlm: move recovery barrier calls
Put all the calls to recovery barriers in the same function
to clarify where they each happen. Should not change any behavior.
Also modify some recovery debug lines to make them consistent.
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/member.c')
-rw-r--r-- | fs/dlm/member.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/dlm/member.c b/fs/dlm/member.c index b12532e553f8..5ebd1df69675 100644 --- a/fs/dlm/member.c +++ b/fs/dlm/member.c | |||
@@ -251,7 +251,6 @@ int dlm_recover_members(struct dlm_ls *ls, struct dlm_recover *rv, int *neg_out) | |||
251 | ls->ls_low_nodeid = low; | 251 | ls->ls_low_nodeid = low; |
252 | 252 | ||
253 | make_member_array(ls); | 253 | make_member_array(ls); |
254 | dlm_set_recover_status(ls, DLM_RS_NODES); | ||
255 | *neg_out = neg; | 254 | *neg_out = neg; |
256 | 255 | ||
257 | error = ping_members(ls); | 256 | error = ping_members(ls); |
@@ -261,12 +260,8 @@ int dlm_recover_members(struct dlm_ls *ls, struct dlm_recover *rv, int *neg_out) | |||
261 | ls->ls_members_result = error; | 260 | ls->ls_members_result = error; |
262 | complete(&ls->ls_members_done); | 261 | complete(&ls->ls_members_done); |
263 | } | 262 | } |
264 | if (error) | ||
265 | goto out; | ||
266 | 263 | ||
267 | error = dlm_recover_members_wait(ls); | 264 | log_debug(ls, "dlm_recover_members %d nodes", ls->ls_num_nodes); |
268 | out: | ||
269 | log_debug(ls, "total members %d error %d", ls->ls_num_nodes, error); | ||
270 | return error; | 265 | return error; |
271 | } | 266 | } |
272 | 267 | ||