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/NVMEDevice.cc: fix the random read issue. #13055

Merged
merged 1 commit into from Jan 24, 2017

Conversation

Projects
None yet
3 participants
@optimistyzy
Contributor

optimistyzy commented Jan 23, 2017

Random read is failed, with this patch it can
be fixed.

Signed-off-by: optimistyzy optimistyzy@gmail.com

@yuyuyu101

This comment has been minimized.

Show comment
Hide comment
@yuyuyu101

yuyuyu101 Jan 23, 2017

Member

plz update NVMEDeivce::read with aio_wakeup too

Member

yuyuyu101 commented Jan 23, 2017

plz update NVMEDeivce::read with aio_wakeup too

@liewegas liewegas added the bluestore label Jan 23, 2017

@liewegas liewegas changed the title from NVMEDevice.cc: fix the random read issue. to os/bluestore/NVMEDevice.cc: fix the random read issue. Jan 23, 2017

@optimistyzy

This comment has been minimized.

Show comment
Hide comment
@optimistyzy
Contributor

optimistyzy commented Jan 24, 2017

@yuyuyu101 done

os/bluestore/NVMEDevice.cc: use aio_wake in both read and read_random
There are some issues with random read, it seems that
we did not call aio_wake.

Signed-off-by: optimistyzy <optimistyzy@gmail.com>

@yuyuyu101 yuyuyu101 merged commit 8558a0f into ceph:master Jan 24, 2017

2 of 3 checks passed

default Build started sha1 is merged.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment