On Tue, 24 Jul 2012, Yehuda Sadeh wrote: > On Fri, Jul 20, 2012 at 5:41 PM, Sage Weil <[email protected]> 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 <[email protected]> > > --- > > 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?
Yeah, adding that. > > > 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 [email protected] > > 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 [email protected] > 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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
