{"action":"created","issue":{"url":"https://api.github.com/repos/ceph/ceph/issues/6996","labels_url":"https://api.github.com/repos/ceph/ceph/issues/6996/labels{/name}","comments_url":"https://api.github.com/repos/ceph/ceph/issues/6996/comments","events_url":"https://api.github.com/repos/ceph/ceph/issues/6996/events","html_url":"https://github.com/ceph/ceph/pull/6996","id":123219973,"number":6996,"title":"mds: we should wait messenger when MDSDaemon suicide","user":{"login":"flywayway","id":13265413,"avatar_url":"https://avatars.githubusercontent.com/u/13265413?v=3","gravatar_id":"","url":"https://api.github.com/users/flywayway","html_url":"https://github.com/flywayway","followers_url":"https://api.github.com/users/flywayway/followers","following_url":"https://api.github.com/users/flywayway/following{/other_user}","gists_url":"https://api.github.com/users/flywayway/gists{/gist_id}","starred_url":"https://api.github.com/users/flywayway/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/flywayway/subscriptions","organizations_url":"https://api.github.com/users/flywayway/orgs","repos_url":"https://api.github.com/users/flywayway/repos","events_url":"https://api.github.com/users/flywayway/events{/privacy}","received_events_url":"https://api.github.com/users/flywayway/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ceph/ceph/labels/cephfs","name":"cephfs","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-12-21T06:44:29Z","updated_at":"2015-12-21T13:08:55Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ceph/ceph/pulls/6996","html_url":"https://github.com/ceph/ceph/pull/6996","diff_url":"https://github.com/ceph/ceph/pull/6996.diff","patch_url":"https://github.com/ceph/ceph/pull/6996.patch"},"body":"Signed-off-by: Wei Feng <feng.wei@h3c.com>"},"comment":{"url":"https://api.github.com/repos/ceph/ceph/issues/comments/166301359","html_url":"https://github.com/ceph/ceph/pull/6996#issuecomment-166301359","issue_url":"https://api.github.com/repos/ceph/ceph/issues/6996","id":166301359,"user":{"login":"jcsp","id":944640,"avatar_url":"https://avatars.githubusercontent.com/u/944640?v=3","gravatar_id":"","url":"https://api.github.com/users/jcsp","html_url":"https://github.com/jcsp","followers_url":"https://api.github.com/users/jcsp/followers","following_url":"https://api.github.com/users/jcsp/following{/other_user}","gists_url":"https://api.github.com/users/jcsp/gists{/gist_id}","starred_url":"https://api.github.com/users/jcsp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jcsp/subscriptions","organizations_url":"https://api.github.com/users/jcsp/orgs","repos_url":"https://api.github.com/users/jcsp/repos","events_url":"https://api.github.com/users/jcsp/events{/privacy}","received_events_url":"https://api.github.com/users/jcsp/received_events","type":"User","site_admin":false},"created_at":"2015-12-21T13:08:55Z","updated_at":"2015-12-21T13:08:55Z","body":"msgr->wait() is called from ceph_mds.cc  -- I don't think you can wait() in suicide() because you might call suicide() while handling a message (i.e. you are inside the messenger handling code, so would deadlock).  Unless I'm missing something?  What case is this meant to handle?"}}