os/bluestore/KernelDevice: fix race in aio_thread vs aio_wait #12204

Merged
merged 1 commit into from Nov 29, 2016

Projects

None yet

2 participants

@liewegas
Member

Fixes: http://tracker.ceph.com/issues/17824
Signed-off-by: Sage Weil sage@redhat.com

@liewegas liewegas added this to the kraken milestone Nov 28, 2016
@liewegas liewegas os/bluestore/KernelDevice: fix race in aio_thread vs aio_wait
The caller is free to destroy the aio vector contents
once aio_wait completes.  This is exactly what
BlueFS::_fsync() does.  Delay the num_running dec
(which is what aio_wait waits for) until after we've
examined the aios.

Fixes: http://tracker.ceph.com/issues/17824
Signed-off-by: Sage Weil <sage@redhat.com>
6ffaa54
@markhpc

makes sense, lgtm

@markhpc markhpc merged commit b67df1e into ceph:master Nov 29, 2016

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment