net/smc: sockopts TCP_NODELAY and TCP_CORK
Setting sockopt TCP_NODELAY or resetting sockopt TCP_CORK triggers data transfer. For a corked SMC socket RDMA writes are deferred, if there is still sufficient send buffer space available. Signed-off-by: Ursula Braun <ubraun@linux.ibm.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
ee9dfbef02
commit
01d2f7e2cd
@ -1291,7 +1291,7 @@ static int smc_setsockopt(struct socket *sock, int level, int optname,
|
|||||||
{
|
{
|
||||||
struct sock *sk = sock->sk;
|
struct sock *sk = sock->sk;
|
||||||
struct smc_sock *smc;
|
struct smc_sock *smc;
|
||||||
int rc;
|
int val, rc;
|
||||||
|
|
||||||
smc = smc_sk(sk);
|
smc = smc_sk(sk);
|
||||||
|
|
||||||
@ -1307,6 +1307,10 @@ static int smc_setsockopt(struct socket *sock, int level, int optname,
|
|||||||
if (rc)
|
if (rc)
|
||||||
return rc;
|
return rc;
|
||||||
|
|
||||||
|
if (optlen < sizeof(int))
|
||||||
|
return rc;
|
||||||
|
get_user(val, (int __user *)optval);
|
||||||
|
|
||||||
lock_sock(sk);
|
lock_sock(sk);
|
||||||
switch (optname) {
|
switch (optname) {
|
||||||
case TCP_ULP:
|
case TCP_ULP:
|
||||||
@ -1322,6 +1326,20 @@ static int smc_setsockopt(struct socket *sock, int level, int optname,
|
|||||||
rc = -EINVAL;
|
rc = -EINVAL;
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
|
case TCP_NODELAY:
|
||||||
|
if (sk->sk_state != SMC_INIT && sk->sk_state != SMC_LISTEN) {
|
||||||
|
if (val)
|
||||||
|
mod_delayed_work(system_wq, &smc->conn.tx_work,
|
||||||
|
0);
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
case TCP_CORK:
|
||||||
|
if (sk->sk_state != SMC_INIT && sk->sk_state != SMC_LISTEN) {
|
||||||
|
if (!val)
|
||||||
|
mod_delayed_work(system_wq, &smc->conn.tx_work,
|
||||||
|
0);
|
||||||
|
}
|
||||||
|
break;
|
||||||
default:
|
default:
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
@ -19,6 +19,7 @@
|
|||||||
#include <linux/sched/signal.h>
|
#include <linux/sched/signal.h>
|
||||||
|
|
||||||
#include <net/sock.h>
|
#include <net/sock.h>
|
||||||
|
#include <net/tcp.h>
|
||||||
|
|
||||||
#include "smc.h"
|
#include "smc.h"
|
||||||
#include "smc_wr.h"
|
#include "smc_wr.h"
|
||||||
@ -26,6 +27,7 @@
|
|||||||
#include "smc_tx.h"
|
#include "smc_tx.h"
|
||||||
|
|
||||||
#define SMC_TX_WORK_DELAY HZ
|
#define SMC_TX_WORK_DELAY HZ
|
||||||
|
#define SMC_TX_CORK_DELAY (HZ >> 2) /* 250 ms */
|
||||||
|
|
||||||
/***************************** sndbuf producer *******************************/
|
/***************************** sndbuf producer *******************************/
|
||||||
|
|
||||||
@ -115,6 +117,13 @@ static int smc_tx_wait_memory(struct smc_sock *smc, int flags)
|
|||||||
return rc;
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static bool smc_tx_is_corked(struct smc_sock *smc)
|
||||||
|
{
|
||||||
|
struct tcp_sock *tp = tcp_sk(smc->clcsock->sk);
|
||||||
|
|
||||||
|
return (tp->nonagle & TCP_NAGLE_CORK) ? true : false;
|
||||||
|
}
|
||||||
|
|
||||||
/* sndbuf producer: main API called by socket layer.
|
/* sndbuf producer: main API called by socket layer.
|
||||||
* called under sock lock.
|
* called under sock lock.
|
||||||
*/
|
*/
|
||||||
@ -209,7 +218,16 @@ int smc_tx_sendmsg(struct smc_sock *smc, struct msghdr *msg, size_t len)
|
|||||||
/* since we just produced more new data into sndbuf,
|
/* since we just produced more new data into sndbuf,
|
||||||
* trigger sndbuf consumer: RDMA write into peer RMBE and CDC
|
* trigger sndbuf consumer: RDMA write into peer RMBE and CDC
|
||||||
*/
|
*/
|
||||||
smc_tx_sndbuf_nonempty(conn);
|
if ((msg->msg_flags & MSG_MORE || smc_tx_is_corked(smc)) &&
|
||||||
|
(atomic_read(&conn->sndbuf_space) >
|
||||||
|
(conn->sndbuf_size >> 1)))
|
||||||
|
/* for a corked socket defer the RDMA writes if there
|
||||||
|
* is still sufficient sndbuf_space available
|
||||||
|
*/
|
||||||
|
schedule_delayed_work(&conn->tx_work,
|
||||||
|
SMC_TX_CORK_DELAY);
|
||||||
|
else
|
||||||
|
smc_tx_sndbuf_nonempty(conn);
|
||||||
} /* while (msg_data_left(msg)) */
|
} /* while (msg_data_left(msg)) */
|
||||||
|
|
||||||
return send_done;
|
return send_done;
|
||||||
@ -409,8 +427,8 @@ int smc_tx_sndbuf_nonempty(struct smc_connection *conn)
|
|||||||
}
|
}
|
||||||
rc = 0;
|
rc = 0;
|
||||||
if (conn->alert_token_local) /* connection healthy */
|
if (conn->alert_token_local) /* connection healthy */
|
||||||
schedule_delayed_work(&conn->tx_work,
|
mod_delayed_work(system_wq, &conn->tx_work,
|
||||||
SMC_TX_WORK_DELAY);
|
SMC_TX_WORK_DELAY);
|
||||||
}
|
}
|
||||||
goto out_unlock;
|
goto out_unlock;
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user