commit 188048bce311ee41e5178bc3255415d0eae28423
author Alex Elder <elder@inktank.com> Wed Jun 20 21:53:53 2012 -0500
committer Sage Weil <sage@inktank.com> Thu Jul 05 21:14:20 2012 -0700
tree 664eb69384323440f984f1fcd96075753fc1b286
parent a8d00e3cdef4c1c4f194414b72b24cd995439a05

libceph: don't change socket state on sock event

Currently the socket state change event handler records an error
message on a connection to distinguish a close while connecting from
a close while a connection was already established.

Changing connection information during handling of a socket event is
not very clean, so instead move this assignment inside con_work(),
where it can be done during normal connection-level processing (and
under protection of the connection mutex as well).

Move the handling of a socket closed event up to the top of the
processing loop in con_work(); there's no point in handling backoff
etc. if we have a newly-closed socket to take care of.

Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Sage Weil <sage@inktank.com>