Commit 00908b33 authored by Alexander Aring's avatar Alexander Aring Committed by David Teigland
Browse files

fs: dlm: send FIN ack back in right cases 



This patch moves to send a ack back for receiving a FIN message only
when we are in valid states. In other cases and there might be a sender
waiting for a ack we just let it timeout at the senders time and
hopefully all other cleanups will remove the FIN message on their
sending queue. As an example we should never send out an ACK being in
LAST_ACK state or we cannot assume a working socket communication when
we are in CLOSED state.

Cc: stable@vger.kernel.org
Fixes: 489d8e55 ("fs: dlm: add reliable connection if reconnect")
Signed-off-by: default avatarAlexander Aring <aahringo@redhat.com>
Signed-off-by: default avatarDavid Teigland <teigland@redhat.com>
parent a5849636

fs/dlm/midcomms.c

+5 −4
Original line number Diff line number Diff line
@@ -375,7 +375,7 @@ static int dlm_send_ack(int nodeid, uint32_t seq) 
	struct dlm_msg *msg;

	char *ppc;



	msg = dlm_lowcomms_new_msg(nodeid, mb_len, GFP_NOFS, &ppc,

	msg = dlm_lowcomms_new_msg(nodeid, mb_len, GFP_ATOMIC, &ppc,

				   NULL, NULL);

	if (!msg)

		return -ENOMEM;
@@ -498,15 +498,14 @@ static void dlm_midcomms_receive_buffer(union dlm_packet *p, 


		switch (p->header.h_cmd) {

		case DLM_FIN:

			/* send ack before fin */

			dlm_send_ack(node->nodeid, node->seq_next);



			spin_lock(&node->state_lock);

			pr_debug("receive fin msg from node %d with state %s\n",

				 node->nodeid, dlm_state_str(node->state));



			switch (node->state) {

			case DLM_ESTABLISHED:

				dlm_send_ack(node->nodeid, node->seq_next);



				node->state = DLM_CLOSE_WAIT;

				pr_debug("switch node %d to state %s\n",

					 node->nodeid, dlm_state_str(node->state));
@@ -523,12 +522,14 @@ static void dlm_midcomms_receive_buffer(union dlm_packet *p, 
				}

				break;

			case DLM_FIN_WAIT1:

				dlm_send_ack(node->nodeid, node->seq_next);

				node->state = DLM_CLOSING;

				set_bit(DLM_NODE_FLAG_STOP_RX, &node->flags);

				pr_debug("switch node %d to state %s\n",

					 node->nodeid, dlm_state_str(node->state));

				break;

			case DLM_FIN_WAIT2:

				dlm_send_ack(node->nodeid, node->seq_next);

				midcomms_node_reset(node);

				pr_debug("switch node %d to state %s\n",

					 node->nodeid, dlm_state_str(node->state));