kevin/linux
1
Fork 0
Code

fs: dlm: remove unnecessary waker_up() calls

Browse Source 
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>
This commit is contained in:
Alexander Aring 2023-01-12 17:18:49 -05:00 committed by David Teigland
parent ef7ef015eb
commit 723b197bbd
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
fs/dlm/midcomms.c
View File

@ -534,7 +534,6 @@ static void dlm_midcomms_receive_buffer(union dlm_packet *p,
midcomms_node_reset(node); midcomms_node_reset(node);
pr_debug("switch node %d to state %s\n", pr_debug("switch node %d to state %s\n",
node->nodeid, dlm_state_str(node->state)); node->nodeid, dlm_state_str(node->state));
wake_up(&node->shutdown_wait);
break; break;
case DLM_LAST_ACK: case DLM_LAST_ACK:
/* probably remove_member caught it, do nothing */ /* 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); midcomms_node_reset(node);
pr_debug("switch node %d to state %s\n", pr_debug("switch node %d to state %s\n",
node->nodeid, dlm_state_str(node->state)); node->nodeid, dlm_state_str(node->state));
wake_up(&node->shutdown_wait);
break; break;
case DLM_CLOSED: case DLM_CLOSED:
/* not valid but somehow we got what we want */ /* not valid but somehow we got what we want */