From nobody Sat Feb 7 09:46:36 2026 Delivered-To: wpasupplicant.patchew@gmail.com Received: by 2002:ab0:35eb:0:0:0:0:0 with SMTP id w11csp24925uau; Tue, 28 Jun 2022 14:29:30 -0700 (PDT) X-Google-Smtp-Source: AGRyM1vnES99kI9qO+3JYZUaw7DNDaVZWqFW5juN90k3fWF2yOdyxStRlIkKJfLWZGIR5bnfgC6v X-Received: by 2002:a05:6870:5a0:b0:101:229:eb16 with SMTP id m32-20020a05687005a000b001010229eb16mr27336oap.268.1656451770288; Tue, 28 Jun 2022 14:29:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1656451770; cv=none; d=google.com; s=arc-20160816; b=FiVuy2Uo2d17CEMjenksKPbLzorsvIDxDZ8OLMOe7kijEkQsMirvuHDXy8WYZ9Uzjx pgkT/PF1GN9MNqhMNvw41gLjLp42USn/cfLz+U+jIYskgyxIXXGMRLt7Xzlo4VIyN2De gxZC9sEWshF6QgdaOIMy3ymML6+ayNzagykUWpA0QO1PzNTKLzGYoOZJnONHXlAeGkFk 45uaVoXHfkSb3MxMifAqMFw7ZSPkSMpvLRKL9PsSQPJCA5Xd/ZBeNVBM088+15tV64DN frPFy8bxI5t6ZFgl1ttjQQgGEoPZFxkg8rZgRM1JpTUmx/fzMHfDRb7srYsFJCas9f5g 3L+Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :date:subject:to:from:dkim-signature; bh=dqia7nL+D+pH4osJWzT60AkIntYvqJleXFJGLOSv7IM=; b=uH1BMOV9VtFz1KpSRLtpwg1UYTS6QFdoK1OB5oxnRUJRNuA1+hPwnuQb5b+ZyPRotk 5xMC+Atm3d1Vae3/qAufCSyNHpJS5PRyb8cwklpa4I4VgijdWOhi3wzAiP3+2xeDmBV+ Fjrfec0rZxNnpm/qbE1oYZSI7uW4CPhidiVlEtLEIStoSuyMtYvzpbhpxD6GzWaREBJO CAPX9PlLoGU7tgKlmTOlXXdRCfcqTZTecolWWK7KVYvDvRdsAzofH/8YkkX1ANOk3fFF 8rfaftEo+yYEH8h7gMsFy6vZvkTRc1nh+VlnwACipucL3z4e2h8rjZMLYxsrXJtUBi9K FhDw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=W0Wx2I5Y; spf=pass (google.com: domain of mptcp+bounces-5870-wpasupplicant.patchew=gmail.com@lists.linux.dev designates 139.178.84.19 as permitted sender) smtp.mailfrom="mptcp+bounces-5870-wpasupplicant.patchew=gmail.com@lists.linux.dev"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from da.mirrors.kernel.org (da.mirrors.kernel.org. [139.178.84.19]) by mx.google.com with ESMTPS id q12-20020a056830440c00b00616caae239bsi11469724otv.320.2022.06.28.14.29.30 for (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 28 Jun 2022 14:29:30 -0700 (PDT) Received-SPF: pass (google.com: domain of mptcp+bounces-5870-wpasupplicant.patchew=gmail.com@lists.linux.dev designates 139.178.84.19 as permitted sender) client-ip=139.178.84.19; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=W0Wx2I5Y; spf=pass (google.com: domain of mptcp+bounces-5870-wpasupplicant.patchew=gmail.com@lists.linux.dev designates 139.178.84.19 as permitted sender) smtp.mailfrom="mptcp+bounces-5870-wpasupplicant.patchew=gmail.com@lists.linux.dev"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.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 da.mirrors.kernel.org (Postfix) with ESMTPS id 259262E0A2B for ; Tue, 28 Jun 2022 21:29:30 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id DA4623FFD; Tue, 28 Jun 2022 21:29:28 +0000 (UTC) X-Original-To: mptcp@lists.linux.dev Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (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 1A2044C98 for ; Tue, 28 Jun 2022 21:29:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1656451766; x=1687987766; h=from:to:subject:date:message-id:in-reply-to:references: mime-version:content-transfer-encoding; bh=19dWtm5lcMflKYg5JCI67jIQrtUZXabCHNUzoA4OKI8=; b=W0Wx2I5YDIBZZVxWz41O6PYQv2Po7AXQkLRoRedauMXXB7Npy3r706a5 cHZLLSxuLkrhC4MiMAWo5NSegvmPKOoa5FyF+Ak6jBLwa0TBww7UgrUcC 88JafpttleCEKjVjczN3nkby9n0E38WRB9d/LW208kf+obUGYcq66ajkP UWdwve4uJUDofQ69Jhe+I53dYFo8xOD8eOMnemaJiGZm4nn4nEz8Ngvad D38QFTAA5kf2fB+hnWNP2r5yI/LihBk9fmlmVVwoi7z7pUNTexur7hmDM +qKpjTZ3DGeDS8U9MxfOWUmLXHKXMwffz00JyeT0Qy5WYfHi4+C4pXS6d w==; X-IronPort-AV: E=McAfee;i="6400,9594,10392"; a="345847047" X-IronPort-AV: E=Sophos;i="5.92,229,1650956400"; d="scan'208";a="345847047" Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga105.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Jun 2022 14:29:23 -0700 X-IronPort-AV: E=Sophos;i="5.92,229,1650956400"; d="scan'208";a="732911205" Received: from otc-tsn-4.jf.intel.com ([10.23.153.135]) by fmsmga001-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Jun 2022 14:29:23 -0700 From: Kishen Maloor To: kishen.maloor@intel.com, mptcp@lists.linux.dev Subject: [PATCH mptcp-net v3 1/2] mptcp: netlink: issue MP_PRIO signals from userspace PMs Date: Tue, 28 Jun 2022 17:29:17 -0400 Message-Id: <20220628212918.417515-2-kishen.maloor@intel.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20220628212918.417515-1-kishen.maloor@intel.com> References: <20220628212918.417515-1-kishen.maloor@intel.com> Precedence: bulk X-Mailing-List: mptcp@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" This change updates MPTCP_PM_CMD_SET_FLAGS to allow userspace PMs to issue MP_PRIO signals over a specific subflow selected by the connection token, local and remote address+port. Closes: https://github.com/multipath-tcp/mptcp_net-next/issues/286 Fixes: 702c2f646d42 ("mptcp: netlink: allow userspace-driven subflow establ= ishment") Signed-off-by: Kishen Maloor --- v3: -use local and remote address+port (instead of address ID) alongwith the connection token to select a subflow. --- net/mptcp/pm_netlink.c | 30 +++++++++++++++++++++++++----- net/mptcp/pm_userspace.c | 30 ++++++++++++++++++++++++++++++ net/mptcp/protocol.h | 8 +++++++- 3 files changed, 62 insertions(+), 6 deletions(-) diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c index 05c6a95e9c28..602892f2f921 100644 --- a/net/mptcp/pm_netlink.c +++ b/net/mptcp/pm_netlink.c @@ -717,9 +717,10 @@ void mptcp_pm_nl_addr_send_ack(struct mptcp_sock *msk) } } =20 -static int mptcp_pm_nl_mp_prio_send_ack(struct mptcp_sock *msk, - struct mptcp_addr_info *addr, - u8 bkup) +int mptcp_pm_nl_mp_prio_send_ack(struct mptcp_sock *msk, + struct mptcp_addr_info *addr, + struct mptcp_addr_info *rem, + u8 bkup) { struct mptcp_subflow_context *subflow; =20 @@ -728,7 +729,7 @@ static int mptcp_pm_nl_mp_prio_send_ack(struct mptcp_so= ck *msk, mptcp_for_each_subflow(msk, subflow) { struct sock *ssk =3D mptcp_subflow_tcp_sock(subflow); struct sock *sk =3D (struct sock *)msk; - struct mptcp_addr_info local; + struct mptcp_addr_info local, remote; bool slow; =20 local_address((struct sock_common *)ssk, &local); @@ -736,6 +737,12 @@ static int mptcp_pm_nl_mp_prio_send_ack(struct mptcp_s= ock *msk, continue; =20 slow =3D lock_sock_fast(ssk); + if (rem && rem->family !=3D AF_UNSPEC) { + remote_address((struct sock_common *)ssk, &remote); + if (!mptcp_addresses_equal(&remote, rem, rem->port)) + continue; + } + if (subflow->backup !=3D bkup) msk->last_snd =3D NULL; subflow->backup =3D bkup; @@ -1839,7 +1846,7 @@ static int mptcp_nl_set_flags(struct net *net, =20 lock_sock(sk); if (changed & MPTCP_PM_ADDR_FLAG_BACKUP) - ret =3D mptcp_pm_nl_mp_prio_send_ack(msk, addr, bkup); + ret =3D mptcp_pm_nl_mp_prio_send_ack(msk, addr, NULL, bkup); if (changed & MPTCP_PM_ADDR_FLAG_FULLMESH) mptcp_pm_nl_fullmesh(msk, addr); release_sock(sk); @@ -1855,6 +1862,9 @@ static int mptcp_nl_set_flags(struct net *net, static int mptcp_nl_cmd_set_flags(struct sk_buff *skb, struct genl_info *i= nfo) { struct mptcp_pm_addr_entry addr =3D { .addr =3D { .family =3D AF_UNSPEC }= , }, *entry; + struct mptcp_pm_addr_entry remote =3D { .addr =3D { .family =3D AF_UNSPEC= }, }; + struct nlattr *attr_rem =3D info->attrs[MPTCP_PM_ATTR_ADDR_REMOTE]; + struct nlattr *token =3D info->attrs[MPTCP_PM_ATTR_TOKEN]; struct nlattr *attr =3D info->attrs[MPTCP_PM_ATTR_ADDR]; struct pm_nl_pernet *pernet =3D genl_info_pm_nl(info); u8 changed, mask =3D MPTCP_PM_ADDR_FLAG_BACKUP | @@ -1867,6 +1877,12 @@ static int mptcp_nl_cmd_set_flags(struct sk_buff *sk= b, struct genl_info *info) if (ret < 0) return ret; =20 + if (attr_rem) { + ret =3D mptcp_pm_parse_entry(attr_rem, info, false, &remote); + if (ret < 0) + return ret; + } + if (addr.flags & MPTCP_PM_ADDR_FLAG_BACKUP) bkup =3D 1; if (addr.addr.family =3D=3D AF_UNSPEC) { @@ -1875,6 +1891,10 @@ static int mptcp_nl_cmd_set_flags(struct sk_buff *sk= b, struct genl_info *info) return -EOPNOTSUPP; } =20 + if (token) + return mptcp_userspace_pm_set_flags(sock_net(skb->sk), + token, &addr, &remote, bkup); + spin_lock_bh(&pernet->lock); entry =3D __lookup_addr(pernet, &addr.addr, lookup_by_id); if (!entry) { diff --git a/net/mptcp/pm_userspace.c b/net/mptcp/pm_userspace.c index 26212bebc5ed..51e2f066d54f 100644 --- a/net/mptcp/pm_userspace.c +++ b/net/mptcp/pm_userspace.c @@ -420,3 +420,33 @@ int mptcp_nl_cmd_sf_destroy(struct sk_buff *skb, struc= t genl_info *info) sock_put((struct sock *)msk); return err; } + +int mptcp_userspace_pm_set_flags(struct net *net, struct nlattr *token, + struct mptcp_pm_addr_entry *loc, + struct mptcp_pm_addr_entry *rem, u8 bkup) +{ + struct mptcp_sock *msk; + int ret =3D -EINVAL; + u32 token_val; + + token_val =3D nla_get_u32(token); + + msk =3D mptcp_token_get_sock(net, token_val); + if (!msk) + return ret; + + if (!mptcp_pm_is_userspace(msk)) + goto set_flags_err; + + if (loc->addr.family =3D=3D AF_UNSPEC || + rem->addr.family =3D=3D AF_UNSPEC) + goto set_flags_err; + + lock_sock((struct sock *)msk); + ret =3D mptcp_pm_nl_mp_prio_send_ack(msk, &loc->addr, &rem->addr, bkup); + release_sock((struct sock *)msk); + +set_flags_err: + sock_put((struct sock *)msk); + return ret; +} diff --git a/net/mptcp/protocol.h b/net/mptcp/protocol.h index 033c995772dc..480c5320b86e 100644 --- a/net/mptcp/protocol.h +++ b/net/mptcp/protocol.h @@ -772,6 +772,10 @@ void mptcp_pm_rm_addr_received(struct mptcp_sock *msk, const struct mptcp_rm_list *rm_list); void mptcp_pm_mp_prio_received(struct sock *sk, u8 bkup); void mptcp_pm_mp_fail_received(struct sock *sk, u64 fail_seq); +int mptcp_pm_nl_mp_prio_send_ack(struct mptcp_sock *msk, + struct mptcp_addr_info *addr, + struct mptcp_addr_info *rem, + u8 bkup); bool mptcp_pm_alloc_anno_list(struct mptcp_sock *msk, const struct mptcp_pm_addr_entry *entry); void mptcp_pm_free_anno_list(struct mptcp_sock *msk); @@ -788,7 +792,9 @@ int mptcp_pm_get_flags_and_ifindex_by_id(struct mptcp_s= ock *msk, int mptcp_userspace_pm_get_flags_and_ifindex_by_id(struct mptcp_sock *msk, unsigned int id, u8 *flags, int *ifindex); - +int mptcp_userspace_pm_set_flags(struct net *net, struct nlattr *token, + struct mptcp_pm_addr_entry *loc, + struct mptcp_pm_addr_entry *rem, u8 bkup); int mptcp_pm_announce_addr(struct mptcp_sock *msk, const struct mptcp_addr_info *addr, bool echo); --=20 2.31.1