diff options
author | Yan, Zheng <ukernel@gmail.com> | 2014-09-11 02:25:18 -0400 |
---|---|---|
committer | Ilya Dryomov <idryomov@redhat.com> | 2014-10-14 13:03:24 -0400 |
commit | 656e4382948d4b2c81bdaf707f1400f53eff2625 (patch) | |
tree | b27db1d91f9f4248db216783889216ca9abbfe3f /fs/ceph | |
parent | b9a678994b4a64b1106ab2cf7cfe7cbc10bb6f40 (diff) |
ceph: protect kick_requests() with mdsc->mutex
Signed-off-by: Yan, Zheng <zyan@redhat.com>
Reviewed-by: Sage Weil <sage@redhat.com>
Diffstat (limited to 'fs/ceph')
-rw-r--r-- | fs/ceph/mds_client.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/ceph/mds_client.c b/fs/ceph/mds_client.c index f751fea416e5..267ba4496dc8 100644 --- a/fs/ceph/mds_client.c +++ b/fs/ceph/mds_client.c | |||
@@ -2471,9 +2471,8 @@ static void handle_session(struct ceph_mds_session *session, | |||
2471 | if (session->s_state == CEPH_MDS_SESSION_RECONNECTING) | 2471 | if (session->s_state == CEPH_MDS_SESSION_RECONNECTING) |
2472 | pr_info("mds%d reconnect denied\n", session->s_mds); | 2472 | pr_info("mds%d reconnect denied\n", session->s_mds); |
2473 | remove_session_caps(session); | 2473 | remove_session_caps(session); |
2474 | wake = 1; /* for good measure */ | 2474 | wake = 2; /* for good measure */ |
2475 | wake_up_all(&mdsc->session_close_wq); | 2475 | wake_up_all(&mdsc->session_close_wq); |
2476 | kick_requests(mdsc, mds); | ||
2477 | break; | 2476 | break; |
2478 | 2477 | ||
2479 | case CEPH_SESSION_STALE: | 2478 | case CEPH_SESSION_STALE: |
@@ -2503,6 +2502,8 @@ static void handle_session(struct ceph_mds_session *session, | |||
2503 | if (wake) { | 2502 | if (wake) { |
2504 | mutex_lock(&mdsc->mutex); | 2503 | mutex_lock(&mdsc->mutex); |
2505 | __wake_requests(mdsc, &session->s_waiting); | 2504 | __wake_requests(mdsc, &session->s_waiting); |
2505 | if (wake == 2) | ||
2506 | kick_requests(mdsc, mds); | ||
2506 | mutex_unlock(&mdsc->mutex); | 2507 | mutex_unlock(&mdsc->mutex); |
2507 | } | 2508 | } |
2508 | return; | 2509 | return; |