diff options
| author | Alexander Aring <aahringo@redhat.com> | 2023-03-06 21:48:15 +0100 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2023-03-06 22:49:07 +0100 |
| commit | 8a39dcd9c32dd3f0d3af745c72834d58a43ed90a (patch) | |
| tree | 3bada81c1d1ec675af9147cf7653f0d71ab4a167 /fs/dlm/debug_fs.c | |
| parent | fs: dlm: store lkb distributed flags into own value (diff) | |
| download | linux-8a39dcd9c32dd3f0d3af745c72834d58a43ed90a.tar.xz linux-8a39dcd9c32dd3f0d3af745c72834d58a43ed90a.zip | |
fs: dlm: change dflags to use atomic bits
Currently manipulating lkb_dflags assumes to held the rsb lock assigned
to the lkb. This is held by dlm message processing after certain
time to lookup the right rsb from the received lkb message id. For user
space locks flags, which is currently the only use case for lkb_dflags,
flags are also being set during dlm character device handling without
holding the rsb lock. To minimize the risk that bit operations are
getting corrupted we switch to atomic bit operations. This patch will
also introduce helpers to snapshot atomic bit values in an non atomic
way. There might be still issues with the flag handling e.g. running in
case of manipulating bit ops and snapshot them at the same time, but this
patch minimize them and will start to use atomic bit operations.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/debug_fs.c')
| -rw-r--r-- | fs/dlm/debug_fs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/dlm/debug_fs.c b/fs/dlm/debug_fs.c index 65cfb58eaa6b..370a0c81a0b3 100644 --- a/fs/dlm/debug_fs.c +++ b/fs/dlm/debug_fs.c @@ -170,7 +170,7 @@ static void print_format2_lock(struct seq_file *s, struct dlm_lkb *lkb, u64 xid = 0; u64 us; - if (lkb->lkb_dflags & DLM_DFL_USER) { + if (test_bit(DLM_DFL_USER_BIT, &lkb->lkb_dflags)) { if (lkb->lkb_ua) xid = lkb->lkb_ua->xid; } @@ -230,7 +230,7 @@ static void print_format3_lock(struct seq_file *s, struct dlm_lkb *lkb, { u64 xid = 0; - if (lkb->lkb_dflags & DLM_DFL_USER) { + if (test_bit(DLM_DFL_USER_BIT, &lkb->lkb_dflags)) { if (lkb->lkb_ua) xid = lkb->lkb_ua->xid; } |