Skip to content

Commit 1ca9569

Browse files
Geliang Tangintel-lab-lkp
authored andcommitted
mptcp: pm: drop is_backup helpers
Now mptcp_pm_nl_is_backup() and mptcp_userspace_pm_is_backup() helpers can be dropped, and mptcp_pm_is_backup() can directly invoke get_priority() interface through "ops" of "msk->pm". Signed-off-by: Geliang Tang <[email protected]>
1 parent 10dfe5f commit 1ca9569

File tree

4 files changed

+5
-23
lines changed

4 files changed

+5
-23
lines changed

net/mptcp/pm.c

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -447,13 +447,14 @@ int mptcp_pm_get_local_id(struct mptcp_sock *msk, struct sock_common *skc)
447447
bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc)
448448
{
449449
struct mptcp_addr_info skc_local;
450+
struct mptcp_pm_param param;
450451

451452
mptcp_local_address((struct sock_common *)skc, &skc_local);
452453

453-
if (mptcp_pm_is_userspace(msk))
454-
return mptcp_userspace_pm_is_backup(msk, &skc_local);
455-
456-
return mptcp_pm_nl_is_backup(msk, &skc_local);
454+
if (!msk->pm.ops || !msk->pm.ops->get_priority)
455+
return -ENOTSUPP;
456+
mptcp_pm_param_set_contexts(&param, NULL, &skc_local);
457+
return msk->pm.ops->get_priority(msk, &param);
457458
}
458459

459460
static int mptcp_pm_get_addr(u8 id, struct mptcp_pm_addr_entry *addr,

net/mptcp/pm_netlink.c

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1188,14 +1188,6 @@ static bool mptcp_pm_nl_get_priority(struct mptcp_sock *msk,
11881188
return backup;
11891189
}
11901190

1191-
bool mptcp_pm_nl_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc)
1192-
{
1193-
struct mptcp_pm_param param;
1194-
1195-
mptcp_pm_param_set_contexts(&param, NULL, skc);
1196-
return mptcp_pm_nl_get_priority(msk, &param);
1197-
}
1198-
11991191
#define MPTCP_PM_CMD_GRP_OFFSET 0
12001192
#define MPTCP_PM_EV_GRP_OFFSET 1
12011193

net/mptcp/pm_userspace.c

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -165,15 +165,6 @@ static bool mptcp_userspace_pm_get_priority(struct mptcp_sock *msk,
165165
return backup;
166166
}
167167

168-
bool mptcp_userspace_pm_is_backup(struct mptcp_sock *msk,
169-
struct mptcp_addr_info *skc)
170-
{
171-
struct mptcp_pm_param param;
172-
173-
mptcp_pm_param_set_contexts(&param, NULL, skc);
174-
return mptcp_userspace_pm_get_priority(msk, &param);
175-
}
176-
177168
static struct mptcp_sock *mptcp_userspace_pm_get_sock(const struct genl_info *info)
178169
{
179170
struct mptcp_sock *msk;

net/mptcp/protocol.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1130,8 +1130,6 @@ void mptcp_pm_param_set_contexts(struct mptcp_pm_param *param,
11301130
const struct mptcp_addr_info *addr);
11311131
int mptcp_pm_get_local_id(struct mptcp_sock *msk, struct sock_common *skc);
11321132
bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc);
1133-
bool mptcp_pm_nl_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);
1134-
bool mptcp_userspace_pm_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);
11351133
int mptcp_pm_nl_dump_addr(struct sk_buff *msg,
11361134
struct netlink_callback *cb);
11371135
int mptcp_userspace_pm_dump_addr(struct sk_buff *msg,

0 commit comments

Comments
 (0)