Skip to content

Commit 297d091

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 106b6db commit 297d091

File tree

4 files changed

+1
-17
lines changed

4 files changed

+1
-17
lines changed

net/mptcp/pm.c

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -434,10 +434,7 @@ bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc)
434434

435435
mptcp_local_address((struct sock_common *)skc, &skc_local);
436436

437-
if (mptcp_pm_is_userspace(msk))
438-
return mptcp_userspace_pm_is_backup(msk, &skc_local);
439-
440-
return mptcp_pm_nl_is_backup(msk, &skc_local);
437+
return msk->pm.ops->get_priority(msk, &skc_local);
441438
}
442439

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

net/mptcp/pm_netlink.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1184,11 +1184,6 @@ static bool mptcp_pm_nl_get_priority(struct mptcp_sock *msk,
11841184
return backup;
11851185
}
11861186

1187-
bool mptcp_pm_nl_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc)
1188-
{
1189-
return mptcp_pm_nl_get_priority(msk, skc);
1190-
}
1191-
11921187
#define MPTCP_PM_CMD_GRP_OFFSET 0
11931188
#define MPTCP_PM_EV_GRP_OFFSET 1
11941189

net/mptcp/pm_userspace.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -163,12 +163,6 @@ static bool mptcp_userspace_pm_get_priority(struct mptcp_sock *msk,
163163
return backup;
164164
}
165165

166-
bool mptcp_userspace_pm_is_backup(struct mptcp_sock *msk,
167-
struct mptcp_addr_info *skc)
168-
{
169-
return mptcp_userspace_pm_get_priority(msk, skc);
170-
}
171-
172166
static struct mptcp_sock *mptcp_userspace_pm_get_sock(const struct genl_info *info)
173167
{
174168
struct mptcp_sock *msk;

net/mptcp/protocol.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1137,8 +1137,6 @@ 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);
11391139
bool mptcp_pm_is_backup(struct mptcp_sock *msk, struct sock_common *skc);
1140-
bool mptcp_pm_nl_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);
1141-
bool mptcp_userspace_pm_is_backup(struct mptcp_sock *msk, struct mptcp_addr_info *skc);
11421140
int mptcp_pm_nl_dump_addr(struct sk_buff *msg,
11431141
struct netlink_callback *cb);
11441142
int mptcp_userspace_pm_dump_addr(struct sk_buff *msg,

0 commit comments

Comments
 (0)