mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	mptcp: add the fallback check
This patch adds the fallback check in subflow_check_data_avail(). Only do the fallback when the msk hasn't fallen back yet. Suggested-by: Paolo Abeni <pabeni@redhat.com> Signed-off-by: Geliang Tang <geliang.tang@suse.com> Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									1761fed256
								
							
						
					
					
						commit
						0348c690ed
					
				
					 1 changed files with 25 additions and 22 deletions
				
			
		| 
						 | 
					@ -1203,35 +1203,38 @@ static bool subflow_check_data_avail(struct sock *ssk)
 | 
				
			||||||
	return false;
 | 
						return false;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
fallback:
 | 
					fallback:
 | 
				
			||||||
	/* RFC 8684 section 3.7. */
 | 
						if (!__mptcp_check_fallback(msk)) {
 | 
				
			||||||
	if (subflow->send_mp_fail) {
 | 
							/* RFC 8684 section 3.7. */
 | 
				
			||||||
		if (mptcp_has_another_subflow(ssk)) {
 | 
							if (subflow->send_mp_fail) {
 | 
				
			||||||
 | 
								if (mptcp_has_another_subflow(ssk)) {
 | 
				
			||||||
 | 
									ssk->sk_err = EBADMSG;
 | 
				
			||||||
 | 
									tcp_set_state(ssk, TCP_CLOSE);
 | 
				
			||||||
 | 
									subflow->reset_transient = 0;
 | 
				
			||||||
 | 
									subflow->reset_reason = MPTCP_RST_EMIDDLEBOX;
 | 
				
			||||||
 | 
									tcp_send_active_reset(ssk, GFP_ATOMIC);
 | 
				
			||||||
 | 
									while ((skb = skb_peek(&ssk->sk_receive_queue)))
 | 
				
			||||||
 | 
										sk_eat_skb(ssk, skb);
 | 
				
			||||||
 | 
								}
 | 
				
			||||||
 | 
								WRITE_ONCE(subflow->data_avail, MPTCP_SUBFLOW_NODATA);
 | 
				
			||||||
 | 
								return true;
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
							if (subflow->mp_join || subflow->fully_established) {
 | 
				
			||||||
 | 
								/* fatal protocol error, close the socket.
 | 
				
			||||||
 | 
								 * subflow_error_report() will introduce the appropriate barriers
 | 
				
			||||||
 | 
								 */
 | 
				
			||||||
			ssk->sk_err = EBADMSG;
 | 
								ssk->sk_err = EBADMSG;
 | 
				
			||||||
			tcp_set_state(ssk, TCP_CLOSE);
 | 
								tcp_set_state(ssk, TCP_CLOSE);
 | 
				
			||||||
			subflow->reset_transient = 0;
 | 
								subflow->reset_transient = 0;
 | 
				
			||||||
			subflow->reset_reason = MPTCP_RST_EMIDDLEBOX;
 | 
								subflow->reset_reason = MPTCP_RST_EMPTCP;
 | 
				
			||||||
			tcp_send_active_reset(ssk, GFP_ATOMIC);
 | 
								tcp_send_active_reset(ssk, GFP_ATOMIC);
 | 
				
			||||||
			while ((skb = skb_peek(&ssk->sk_receive_queue)))
 | 
								WRITE_ONCE(subflow->data_avail, MPTCP_SUBFLOW_NODATA);
 | 
				
			||||||
				sk_eat_skb(ssk, skb);
 | 
								return false;
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
		WRITE_ONCE(subflow->data_avail, MPTCP_SUBFLOW_NODATA);
 | 
					
 | 
				
			||||||
		return true;
 | 
							__mptcp_do_fallback(msk);
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (subflow->mp_join || subflow->fully_established) {
 | 
					 | 
				
			||||||
		/* fatal protocol error, close the socket.
 | 
					 | 
				
			||||||
		 * subflow_error_report() will introduce the appropriate barriers
 | 
					 | 
				
			||||||
		 */
 | 
					 | 
				
			||||||
		ssk->sk_err = EBADMSG;
 | 
					 | 
				
			||||||
		tcp_set_state(ssk, TCP_CLOSE);
 | 
					 | 
				
			||||||
		subflow->reset_transient = 0;
 | 
					 | 
				
			||||||
		subflow->reset_reason = MPTCP_RST_EMPTCP;
 | 
					 | 
				
			||||||
		tcp_send_active_reset(ssk, GFP_ATOMIC);
 | 
					 | 
				
			||||||
		WRITE_ONCE(subflow->data_avail, MPTCP_SUBFLOW_NODATA);
 | 
					 | 
				
			||||||
		return false;
 | 
					 | 
				
			||||||
	}
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
	__mptcp_do_fallback(msk);
 | 
					 | 
				
			||||||
	skb = skb_peek(&ssk->sk_receive_queue);
 | 
						skb = skb_peek(&ssk->sk_receive_queue);
 | 
				
			||||||
	subflow->map_valid = 1;
 | 
						subflow->map_valid = 1;
 | 
				
			||||||
	subflow->map_seq = READ_ONCE(msk->ack_seq);
 | 
						subflow->map_seq = READ_ONCE(msk->ack_seq);
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue