diff options
author | Alex Elder <elder@inktank.com> | 2012-06-20 22:53:53 -0400 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2012-07-06 00:14:24 -0400 |
commit | bb9e6bba5d8b85b631390f8dbe8a24ae1ff5b48a (patch) | |
tree | e3923abfae2f2cb8af685dfb829a45df33e45779 /net/ceph | |
parent | 456ea46865787283088b23a8a7f69244513b95f0 (diff) |
libceph: clear CONNECTING in ceph_con_close()
A connection that is closed will no longer be connecting. So
clear the CONNECTING state bit in ceph_con_close(). Similarly,
if the socket has been closed we no longer are in connecting
state (a new connect sequence will need to be initiated).
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>
Diffstat (limited to 'net/ceph')
-rw-r--r-- | net/ceph/messenger.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/ceph/messenger.c b/net/ceph/messenger.c index cfcca1f5be67..beee382d784e 100644 --- a/net/ceph/messenger.c +++ b/net/ceph/messenger.c | |||
@@ -462,6 +462,7 @@ void ceph_con_close(struct ceph_connection *con) | |||
462 | dout("con_close %p peer %s\n", con, | 462 | dout("con_close %p peer %s\n", con, |
463 | ceph_pr_addr(&con->peer_addr.in_addr)); | 463 | ceph_pr_addr(&con->peer_addr.in_addr)); |
464 | clear_bit(NEGOTIATING, &con->state); | 464 | clear_bit(NEGOTIATING, &con->state); |
465 | clear_bit(CONNECTING, &con->state); | ||
465 | clear_bit(STANDBY, &con->state); /* avoid connect_seq bump */ | 466 | clear_bit(STANDBY, &con->state); /* avoid connect_seq bump */ |
466 | set_bit(CLOSED, &con->state); | 467 | set_bit(CLOSED, &con->state); |
467 | 468 | ||
@@ -2189,7 +2190,7 @@ static void con_work(struct work_struct *work) | |||
2189 | mutex_lock(&con->mutex); | 2190 | mutex_lock(&con->mutex); |
2190 | restart: | 2191 | restart: |
2191 | if (test_and_clear_bit(SOCK_CLOSED, &con->flags)) { | 2192 | if (test_and_clear_bit(SOCK_CLOSED, &con->flags)) { |
2192 | if (test_bit(CONNECTING, &con->state)) | 2193 | if (test_and_clear_bit(CONNECTING, &con->state)) |
2193 | con->error_msg = "connection failed"; | 2194 | con->error_msg = "connection failed"; |
2194 | else | 2195 | else |
2195 | con->error_msg = "socket closed"; | 2196 | con->error_msg = "socket closed"; |