diff options
| author | Alexander Aring <aahringo@redhat.com> | 2022-06-22 20:45:13 +0200 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2022-06-24 18:57:40 +0200 |
| commit | ca8031d917691e5c4f101c9ec39fc41b105b7e14 (patch) | |
| tree | ef76519c0ec46283408991fec76febd06810402c /fs/dlm/recoverd.c | |
| parent | fs: dlm: add resource name to tracepoints (diff) | |
| download | linux-ca8031d917691e5c4f101c9ec39fc41b105b7e14.tar.xz linux-ca8031d917691e5c4f101c9ec39fc41b105b7e14.zip | |
fs: dlm: update comments about recovery and membership handling
Make clear that a particular recovery iteration must not be aborted
before membership changes are applied to the members list (ls_nodes)
and midcomms layer. Interrupting recovery before this can result
in missing node-specific changes in midcomms or through lsops.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/recoverd.c')
| -rw-r--r-- | fs/dlm/recoverd.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/dlm/recoverd.c b/fs/dlm/recoverd.c index a55dfce705dd..2d46e65d0101 100644 --- a/fs/dlm/recoverd.c +++ b/fs/dlm/recoverd.c @@ -70,6 +70,10 @@ static int ls_recover(struct dlm_ls *ls, struct dlm_recover *rv) /* * Add or remove nodes from the lockspace's ls_nodes list. + * + * Due to the fact that we must report all membership changes to lsops + * or midcomms layer, it is not permitted to abort ls_recover() until + * this is done. */ error = dlm_recover_members(ls, rv, &neg); |