From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============7333739872981822134==" MIME-Version: 1.0 From: Geliang Tang To: mptcp at lists.01.org Subject: [MPTCP] Re: [MPTCP][PATCH v5 mptcp-next 0/4] Add REMOVE_ADDR support Date: Mon, 10 Aug 2020 14:24:56 +0800 Message-ID: <20200810062456.GA19671@OptiPlex> In-Reply-To: 3fa8ca3b-58f4-6a6f-ca00-49d034ee3e83@tessares.net X-Status: X-Keywords: X-UID: 5529 --===============7333739872981822134== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Hi Matt, Paolo, On Thu, Aug 06, 2020 at 08:33:04PM +0200, Matthieu Baerts wrote: > Hi Geliang, Paolo, > = > On 04/08/2020 12:01, Geliang Tang wrote: > > v5: > > - merge mptcp_nl_remove_subflow() and mptcp_nl_remove_addr() > > - add cond_resched > > - reduce the indentation level in mptcp_pm_nl_rm_addr_received > > = > > v4: > > - update mptcp_subflow_shutdown()'s args. > > - add rm_id check to make sure we don't shutdown the first subflow. > > - add conn_list empty check. > > - move anno_list to mptcp_pm_data. > > - add a new patch 'mptcp: add remove subflow support'. > > = > > v3: > > - fix memory leak and lock issue in v2. > > - drop alist in v2. > > - fix mptcp_subflow_shutdown's arguments. > > - bzero remote in mptcp_pm_create_subflow_or_signal_addr. > > - add more commit message. > > = > > Geliang Tang (4): > > mptcp: rename addr_signal and the related functions > > mptcp: add the outgoing RM_ADDR support > > mptcp: add the incoming RM_ADDR support > > mptcp: remove addr and subflow in PM netlink > = > Thank you for the patches and the reviews! > = > I just added these patches except the last one in the "export" branch. > = > Tests are still green! > Thanks for applying my patches. The last patch is still have a bug in it, sometimes it will trigger a deadlock. I'll try my best to fix this bug. Then I'll send out the final version of the last patch. Meanwhile, I'm writting the selttest test case for RM_ADDR. It's not finish yet. I'll send out the test patch recently. -Geliang > Cheers, > Matt > -- = > Tessares | Belgium | Hybrid Access Solutions > www.tessares.net --===============7333739872981822134==--