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

hammer: osd: race condition detected during send_failures #6755

Merged
merged 1 commit into from Jan 14, 2016

Conversation

smithfarm
Copy link
Contributor

It is unsafe to check failure_queue.empty() without the lock.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit b3ca828)
@smithfarm smithfarm self-assigned this Dec 1, 2015
@smithfarm smithfarm added this to the hammer milestone Dec 1, 2015
ghost pushed a commit that referenced this pull request Jan 11, 2016
… send_failures

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jan 11, 2016
… send_failures

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jan 11, 2016
… send_failures

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost
Copy link

ghost commented Jan 14, 2016

@liewegas does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/13356#note-34 for details ) and the upgrade/hammer suite ( see http://tracker.ceph.com/issues/13356#note-37 for details ).

@ghost ghost assigned liewegas and unassigned smithfarm Jan 14, 2016
liewegas added a commit that referenced this pull request Jan 14, 2016
hammer: OSD: race condition detected during send_failures

Reviewed-by: Sage Weil <sage@redhat.com>
@liewegas liewegas merged commit d064f9f into ceph:hammer Jan 14, 2016
@smithfarm smithfarm deleted the wip-13870-hammer branch January 14, 2016 14:27
@ghost ghost changed the title hammer: OSD: race condition detected during send_failures hammer: osd: race condition detected during send_failures Feb 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants