From nobody Fri Apr 26 22:52:12 2024 Delivered-To: wpasupplicant.patchew@gmail.com Received: by 2002:a9f:3ec7:0:0:0:0:0 with SMTP id n7csp1816909uaj; Tue, 23 Aug 2022 05:52:17 -0700 (PDT) X-Google-Smtp-Source: AA6agR65STLdTUzxdO0sC7IfsPeXQ7AEw7PBLOBfMGj2KBOwQgL1K2fBJw96TvoIxjG1u0nSnq93 X-Received: by 2002:a17:90a:930b:b0:1ed:5441:1fff with SMTP id p11-20020a17090a930b00b001ed54411fffmr3214638pjo.238.1661259137678; Tue, 23 Aug 2022 05:52:17 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1661259137; cv=pass; d=google.com; s=arc-20160816; b=zn/ldQlzX+XIb1sMit3G1CRO+40Ygyo0KsLiuMVi72AnN4BhU4tdHYOacI33GOT+vx VFloeW+aoZ1wFPJesyJw712yT/FEf6TOg6YwjhPdnl0thQMbkgTxhzTAeFMLfAI6Yqdq FOARYGDB1O9ocfc81IYNe6bD50cych90wa4J9NUj0gyhlAkqqveqeHyla07ZXy1437N9 gjtHY8eY3UQ9RF4g/LViRf0drj/REo/3RZmT++1a5p4V10Bke0h77YEoNXBQHEdXbVPY Dc6574ngIOPPzBdbg6DWwjC5+/TL1EfelnmFbfRSPWcW5q8rrQMhaejV/PxIq+hZPOFK Cpww== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :content-transfer-encoding:message-id:date:subject:cc:to:from :dkim-signature; bh=ME80g9GUDRksgLLG0UAbmcjLLAuOhLyLQ3E6Npf+ihE=; b=JrGgExFJ1aTKg39xTrOX1KEWCohBHnOj1gQyPLDOT5eH8ZuG2woL21IrHSrHXuZ9N9 MwEibpZAwMQ7i9unlO6MDSLTFQBPNmb0qw8g9kQq2LLo4GQVSnh0zIS/zNNjjXF+ovTS OCcxhnXkD75Y/uertSKc/6aB4wVITocGuFoDKxcTbJDLZmkATTIiMPsT8VwwiIJ3J4sL q5gPLhB9CHhCC7ku8qTHLAyrcrqi27Mcda42YnUVsYurrCSRxZ9zkrwdaVqcsYYttNTj 9+qtXDuyCLkebW+cVLIS34OvP//+qeCwwGVtIST9+d6AmWCrx1yyBWx9luufC27F0iLC Nodg== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@suse.com header.s=selector1 header.b=XB2wY6Mt; arc=pass (i=1 spf=pass spfdomain=suse.com dkim=pass dkdomain=suse.com dmarc=pass fromdomain=suse.com); spf=pass (google.com: domain of mptcp+bounces-6152-wpasupplicant.patchew=gmail.com@lists.linux.dev designates 139.178.88.99 as permitted sender) smtp.mailfrom="mptcp+bounces-6152-wpasupplicant.patchew=gmail.com@lists.linux.dev"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=suse.com Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id me11-20020a17090b17cb00b001f2c6fe118dsi19975932pjb.38.2022.08.23.05.52.17 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 23 Aug 2022 05:52:17 -0700 (PDT) Received-SPF: pass (google.com: domain of mptcp+bounces-6152-wpasupplicant.patchew=gmail.com@lists.linux.dev designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@suse.com header.s=selector1 header.b=XB2wY6Mt; arc=pass (i=1 spf=pass spfdomain=suse.com dkim=pass dkdomain=suse.com dmarc=pass fromdomain=suse.com); spf=pass (google.com: domain of mptcp+bounces-6152-wpasupplicant.patchew=gmail.com@lists.linux.dev designates 139.178.88.99 as permitted sender) smtp.mailfrom="mptcp+bounces-6152-wpasupplicant.patchew=gmail.com@lists.linux.dev"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=suse.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id ECC2A280C2F for ; Tue, 23 Aug 2022 12:52:16 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id C1783256E; Tue, 23 Aug 2022 12:52:15 +0000 (UTC) X-Original-To: mptcp@lists.linux.dev Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2041.outbound.protection.outlook.com [40.107.22.41]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F17E7A5E for ; Tue, 23 Aug 2022 12:52:13 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=FQ0hF+KgsiufLX/9Q4l8FHcTSVZWUmH8wX1EPZVPZdIeJ6F3DqOZ+kEmyn7G2e7nvzUaZtGUCBu+PwfiFky5NYFVeY7Xj7XioPnst6t/a7Mdb7UgBtz4pkcyELWONuCZ50gukfxGWDnqMeoW69J3cLURSQZrLN39Pa5JMD029MEEx14xEIlbJDoXoEe0qCcayITlV2KKydZJyV7tKqMr8GQzljLizahikluNo7qdZ9T6Lk7ZJxdcL7/BiCfMdPEJWUQQaDmAqQkkF7e8PSUL45IqyuuEyMkqzleEG0q5mHDfk2t4tPb36lvZFPw2rs9a2n9BcyN9Po6HFnUBEu7tCw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=ME80g9GUDRksgLLG0UAbmcjLLAuOhLyLQ3E6Npf+ihE=; b=keIcB5KLDtR2aeCU+XB6nRGiwWtuNVmkuDQctUJBsaMk3VXI5ZtlofGVuqBaRylS6cPO3zPJqGWXC/wNgEzuFJVDcb2bNLaUEWgvbzywesKl9YB8E6i7XUtgf2tY8KFbZ2PFv7YemL/qYdOptqkK3pE7KwgmggyD01Db08jk7fwctFxQONAwONkWyg864NavQYiwVZ3obrAnAD2qVW5Wlb4YTPKjhbwgwMU3yU4qx1e9pRggvQtFrwcTY2TB+Jt++UqZDYxTdfqAuUev3C0YFe/LBW0y9ucpwtXZX05GgNEV+8PsLAEttQ8T5caxH16lPpMT8ltDMoQKRk6UF9FdnA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ME80g9GUDRksgLLG0UAbmcjLLAuOhLyLQ3E6Npf+ihE=; b=XB2wY6MtSkg6aWrRJvoYpXBlI8dJjRFoOIDAIQQTAjIcEK7LZ04fgbRaPCB5tnTgAJ5gwUs6RZ8SkAUWz23V/KnLpP+FCPxvUUoe+unrOA1Z/9q2RQKYFLTvtYN0tiCoJDkiNjUi0YHgfNz0l6hV1AhiWmsRC8PuCZa5gThV/uEdGeGVIDB/1nNk8fA2kRMLenEfF9XLZXdegIQWNB1GOutB5u9inJX+htRV7B9tdq5RHhvZqASgFVHyQW/LGPU7rQ9ZgFQpTCeeUUE1dBMPF7wl0LG+tLfdErSd0qM92ym+1Y9rq747hUIgdKB5yB3s+4IrOeJtpApdoAFrIbSXFg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Received: from HE1PR0402MB3497.eurprd04.prod.outlook.com (2603:10a6:7:83::14) by AM8PR04MB7201.eurprd04.prod.outlook.com (2603:10a6:20b:1d3::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5546.24; Tue, 23 Aug 2022 12:52:11 +0000 Received: from HE1PR0402MB3497.eurprd04.prod.outlook.com ([fe80::c9cf:b207:4052:42e8]) by HE1PR0402MB3497.eurprd04.prod.outlook.com ([fe80::c9cf:b207:4052:42e8%4]) with mapi id 15.20.5546.022; Tue, 23 Aug 2022 12:52:11 +0000 From: Geliang Tang To: mptcp@lists.linux.dev Cc: Geliang Tang , Mat Martineau Subject: [PATCH mptcp-next] mptcp: add do_check_data_fin to replace copied Date: Tue, 23 Aug 2022 20:52:22 +0800 Message-Id: <95f0f96d90264cc6411b024753fe8430f6998e4f.1661258990.git.geliang.tang@suse.com> X-Mailer: git-send-email 2.35.3 Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: SI2PR02CA0044.apcprd02.prod.outlook.com (2603:1096:4:196::17) To HE1PR0402MB3497.eurprd04.prod.outlook.com (2603:10a6:7:83::14) Precedence: bulk X-Mailing-List: mptcp@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d524fa61-d61e-4037-a720-08da85064b6c X-MS-TrafficTypeDiagnostic: AM8PR04MB7201:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 76ysNxt4Do5y3kKrvAzQPdhT8OTQE6Rk4ogAMIU2HgVHFSqIvR2xqTN8insUggvH2+bb0IuTvoVVD39AEAJVXMicQ9fR+FVC0LveFyobuyBoD+O4Q/JJAP0tOIioClX454XNplm1IrL6nVoobC/37l8Ypi6fTa9niVbg2tSYa7+1OPFcs0eFbcqfuaZMaFplQrUPw0LYb69ZunkHEqtj17848oF+lW2RS+AK/SFdxPDrmwq72RJLEgz/3GXlDetJy2pApfTjYq/4PciHAYlcojOJo2FcSfFikRA19JH1v+C0KyFGLUsCupMfQboR8Wg2xq6Ixt/5Ex5B3/QkhKKCEqj6O0Rf4yR4yNtV2Xy0Tmg3SkiqjyOJISsXzuK7mbXH23m8S7vG1KbXDmdX6t+a/nMgN8GE1KTVGKXUkBRTYoXrNFcC6yZWVq7feOU8/2o2pC0s48vfrk+iGuOgZSthn1iKGfDSMypoHRNWVkucJRyU47MwqKINCDMBDqrdlQFETVj151nUNClNNAzyKVB1M7iIcWluyKieB0NaTHZC3T69YcZqwHPeVk23TukhzrNa1ZHs5NFk9OeG4wfn1doZPgkNqo3nTmtQbyeXEF/2ptDCTSCLj2ATs5RxeTmbin7Br5ub9LTk7wBtqK2LybjinEkDmmuD1MuAPlyp1QMSUIom75HtQJ1Z8L/Z16AKrbR4gOJW6ftPeRANlT9rizrtC30VU3wi82RHENIc/M3UCHgjv4SSRZveYLWXWqALE7/4oo7lyAxby6RjMPtGwVb48Yh9x83d/9Wb2eICc6X5Szc= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:HE1PR0402MB3497.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230016)(136003)(396003)(376002)(39860400002)(346002)(366004)(26005)(8936002)(6666004)(6512007)(38100700002)(41300700001)(6506007)(66556008)(66946007)(66476007)(83380400001)(4326008)(8676002)(44832011)(2906002)(2616005)(5660300002)(36756003)(186003)(316002)(6486002)(54906003)(6916009)(966005)(86362001)(478600001)(13296009);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?2n2eOH3jhPwbIh39/RBCSXlJ4YBBNC0p3wNF8BgbS2vfCmC8jvnZyd/VZXYz?= =?us-ascii?Q?3umCeXlGQP9Fp0BQK1XWNOjdx4MetdYnzeaxK1FfSpdA4tIKeG4STeWMHRX8?= =?us-ascii?Q?7RyMXAYVtAkmQM4vsajEtBOspX2u+4abq8QCKnloUZseolnj9q8Vofk4YCY6?= =?us-ascii?Q?utessku3p37rJlUrZJHcI4i1W6i2YLqgDXdYjWKaH0/xpYu5XuRVi2+jQGDW?= =?us-ascii?Q?mvwIjJbSwsB/EEI8Pp6gymmLm6sAzNrV50o6VfjZN4kXtWoe4MmrI7H1EuHb?= =?us-ascii?Q?lCfLikib2SqpRhTvhGIX015GoOXG1Ao5JYcWsffK5y5iKuFZZdyRSQaHugVW?= =?us-ascii?Q?sdA+sUPdlnfti+3WNn1cwAXYOeROuVrVCBcQg3LiV0Ib5zOZpG6U22+D3Zsg?= =?us-ascii?Q?HVdzv66V5d4U8YIUEmuti4H/KgHVFLVOYJvjvGgckObwJqtF4Bg8Uf9WU2AS?= =?us-ascii?Q?6f7B002M+bm0BCfB1zIO5FUaUJR0x7XVzjBfXuUyi6PmU0SomWy3hzIVaAtf?= =?us-ascii?Q?xwZIAwDdEGYfZDW9gpx79qjEfXZEOSmrszp4U1Sn8kC3H2gRs6nsEQ0XVWta?= =?us-ascii?Q?1RKeCvpDFoEtIDCMK0nrkhAcuslOPpq1X5F5gT36CNJIBbFrtVKmGNfT1irh?= =?us-ascii?Q?4zyqlCop/QhRSS2pNYbrFdti+fLeaNGrYshRKDtdfb0n8sYZtyJAz1J5tWvR?= =?us-ascii?Q?odB8jqsAwrbSk83zXzZaZOXP4xRKUWuFC7rZUdc5nITae6+dq9bRU7XsbhFS?= =?us-ascii?Q?3kZZFyvTM52K2ULB0ptSfmixcpNOTgu62s4L1/hWMY5A++9AGYZHxef985Ys?= =?us-ascii?Q?H7oFBQatdTKCwemgwSTCET86WV4v/62kiMTIQ00F01F9tsCrLoiTlVFhO+BI?= =?us-ascii?Q?ndSRgtpNkOqgvi8VAFmDmbX7XJ9m0rX0+g0dri6yjHbnRFrkL59UrbDVibF5?= =?us-ascii?Q?j4nuMK9Jim9jHcyAzlzRVG/yAIW0qEsmvS+dJZKhbMK2HNX/SYHAUQB6tj5T?= =?us-ascii?Q?wIY6cVLybPlv88G6wsho6REwbtwipEc1fZom9jlbkg1KCI9Eo6OzwKiUS7O8?= =?us-ascii?Q?vDN2nmTYtvO7doJCtCLCnKEeZ1aT0BVJ4lUUguH0+ELUcFtcmslMR+Wfproo?= =?us-ascii?Q?OX1pGvRXOKUBZc94/yDJLVT0GyDEUREPntaqWhyJ3GYeBkCj3coFOuHb136x?= =?us-ascii?Q?UZQlsy4RfHYxbEYrem7jIRRuzPiMWIgH/ZthTKW9HbMl2WKlViT7SsFGYU4U?= =?us-ascii?Q?Co1UpdaZb7vcl35/WijYTJOUf8hp22MZMk4frLXuZrj4/U6ooZ/AgETkHFRv?= =?us-ascii?Q?wFfPvZ9OZnZ9Qnq2TCVvNKY/n/dY2SJfTVg5aUen071BhqT4Dx5NBHcjTejG?= =?us-ascii?Q?Y3iI7QK01j1X9FYEqWUT2wh9Biv/8Dnrlb9HYPPDfPj7SjCMT8tNRhKHoymQ?= =?us-ascii?Q?KZ/Xtv14HkDqOcq+AWEKtAId7/RD8b+GKQb+FzPFloFz/aCBD/xodNdHS0IL?= =?us-ascii?Q?L3jQvxXmZkrrtiMTgtJ5RSj83JhkQf4PCAtVQ2P3OFZboKgi9eCJIImVc6JT?= =?us-ascii?Q?dZPOGXBpxqVckZZanrONCVd6g2mNQoB0uK6M6dZx?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: d524fa61-d61e-4037-a720-08da85064b6c X-MS-Exchange-CrossTenant-AuthSource: HE1PR0402MB3497.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Aug 2022 12:52:11.0712 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Bx/GpSAStE9mcQwa6dH1daY6Qp3k9en4Vz6ZGk9t7+tq4ARVr6CNKiMSi/0lQdva4NMEW9bv9URbBZr6/KOC6w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR04MB7201 Content-Type: text/plain; charset="utf-8" This patch adds a new bool variable 'do_check_data_fin' to replace the original int variable 'copied' in __mptcp_push_pending(), check it to determine whether to call __mptcp_check_send_data_fin(). Suggested-by: Mat Martineau Signed-off-by: Geliang Tang --- Do a small cleanup as Mat suggested in [1]. [1] https://patchwork.kernel.org/project/mptcp/patch/ef3d52ff98c97b3c3e5b1efd84= a1d4bf3448df5d.1656578856.git.geliang.tang@suse.com/ --- net/mptcp/protocol.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/net/mptcp/protocol.c b/net/mptcp/protocol.c index 74699bd47edf..fb574d135002 100644 --- a/net/mptcp/protocol.c +++ b/net/mptcp/protocol.c @@ -1473,8 +1473,9 @@ void __mptcp_push_pending(struct sock *sk, unsigned i= nt flags) struct mptcp_sendmsg_info info =3D { .flags =3D flags, }; + bool do_check_data_fin =3D false; struct mptcp_data_frag *dfrag; - int len, copied =3D 0; + int len; =20 while ((dfrag =3D mptcp_send_head(sk))) { info.sent =3D dfrag->already_sent; @@ -1510,7 +1511,7 @@ void __mptcp_push_pending(struct sock *sk, unsigned i= nt flags) } =20 info.sent +=3D ret; - copied +=3D ret; + do_check_data_fin =3D true; len -=3D ret; =20 mptcp_update_post_push(msk, dfrag, ret); @@ -1526,7 +1527,7 @@ void __mptcp_push_pending(struct sock *sk, unsigned i= nt flags) /* ensure the rtx timer is running */ if (!mptcp_timer_pending(sk)) mptcp_reset_timer(sk); - if (copied) + if (do_check_data_fin) __mptcp_check_send_data_fin(sk); } =20 --=20 2.35.3