From patchwork Sat Jun 26 09:16:33 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jianguo Wu X-Patchwork-Id: 12346343 Received: from m12-18.163.com (m12-18.163.com [220.181.12.18]) by smtp.subspace.kernel.org (Postfix) with ESMTP id B03AB6D00 for ; Sat, 26 Jun 2021 09:16:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com; s=s110527; h=From:Subject:Date:Message-Id; bh=5Ea3h+FAXYlUEEu4eT FyHT/wWRWNJVIVRpqJ9ENazJE=; b=LD2EhfEBce+lZQq3BhoqM0a5bv3W4/Jr4q PDNZSHePGzzB+KiEC9nz//58OFR3Ou2jGDdr5tf0OsMxczotCR2YQWHnG32ua8J1 E5dveY3EJ/7N/ke8HYH8U+Os7nBDTDO5GhelhK9/Gywcbl/8y9HqqGnTbsiMb/ia 6fwN6t1lI= Received: from localhost.localdomain (unknown [36.111.140.26]) by smtp14 (Coremail) with SMTP id EsCowAD3_dV38NZgettArg--.471S2; Sat, 26 Jun 2021 17:16:40 +0800 (CST) From: wujianguo106@163.com To: mptcp@lists.linux.dev Cc: pabeni@redhat.com, mathew.j.martineau@linux.intel.com Subject: [PATCH mptcp-net v8 0/5] Fix some mptcp syncookie process bugs Date: Sat, 26 Jun 2021 17:16:33 +0800 Message-Id: <1624698998-9029-1-git-send-email-wujianguo106@163.com> X-Mailer: git-send-email 1.8.3.1 X-CM-TRANSID: EsCowAD3_dV38NZgettArg--.471S2 X-Coremail-Antispam: 1Uf129KBjvJXoW7ZrW7ZF1DKw1DAF1kWFW5GFg_yoW8Aw48pF Z5uryrJr4kGFyfXFy3Ar48XryF9a1rKFZ8Gay3Kw1Fvw1Dt3Z8Gryftw15CFy7GFsa934a qr47AFWku3WUZ3DanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDUYxBIdaVFxhVjvjDU0xZFpf9x07j1lksUUUUU= X-Originating-IP: [36.111.140.26] X-CM-SenderInfo: 5zxmxt5qjx0iiqw6il2tof0z/1tbiJxq9kF5u+2+jDgABsY Precedence: bulk X-Mailing-List: mptcp@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: From: wujianguo v7->v8: - patch4: using bool for return value, and keep check_fully_established() untouched. v6->v7: - patch4: corret the declaration of mptcp_incoming_options() when CONFIG_MPTCP is not set - patch5: add tag Reported-by: kernel test robot v5->v6: - patch1: describe the testing tools in the commit log - patch4: add return value to mptcp_incoming_options(), return 0 if a subflow has been reset, else return 1, and drop the packet in tcp_data_queue()/tcp_rcv_state_process() if the return value is 0 - patch5: update selftest case "multiple subflows limited by server", since the expected behavior is changed by patch4 v4->v5: - patch4: add comment v3->v4: - patch1: using seq and sport/dport for hashing, and ignore network headers altogether, as suggest by Florian v2->v3: - patch1: directly use inet6_ehashfn() for IPv6 - patch4: add Fixes tag. v1->v2: - patch1: handle ipv6 sockets/addresses, always use 4-tuple drived hash and never look at skb->hash - patch3: split into 2 patches. - patch4: new added. Jianguo Wu (5): mptcp: fix warning in __skb_flow_dissect() when do syn cookie for subflow join mptcp: remove redundant req destruct in subflow_check_req() mptcp: fix syncookie process if mptcp can not_accept new subflow mptcp: avoid processing packet if a subflow reset selftests: mptcp: fix case multiple subflows limited by server include/net/mptcp.h | 5 +++-- net/ipv4/tcp_input.c | 19 +++++++++++++++---- net/mptcp/options.c | 19 +++++++++++++------ net/mptcp/subflow.c | 11 +++-------- net/mptcp/syncookies.c | 16 +++++++++++++++- tools/testing/selftests/net/mptcp/mptcp_join.sh | 2 +- 6 files changed, 50 insertions(+), 22 deletions(-)