Skip to content

Commit 10dfe5f

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 ced2b08 commit 10dfe5f

File tree

4 files changed

+7
-11
lines changed

4 files changed

+7
-11
lines changed

net/mptcp/pm.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -438,10 +438,10 @@ int mptcp_pm_get_local_id(struct mptcp_sock *msk, struct sock_common *skc)
438438
skc_local.addr.id = 0;
439439
skc_local.flags = MPTCP_PM_ADDR_FLAG_IMPLICIT;
440440

441+
if (!msk->pm.ops || !msk->pm.ops->get_local_id)
442+
return -ENOTSUPP;
441443
mptcp_pm_param_set_contexts(&param, &skc_local, NULL);
442-
if (mptcp_pm_is_userspace(msk))
443-
return mptcp_userspace_pm_get_local_id(msk, &param);
444-
return mptcp_pm_nl_get_local_id(msk, &param);
444+
return msk->pm.ops->get_local_id(msk, &param);
445445
}
446446

447447
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_param *param)
1142+
static int mptcp_pm_nl_get_local_id(struct mptcp_sock *msk,
1143+
struct mptcp_pm_param *param)
11441144
{
11451145
struct mptcp_pm_addr_entry *skc = &param->entry;
11461146
struct mptcp_pm_addr_entry *entry;

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_param *param)
133+
static int mptcp_userspace_pm_get_local_id(struct mptcp_sock *msk,
134+
struct mptcp_pm_param *param)
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
@@ -1129,10 +1129,6 @@ void mptcp_pm_param_set_contexts(struct mptcp_pm_param *param,
11291129
const struct mptcp_pm_addr_entry *entry,
11301130
const struct mptcp_addr_info *addr);
11311131
int mptcp_pm_get_local_id(struct mptcp_sock *msk, struct sock_common *skc);
1132-
int mptcp_pm_nl_get_local_id(struct mptcp_sock *msk,
1133-
struct mptcp_pm_param *param);
1134-
int mptcp_userspace_pm_get_local_id(struct mptcp_sock *msk,
1135-
struct mptcp_pm_param *param);
11361132
bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc);
11371133
bool mptcp_pm_nl_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);
11381134
bool mptcp_userspace_pm_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);

0 commit comments

Comments
 (0)