Commit 732720fa authored by Ursula Braun's avatar Ursula Braun Committed by David S. Miller

net/smc: wake up wr_reg_wait when terminating a link group

If a new connection with a new rmb is added to a link group, its
memory region is registered. If a link group is terminated, a pending
registration requires a wake up.

And consolidate setting of tx_flag peer_conn_abort in smc_lgr_terminate().
Signed-off-by: default avatarUrsula Braun <ubraun@linux.vnet.ibm.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 610db66f
...@@ -89,7 +89,7 @@ static int smc_close_abort(struct smc_connection *conn) ...@@ -89,7 +89,7 @@ static int smc_close_abort(struct smc_connection *conn)
} }
/* terminate smc socket abnormally - active abort /* terminate smc socket abnormally - active abort
* RDMA communication no longer possible * link group is terminated, i.e. RDMA communication no longer possible
*/ */
static void smc_close_active_abort(struct smc_sock *smc) static void smc_close_active_abort(struct smc_sock *smc)
{ {
...@@ -113,7 +113,6 @@ static void smc_close_active_abort(struct smc_sock *smc) ...@@ -113,7 +113,6 @@ static void smc_close_active_abort(struct smc_sock *smc)
break; break;
case SMC_APPCLOSEWAIT1: case SMC_APPCLOSEWAIT1:
case SMC_APPCLOSEWAIT2: case SMC_APPCLOSEWAIT2:
txflags->peer_conn_abort = 1;
sock_release(smc->clcsock); sock_release(smc->clcsock);
if (!smc_cdc_rxed_any_close(&smc->conn)) if (!smc_cdc_rxed_any_close(&smc->conn))
sk->sk_state = SMC_PEERABORTWAIT; sk->sk_state = SMC_PEERABORTWAIT;
...@@ -127,7 +126,6 @@ static void smc_close_active_abort(struct smc_sock *smc) ...@@ -127,7 +126,6 @@ static void smc_close_active_abort(struct smc_sock *smc)
case SMC_PEERCLOSEWAIT2: case SMC_PEERCLOSEWAIT2:
if (!txflags->peer_conn_closed) { if (!txflags->peer_conn_closed) {
sk->sk_state = SMC_PEERABORTWAIT; sk->sk_state = SMC_PEERABORTWAIT;
txflags->peer_conn_abort = 1;
sock_release(smc->clcsock); sock_release(smc->clcsock);
} else { } else {
sk->sk_state = SMC_CLOSED; sk->sk_state = SMC_CLOSED;
...@@ -135,10 +133,8 @@ static void smc_close_active_abort(struct smc_sock *smc) ...@@ -135,10 +133,8 @@ static void smc_close_active_abort(struct smc_sock *smc)
break; break;
case SMC_PROCESSABORT: case SMC_PROCESSABORT:
case SMC_APPFINCLOSEWAIT: case SMC_APPFINCLOSEWAIT:
if (!txflags->peer_conn_closed) { if (!txflags->peer_conn_closed)
txflags->peer_conn_abort = 1;
sock_release(smc->clcsock); sock_release(smc->clcsock);
}
sk->sk_state = SMC_CLOSED; sk->sk_state = SMC_CLOSED;
break; break;
case SMC_PEERFINCLOSEWAIT: case SMC_PEERFINCLOSEWAIT:
...@@ -303,8 +299,9 @@ static void smc_close_passive_abort_received(struct smc_sock *smc) ...@@ -303,8 +299,9 @@ static void smc_close_passive_abort_received(struct smc_sock *smc)
} }
} }
/* Some kind of closing has been received: peer_conn_closed, peer_conn_abort, /* Either some kind of closing has been received: peer_conn_closed,
* or peer_done_writing. * peer_conn_abort, or peer_done_writing
* or the link group of the connection terminates abnormally.
*/ */
static void smc_close_passive_work(struct work_struct *work) static void smc_close_passive_work(struct work_struct *work)
{ {
...@@ -327,6 +324,7 @@ static void smc_close_passive_work(struct work_struct *work) ...@@ -327,6 +324,7 @@ static void smc_close_passive_work(struct work_struct *work)
rxflags = &conn->local_rx_ctrl.conn_state_flags; rxflags = &conn->local_rx_ctrl.conn_state_flags;
if (rxflags->peer_conn_abort) { if (rxflags->peer_conn_abort) {
/* peer has not received all data */
smc_close_passive_abort_received(smc); smc_close_passive_abort_received(smc);
release_sock(&smc->sk); release_sock(&smc->sk);
cancel_delayed_work_sync(&conn->tx_work); cancel_delayed_work_sync(&conn->tx_work);
......
...@@ -234,9 +234,7 @@ static void smc_buf_unuse(struct smc_connection *conn) ...@@ -234,9 +234,7 @@ static void smc_buf_unuse(struct smc_connection *conn)
/* remove a finished connection from its link group */ /* remove a finished connection from its link group */
void smc_conn_free(struct smc_connection *conn) void smc_conn_free(struct smc_connection *conn)
{ {
struct smc_link_group *lgr = conn->lgr; if (!conn->lgr)
if (!lgr)
return; return;
smc_cdc_tx_dismiss_slots(conn); smc_cdc_tx_dismiss_slots(conn);
smc_lgr_unregister_conn(conn); smc_lgr_unregister_conn(conn);
...@@ -331,12 +329,16 @@ void smc_lgr_terminate(struct smc_link_group *lgr) ...@@ -331,12 +329,16 @@ void smc_lgr_terminate(struct smc_link_group *lgr)
conn = rb_entry(node, struct smc_connection, alert_node); conn = rb_entry(node, struct smc_connection, alert_node);
smc = container_of(conn, struct smc_sock, conn); smc = container_of(conn, struct smc_sock, conn);
sock_hold(&smc->sk); sock_hold(&smc->sk);
conn->local_tx_ctrl.conn_state_flags.peer_conn_abort = 1;
__smc_lgr_unregister_conn(conn); __smc_lgr_unregister_conn(conn);
write_unlock_bh(&lgr->conns_lock);
schedule_work(&conn->close_work); schedule_work(&conn->close_work);
write_lock_bh(&lgr->conns_lock);
sock_put(&smc->sk); sock_put(&smc->sk);
node = rb_first(&lgr->conns_all); node = rb_first(&lgr->conns_all);
} }
write_unlock_bh(&lgr->conns_lock); write_unlock_bh(&lgr->conns_lock);
wake_up(&lgr->lnk[SMC_SINGLE_LINK].wr_reg_wait);
} }
/* Determine vlan of internal TCP socket. /* Determine vlan of internal TCP socket.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment