SUNRPC: Revert e0a912e8ddba
authorChuck Lever <chuck.lever@oracle.com>
Wed, 19 Jul 2023 18:31:22 +0000 (14:31 -0400)
committerChuck Lever <chuck.lever@oracle.com>
Tue, 29 Aug 2023 21:45:22 +0000 (17:45 -0400)
commit89d2d9fbeadcbdbd6302d3d0cd6bfbe219d85b68
tree5771cfbd5ce1a2fc2c16e95ba286d126e7ea799c
parentbaabf59c24145612e4a975f459a5024389f13f5d
SUNRPC: Revert e0a912e8ddba

Flamegraph analysis showed that the cork/uncork calls consume
nearly a third of the CPU time spent in svc_tcp_sendto(). The
other two consumers are mutex lock/unlock and svc_tcp_sendmsg().

Now that svc_tcp_sendto() coalesces RPC messages properly, there
is no need to introduce artificial delays to prevent sending
partial messages.

After applying this change, I measured a 1.2K read IOPS increase
for 8KB random I/O (several percent) on 56Gb IP over IB.

Reviewed-by: David Howells <dhowells@redhat.com>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
include/linux/sunrpc/svcsock.h
net/sunrpc/svcsock.c