Skip to content

Commit 106b6db

Browse files
Geliang Tangintel-lab-lkp
authored andcommitted
mptcp: pm: drop get_local_id helpers
Now mptcp_pm_nl_get_local_id() and mptcp_userspace_pm_get_local_id() helpers can be dropped, and mptcp_pm_get_local_id() can directly invoke get_local_id() interface through "ops" of "msk->pm". Signed-off-by: Geliang Tang <[email protected]>
1 parent 2ecfbbf commit 106b6db

File tree

4 files changed

+5
-11
lines changed

4 files changed

+5
-11
lines changed

net/mptcp/pm.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -425,9 +425,7 @@ int mptcp_pm_get_local_id(struct mptcp_sock *msk, struct sock_common *skc)
425425
skc_local.addr.id = 0;
426426
skc_local.flags = MPTCP_PM_ADDR_FLAG_IMPLICIT;
427427

428-
if (mptcp_pm_is_userspace(msk))
429-
return mptcp_userspace_pm_get_local_id(msk, &skc_local);
430-
return mptcp_pm_nl_get_local_id(msk, &skc_local);
428+
return msk->pm.ops->get_local_id(msk, &skc_local);
431429
}
432430

433431
bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc)

net/mptcp/pm_netlink.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1139,8 +1139,8 @@ static int mptcp_pm_nl_create_listen_socket(struct sock *sk,
11391139
return err;
11401140
}
11411141

1142-
int mptcp_pm_nl_get_local_id(struct mptcp_sock *msk,
1143-
struct mptcp_pm_addr_entry *skc)
1142+
static int mptcp_pm_nl_get_local_id(struct mptcp_sock *msk,
1143+
struct mptcp_pm_addr_entry *skc)
11441144
{
11451145
struct mptcp_pm_addr_entry *entry;
11461146
struct pm_nl_pernet *pernet;

net/mptcp/pm_userspace.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -130,8 +130,8 @@ mptcp_userspace_pm_lookup_addr_by_id(struct mptcp_sock *msk, unsigned int id)
130130
return NULL;
131131
}
132132

133-
int mptcp_userspace_pm_get_local_id(struct mptcp_sock *msk,
134-
struct mptcp_pm_addr_entry *skc)
133+
static int mptcp_userspace_pm_get_local_id(struct mptcp_sock *msk,
134+
struct mptcp_pm_addr_entry *skc)
135135
{
136136
__be16 msk_sport = ((struct inet_sock *)
137137
inet_sk((struct sock *)msk))->inet_sport;

net/mptcp/protocol.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1136,10 +1136,6 @@ bool mptcp_pm_add_addr_signal(struct mptcp_sock *msk, const struct sk_buff *skb,
11361136
bool mptcp_pm_rm_addr_signal(struct mptcp_sock *msk, unsigned int remaining,
11371137
struct mptcp_rm_list *rm_list);
11381138
int mptcp_pm_get_local_id(struct mptcp_sock *msk, struct sock_common *skc);
1139-
int mptcp_pm_nl_get_local_id(struct mptcp_sock *msk,
1140-
struct mptcp_pm_addr_entry *skc);
1141-
int mptcp_userspace_pm_get_local_id(struct mptcp_sock *msk,
1142-
struct mptcp_pm_addr_entry *skc);
11431139
bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc);
11441140
bool mptcp_pm_nl_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);
11451141
bool mptcp_userspace_pm_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);

0 commit comments

Comments
 (0)