fs: dlm: remove unnecessary waker_up() calls
authorAlexander Aring <aahringo@redhat.com>
Thu, 12 Jan 2023 22:18:49 +0000 (17:18 -0500)
committerDavid Teigland <teigland@redhat.com>
Mon, 23 Jan 2023 22:27:28 +0000 (16:27 -0600)
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>
fs/dlm/midcomms.c

index 69e6f47..c02c43e 100644 (file)
@@ -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 */