SUNRPC: Add helpers to prevent socket create from racing
authorTrond Myklebust <trond.myklebust@primarydata.com>
Sun, 8 Feb 2015 23:19:25 +0000 (18:19 -0500)
committerTrond Myklebust <trond.myklebust@primarydata.com>
Mon, 9 Feb 2015 02:47:29 +0000 (21:47 -0500)
commit718ba5b87343df303017585200ee182e937eabfc
tree445d5bbb8aa4173b00527852b76d1e210fbb78c5
parent6cc7e908362a9dfec3c821f77ec98b6758592060
SUNRPC: Add helpers to prevent socket create from racing

The socket lock is currently held by the task that is requesting the
connection be established. While that is efficient in the case where
the connection happens quickly, it is racy in the case where it doesn't.
What we really want is for the connect helper to be able to block access
to the socket while it is being set up.

This patch does so by arranging to transfer the socket lock from the
task that is requesting the connect attempt, and then releasing that
lock once everything is done.
This scheme also gives us automatic protection against collisions with
the RPC close code, so we can kill the cancel_delayed_work_sync()
call in xs_close().

Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
include/linux/sunrpc/xprt.h
net/sunrpc/xprt.c
net/sunrpc/xprtsock.c