tree 664eb69384323440f984f1fcd96075753fc1b286
parent a8d00e3cdef4c1c4f194414b72b24cd995439a05
author Alex Elder <elder@inktank.com> 1340247233 -0500
committer Sage Weil <sage@inktank.com> 1341548060 -0700

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>