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

os/bluestore: fix false assert in IOContext::aio_wake #15268

Merged
merged 2 commits into from May 26, 2017

Conversation

Projects
None yet
2 participants
@ifed01
Contributor

ifed01 commented May 24, 2017

IOContext::aio_wake might assert improperly on (num_running == 0) due to the race between aio_wake and aio_submit. In fact that's a false assertion hence making it less strict. Some cleanup around aio_wake as well.

Igor Fedotov
os/bluestore: wrap IOContext::aio_wake with additional stuff to remov…
…e copy-paste for its use

Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
Igor Fedotov
os/bluestore: fix false assertion
Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>

@ifed01 ifed01 changed the title from os/bluestore: Fix false assert to os/bluestore: fix false assert May 24, 2017

@ifed01 ifed01 changed the title from os/bluestore: fix false assert to os/bluestore: fix false assert in IOContext::aio_wake May 25, 2017

@liewegas liewegas merged commit dcebdf4 into ceph:master May 26, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@ifed01 ifed01 deleted the ifed01:wip-bluestore-false-assert branch May 26, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment