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: librbd: journal IO error results in failed assertion in AioCompletion #9611

Merged
merged 9 commits into from Jun 10, 2016

Conversation

dillaman
Copy link

@dillaman dillaman commented Jun 9, 2016

Jason Dillaman and others added 9 commits June 9, 2016 14:08
The journal cannot directly fail the AioCompletion because it
might have one-or-more outstanding requests associated with
it.

Fixes: http://tracker.ceph.com/issues/15791
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 79c9347)
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 82587a3)
This change in behavior was introduced in fde9f78.

Signed-off-by: Hector Martin <marcan@marcan.st>
(cherry picked from commit 77f7c1c)
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 133b1ff)
Fixes: http://tracker.ceph.com/issues/15034
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit eb8dda5)
Fixes: http://tracker.ceph.com/issues/16077
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit a7f5bc1)
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 1032f19)
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit d3d4e06)
Ensure all IO has been properly flushed and committed to the
journal before starting an op that could affect the IO path.

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
(cherry picked from commit 6361296)
@dillaman dillaman added this to the jewel milestone Jun 9, 2016
@dillaman dillaman merged commit 49ed4ff into ceph:jewel Jun 10, 2016
@dillaman dillaman deleted the wip-16210 branch June 10, 2016 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants