Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

osd: Mutex Assert from PipeConnection::try_get_pipe #5758

Merged
merged 1 commit into from Sep 7, 2015

Conversation

Projects
None yet
3 participants
@ghost
Copy link

ghost commented Sep 2, 2015

osd: Keep a reference count on Connection while calling send_message()
Fixes: #12437

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit a140085)

@ghost ghost self-assigned this Sep 2, 2015

@ghost ghost added this to the hammer milestone Sep 2, 2015

@ghost ghost added bug fix core labels Sep 2, 2015

ghost pushed a commit that referenced this pull request Sep 2, 2015

Merge pull request #5758: Mutex Assert from PipeConnection::try_get_pipe
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost

This comment has been minimized.

Copy link
Author

ghost commented Sep 6, 2015

@liewegas this hammer backport passed a rados suite (see http://tracker.ceph.com/issues/12701#note-62 for the details). Do you think it is ready to be merged ?

Asking you instead of Sam (who merged the corresponding master commit) because he is on vacation.

@ghost ghost assigned liewegas and unassigned ghost Sep 6, 2015

liewegas added a commit that referenced this pull request Sep 7, 2015

Merge pull request #5758 from dachary/wip-12839-hammer
Mutex Assert from PipeConnection::try_get_pipe

@liewegas liewegas merged commit 021abe7 into ceph:hammer Sep 7, 2015

@ghost ghost changed the title Mutex Assert from PipeConnection::try_get_pipe osd: Mutex Assert from PipeConnection::try_get_pipe Sep 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.