[MPTCP] Re: [PATCH mptcp 2/7] mptcp: move pm netlink work into pm_netlink

[MPTCP] Re: [PATCH mptcp 2/7] mptcp: move pm netlink work into pm_netlink

[Mat Martineau]

[-- Attachment #1: Type: text/plain, Size: 518 bytes --]

On Thu, 14 Jan 2021, Florian Westphal wrote:

> allows to make some functions static and avoids aquire of the pm spinlock
> in protocol.c.
>
> Signed-off-by: Florian Westphal <fw(a)strlen.de>
> ---
> net/mptcp/pm_netlink.c | 45 +++++++++++++++++++++++++++++++++++++-----
> net/mptcp/protocol.c   | 33 +------------------------------
> net/mptcp/protocol.h   |  6 +-----
> 3 files changed, 42 insertions(+), 42 deletions(-)
>

Also ok with me to apply this to the export branch.

--
Mat Martineau
Intel

[MPTCP] Re: [PATCH mptcp 2/7] mptcp: move pm netlink work into pm_netlink

[Matthieu Baerts]

[-- Attachment #1: Type: text/plain, Size: 1032 bytes --]

Hi Florian, Mat,

On 20/01/2021 01:27, Mat Martineau wrote:
> On Thu, 14 Jan 2021, Florian Westphal wrote:
> 
>> allows to make some functions static and avoids aquire of the pm spinlock
>> in protocol.c.
>>
>> Signed-off-by: Florian Westphal <fw(a)strlen.de>
>> ---
>> net/mptcp/pm_netlink.c | 45 +++++++++++++++++++++++++++++++++++++-----
>> net/mptcp/protocol.c   | 33 +------------------------------
>> net/mptcp/protocol.h   |  6 +-----
>> 3 files changed, 42 insertions(+), 42 deletions(-)
>>
> 
> Also ok with me to apply this to the export branch.

Thank you for the patches and the reviews!

Just added the two first patches in our tree with Mat's RvB tag (and I 
fixed a typo in the commit message reported by codespell):

- 168b68baccfa: mptcp: pm: add lockdep assertions
- 6df8f68967a5: mptcp: move pm netlink work into pm_netlink
- Results: a1d71a555f36..44215ba71300

Tests + export have been queued!

Cheers,
Matt
-- 
Tessares | Belgium | Hybrid Access Solutions
www.tessares.net