diff options
| author | Alexander Aring <aahringo@redhat.com> | 2024-03-28 11:48:40 -0400 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2024-04-01 13:31:12 -0500 |
| commit | 0175e51b5134b55c89364aae68ec16271c67e472 (patch) | |
| tree | e66772705d2634eb6cf6450a8fd896cc029bcbcd /fs/dlm/user.c | |
| parent | 4ed424280942f40b033d669eb9204a52e6d41639 (diff) | |
| download | linux-0175e51b5134b55c89364aae68ec16271c67e472.tar.gz linux-0175e51b5134b55c89364aae68ec16271c67e472.tar.bz2 linux-0175e51b5134b55c89364aae68ec16271c67e472.zip | |
dlm: combine switch case fail and default statements
This patch combines the failure and default cases for enqueue and
dequeue a callback to the lkb callback queue that should end in both
cases as it should never happen.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/user.c')
| -rw-r--r-- | fs/dlm/user.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/fs/dlm/user.c b/fs/dlm/user.c index 6f99bbeeac9b..fa99b6074e5c 100644 --- a/fs/dlm/user.c +++ b/fs/dlm/user.c @@ -231,10 +231,6 @@ void dlm_user_add_ast(struct dlm_lkb *lkb, uint32_t flags, int mode, rv = dlm_enqueue_lkb_callback(lkb, flags, mode, status, sbflags); switch (rv) { - case DLM_ENQUEUE_CALLBACK_FAILURE: - spin_unlock(&proc->asts_spin); - WARN_ON_ONCE(1); - goto out; case DLM_ENQUEUE_CALLBACK_NEED_SCHED: kref_get(&lkb->lkb_ref); list_add_tail(&lkb->lkb_cb_list, &proc->asts); @@ -242,9 +238,12 @@ void dlm_user_add_ast(struct dlm_lkb *lkb, uint32_t flags, int mode, break; case DLM_ENQUEUE_CALLBACK_SUCCESS: break; + case DLM_ENQUEUE_CALLBACK_FAILURE: + fallthrough; default: + spin_unlock(&proc->asts_spin); WARN_ON_ONCE(1); - break; + goto out; } spin_unlock(&proc->asts_spin); |