net/smc: Send directly when TCP_CORK is cleared
authorTony Lu <tonylu@linux.alibaba.com>
Sun, 30 Jan 2022 18:02:55 +0000 (02:02 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 31 Jan 2022 15:08:20 +0000 (15:08 +0000)
commitea785a1a573b390a150010b3c5b81e1ccd8c98a8
tree08b9e9691157c96edd94279a6625591a27bcd05d
parent01b2a995156d11166da00ce254d59bd7f7cefb92
net/smc: Send directly when TCP_CORK is cleared

According to the man page of TCP_CORK [1], if set, don't send out
partial frames. All queued partial frames are sent when option is
cleared again.

When applications call setsockopt to disable TCP_CORK, this call is
protected by lock_sock(), and tries to mod_delayed_work() to 0, in order
to send pending data right now. However, the delayed work smc_tx_work is
also protected by lock_sock(). There introduces lock contention for
sending data.

To fix it, send pending data directly which acts like TCP, without
lock_sock() protected in the context of setsockopt (already lock_sock()ed),
and cancel unnecessary dealyed work, which is protected by lock.

[1] https://linux.die.net/man/7/tcp

Signed-off-by: Tony Lu <tonylu@linux.alibaba.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/smc/af_smc.c
net/smc/smc_tx.c
net/smc/smc_tx.h