diff options
| author | Alexander Aring <aahringo@redhat.com> | 2021-05-21 21:08:33 +0200 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2021-05-25 16:22:20 +0200 |
| commit | c937aabbd7f46fa3b283744169a6b48dafbd6cc7 (patch) | |
| tree | 9b6d108b1f0ac7ca4f3bc9b91a0124b774b89458 /fs/dlm | |
| parent | Linux 5.13-rc1 (diff) | |
| download | linux-c937aabbd7f46fa3b283744169a6b48dafbd6cc7.tar.xz linux-c937aabbd7f46fa3b283744169a6b48dafbd6cc7.zip | |
fs: dlm: always run complete for possible waiters
This patch changes the ping_members() result that we always run
complete() for possible waiters. We handle the -EINTR error code as
successful. This error code is returned if the recovery is stopped which
is likely that a new recovery is triggered with a new members
configuration and ping_members() runs again.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm')
| -rw-r--r-- | fs/dlm/member.c | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/fs/dlm/member.c b/fs/dlm/member.c index ceef3f2074ff..48245ba3c640 100644 --- a/fs/dlm/member.c +++ b/fs/dlm/member.c @@ -576,12 +576,18 @@ int dlm_recover_members(struct dlm_ls *ls, struct dlm_recover *rv, int *neg_out) *neg_out = neg; error = ping_members(ls); - if (!error || error == -EPROTO) { - /* new_lockspace() may be waiting to know if the config - is good or bad */ - ls->ls_members_result = error; - complete(&ls->ls_members_done); - } + /* error -EINTR means that a new recovery action is triggered. + * We ignore this recovery action and let run the new one which might + * have new member configuration. + */ + if (error == -EINTR) + error = 0; + + /* new_lockspace() may be waiting to know if the config + * is good or bad + */ + ls->ls_members_result = error; + complete(&ls->ls_members_done); log_rinfo(ls, "dlm_recover_members %d nodes", ls->ls_num_nodes); return error; |