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

jewel: rbd: helgrind: TestLibRBD.TestIOPP potential deadlock closing an image with read-ahead enabled #11463

Merged
1 commit merged into from Oct 20, 2016

Conversation

ghost
Copy link

@ghost ghost commented Oct 13, 2016

Fixes: http://tracker.ceph.com/issues/17198
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit c971d58)
@ghost ghost self-assigned this Oct 13, 2016
@ghost ghost added this to the jewel milestone Oct 13, 2016
@ghost ghost added bug-fix rbd labels Oct 13, 2016
ghost pushed a commit that referenced this pull request Oct 17, 2016
…tial deadlock closing an image with read-ahead enabled

Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Oct 18, 2016
…tial deadlock closing an image with read-ahead enabled

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

ghost commented Oct 20, 2016

@dillaman does this backport look good to merge ? It passed the rbd suite http://tracker.ceph.com/issues/17487#note-17 (except for two dead nbd jobs)

@ghost ghost assigned dillaman Oct 20, 2016
@dillaman
Copy link

👍

@ghost ghost merged commit 51d5ed0 into ceph:jewel Oct 20, 2016
@theanalyst theanalyst changed the title jewel: helgrind: TestLibRBD.TestIOPP potential deadlock closing an image with read-ahead enabled "jewel: rbd: helgrind: TestLibRBD.TestIOPP potential deadlock closing an image with read-ahead enabled" Nov 17, 2016
@theanalyst theanalyst changed the title "jewel: rbd: helgrind: TestLibRBD.TestIOPP potential deadlock closing an image with read-ahead enabled" jewel: rbd: helgrind: TestLibRBD.TestIOPP potential deadlock closing an image with read-ahead enabled Nov 17, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant