diff options
| author | Alexander Aring <aahringo@redhat.com> | 2023-01-13 01:18:49 +0300 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2023-01-24 01:27:28 +0300 |
| commit | 723b197bbdf1e0adbab772b8e5e022c40db6a9fe (patch) | |
| tree | f12d654b2799617ca732fdb9c788177c9b3b45bc | |
| parent | ef7ef015eb333c8954d7165d216ec0c774d7e4e4 (diff) | |
| download | linux-723b197bbdf1e0adbab772b8e5e022c40db6a9fe.tar.xz | |
fs: dlm: remove unnecessary waker_up() calls
The wake_up() is already handled inside of midcomms_node_reset() when
switching the state to CLOSED state. So there is not need to call it
after midcomms_node_reset() again.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
| -rw-r--r-- | fs/dlm/midcomms.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/fs/dlm/midcomms.c b/fs/dlm/midcomms.c index 69e6f4705ebc..c02c43e4980a 100644 --- a/fs/dlm/midcomms.c +++ b/fs/dlm/midcomms.c @@ -534,7 +534,6 @@ static void dlm_midcomms_receive_buffer(union dlm_packet *p, midcomms_node_reset(node); pr_debug("switch node %d to state %s\n", node->nodeid, dlm_state_str(node->state)); - wake_up(&node->shutdown_wait); break; case DLM_LAST_ACK: /* probably remove_member caught it, do nothing */ @@ -1262,7 +1261,6 @@ static void dlm_act_fin_ack_rcv(struct midcomms_node *node) midcomms_node_reset(node); pr_debug("switch node %d to state %s\n", node->nodeid, dlm_state_str(node->state)); - wake_up(&node->shutdown_wait); break; case DLM_CLOSED: /* not valid but somehow we got what we want */ |