diff options
| author | Sage Weil <sage@inktank.com> | 2012-06-09 14:19:21 -0700 |
|---|---|---|
| committer | Sage Weil <sage@inktank.com> | 2012-06-15 11:26:37 -0700 |
| commit | 89a86be0ce20022f6ede8bccec078dbb3d63caaa (patch) | |
| tree | 8f0f125b5f094d647c77eef45b7e33f6d0ab2b15 /net/ceph | |
| parent | a5506049500b30dbc5edb4d07a3577477c1f3643 (diff) | |
| download | blackbird-op-linux-89a86be0ce20022f6ede8bccec078dbb3d63caaa.tar.gz blackbird-op-linux-89a86be0ce20022f6ede8bccec078dbb3d63caaa.zip | |
libceph: transition socket state prior to actual connect
Once we call ->connect(), we are racing against the actual
connection, and a subsequent transition from CONNECTING ->
CONNECTED. Set the state to CONNECTING before that, under the
protection of the mutex, to avoid the race.
This was introduced in 928443cd9644e7cfd46f687dbeffda2d1a357ff9,
with the original socket state code.
Signed-off-by: Sage Weil <sage@inktank.com>
Reviewed-by: Alex Elder <elder@inktank.com>
Diffstat (limited to 'net/ceph')
| -rw-r--r-- | net/ceph/messenger.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c index 769a2c9fe1af..bdbecac2d69d 100644 --- a/net/ceph/messenger.c +++ b/net/ceph/messenger.c @@ -321,6 +321,7 @@ static int ceph_tcp_connect(struct ceph_connection *con) dout("connect %s\n", ceph_pr_addr(&con->peer_addr.in_addr)); + con_sock_state_connecting(con); ret = sock->ops->connect(sock, (struct sockaddr *)paddr, sizeof(*paddr), O_NONBLOCK); if (ret == -EINPROGRESS) { @@ -336,8 +337,6 @@ static int ceph_tcp_connect(struct ceph_connection *con) return ret; } con->sock = sock; - con_sock_state_connecting(con); - return 0; } |
