diff options
| author | Alexander Aring <aahringo@redhat.com> | 2022-04-04 23:06:44 +0300 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2022-04-06 22:02:54 +0300 |
| commit | e91ce03b27b6815cae064bb3d608b7cd26f3fab4 (patch) | |
| tree | c476922d8c39563b57f25cff0dffddc1d8f07e98 /fs/dlm | |
| parent | c087eabde171cf7009e513781d8e81f923630d5e (diff) | |
| download | linux-e91ce03b27b6815cae064bb3d608b7cd26f3fab4.tar.xz | |
dlm: remove found label in dlm_master_lookup
This patch cleanups a not necessary label found which can be replaced by
a proper else handling to jump over a specific code block.
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/lock.c | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/fs/dlm/lock.c b/fs/dlm/lock.c index fd3a9bae5b5b..25468b5e65ad 100644 --- a/fs/dlm/lock.c +++ b/fs/dlm/lock.c @@ -954,18 +954,18 @@ int dlm_master_lookup(struct dlm_ls *ls, int from_nodeid, char *name, int len, hold_rsb(r); spin_unlock(&ls->ls_rsbtbl[b].lock); lock_rsb(r); - goto found; - } + } else { + error = dlm_search_rsb_tree(&ls->ls_rsbtbl[b].toss, name, len, &r); + if (error) + goto not_found; - error = dlm_search_rsb_tree(&ls->ls_rsbtbl[b].toss, name, len, &r); - if (error) - goto not_found; + /* because the rsb is inactive (on toss list), it's not refcounted + * and lock_rsb is not used, but is protected by the rsbtbl lock + */ - /* because the rsb is inactive (on toss list), it's not refcounted - and lock_rsb is not used, but is protected by the rsbtbl lock */ + toss_list = 1; + } - toss_list = 1; - found: if (r->res_dir_nodeid != our_nodeid) { /* should not happen, but may as well fix it and carry on */ log_error(ls, "dlm_master_lookup res_dir %d our %d %s", |