Re: [PATCH mptcp-next] Squash to "mptcp: remote addresses fullmesh"

[Geliang Tang <geliangtang@gmail.com>]

Hi Matt,

Thanks for fixing this.

Acked-by: Geliang Tang <geliangtang@xiaomi.com>

-Geliang

Matthieu Baerts <matthieu.baerts@tessares.net> 于2021年8月4日周三 下午3:37写道：
>
> This fixes 'unused-but-set-variable' warning.
>
> Jenkins Tessares reported [1] this error:
>
>   net/mptcp/pm_netlink.c: In function 'fill_remote_addresses_vec':
>   net/mptcp/pm_netlink.c:435:23: error: variable 'pernet' set but not  used [-Werror=unused-but-set-variable]
>     435 |  struct pm_nl_pernet *pernet;
>         |                       ^~~~~~
>   cc1: all warnings being treated as errors
>
> Indeed, this variable is defined and set but not used. We can remove it.
>
> [1] https://github.com/multipath-tcp/mptcp_net-next/actions/runs/1096568275
>
> Reported-by: Jenkins Tessares <kernel-dev@tessares.net>
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net>
> ---
>
> Notes:
>     to be squashed in "mptcp: remote addresses fullmesh"
>
>  net/mptcp/pm_netlink.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/net/mptcp/pm_netlink.c b/net/mptcp/pm_netlink.c
> index cbef50fff6e8..29ff454411b1 100644
> --- a/net/mptcp/pm_netlink.c
> +++ b/net/mptcp/pm_netlink.c
> @@ -432,11 +432,9 @@ static unsigned int fill_remote_addresses_vec(struct mptcp_sock *msk, bool fullm
>         struct sock *sk = (struct sock *)msk, *ssk;
>         struct mptcp_subflow_context *subflow;
>         struct mptcp_addr_info remote = { 0 };
> -       struct pm_nl_pernet *pernet;
>         unsigned int subflows_max;
>         int i = 0;
>
> -       pernet = net_generic(sock_net(sk), pm_nl_pernet_id);
>         subflows_max = mptcp_pm_get_subflows_max(msk);
>
>         /* Non-fullmesh endpoint, fill in the single entry
> --
> 2.31.1
>
>