Skip to content

Commit 9368fd6

Browse files
olgakorn1Trond Myklebust
authored andcommitted
SUNRPC enable back offline transports in trunking discovery
When we are adding a transport to a xprt_switch that's already on the list but has been marked OFFLINE, then make the state ONLINE since it's been tested now. Signed-off-by: Olga Kornievskaia <[email protected]> Signed-off-by: Trond Myklebust <[email protected]>
1 parent 95d0d30 commit 9368fd6

File tree

2 files changed

+15
-0
lines changed

2 files changed

+15
-0
lines changed

include/linux/sunrpc/clnt.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -242,6 +242,7 @@ void rpc_clnt_xprt_switch_put(struct rpc_clnt *);
242242
void rpc_clnt_xprt_switch_add_xprt(struct rpc_clnt *, struct rpc_xprt *);
243243
bool rpc_clnt_xprt_switch_has_addr(struct rpc_clnt *clnt,
244244
const struct sockaddr *sap);
245+
void rpc_clnt_xprt_set_online(struct rpc_clnt *clnt, struct rpc_xprt *xprt);
245246
void rpc_cleanup_clids(void);
246247

247248
static inline int rpc_reply_expected(struct rpc_task *task)

net/sunrpc/clnt.c

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3095,8 +3095,22 @@ void rpc_clnt_xprt_switch_put(struct rpc_clnt *clnt)
30953095
}
30963096
EXPORT_SYMBOL_GPL(rpc_clnt_xprt_switch_put);
30973097

3098+
void rpc_clnt_xprt_set_online(struct rpc_clnt *clnt, struct rpc_xprt *xprt)
3099+
{
3100+
struct rpc_xprt_switch *xps;
3101+
3102+
rcu_read_lock();
3103+
xps = rcu_dereference(clnt->cl_xpi.xpi_xpswitch);
3104+
rcu_read_unlock();
3105+
xprt_set_online_locked(xprt, xps);
3106+
}
3107+
30983108
void rpc_clnt_xprt_switch_add_xprt(struct rpc_clnt *clnt, struct rpc_xprt *xprt)
30993109
{
3110+
if (rpc_clnt_xprt_switch_has_addr(clnt,
3111+
(const struct sockaddr *)&xprt->addr)) {
3112+
return rpc_clnt_xprt_set_online(clnt, xprt);
3113+
}
31003114
rcu_read_lock();
31013115
rpc_xprt_switch_add_xprt(rcu_dereference(clnt->cl_xpi.xpi_xpswitch),
31023116
xprt);

0 commit comments

Comments
 (0)