On Fri, Jul 20, 2012 at 5:41 PM, Sage Weil <sage@xxxxxxxxxxx> wrote: > When we detect a mds session reset, close the old ceph_connection before > reopening it. This ensures we clean up the old socket properly and keep > the ceph_connection state correct. > > Signed-off-by: Sage Weil <sage@xxxxxxxxxxx> > --- > fs/ceph/mds_client.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/fs/ceph/mds_client.c b/fs/ceph/mds_client.c > index 39b76d6..a5a7354 100644 > --- a/fs/ceph/mds_client.c > +++ b/fs/ceph/mds_client.c > @@ -2518,6 +2518,7 @@ static void send_mds_reconnect(struct ceph_mds_client *mdsc, > session->s_state = CEPH_MDS_SESSION_RECONNECTING; > session->s_seq = 0; > > + ceph_con_close(&session->s_con); > ceph_con_open(&session->s_con, Should we also BUG_ON in ceph_con_open if connection is not closed? > CEPH_ENTITY_TYPE_MDS, mds, > ceph_mdsmap_get_addr(mdsc->mdsmap, mds)); > -- > 1.7.9 > > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html