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

bluestore/NVMEDevice: fix the bug in write function #17086

Merged
merged 1 commit into from Aug 21, 2017

Conversation

Projects
None yet
3 participants
@optimistyzy
Copy link
Contributor

commented Aug 18, 2017

NVMEDevice: fix the write bug
This patch can solve the write issue when we use spdk nvme driver to handle I/Os from db bdev in the future.

Signed-off-by: Ziye Yang optimistyzy@gmail.com
Signed-off-by: Pan Liu wanjun.lp@alibaba-inc.com

NVMEDevice: fix the write bug
This patch can solve the write issue when we use spdk nvme driver
to handle I/Os from db bdev in the future.

Signed-off-by: Ziye Yang <optimistyzy@gmail.com>
Signed-off-by: Pan Liu <wanjun.lp@alibaba-inc.com>
@optimistyzy

This comment has been minimized.

Copy link
Contributor Author

commented Aug 18, 2017

@liupan1111

This comment has been minimized.

Copy link
Contributor

commented Aug 18, 2017

I tried locally and it works well.

@liupan1111 liupan1111 changed the title NVMEDevice: fix the write bug bluestore/NVMEDevice: fix the bug in write function Aug 18, 2017

@liupan1111

This comment has been minimized.

Copy link
Contributor

commented Aug 19, 2017

Jenkins, retest this please

1 similar comment
@liupan1111

This comment has been minimized.

Copy link
Contributor

commented Aug 20, 2017

Jenkins, retest this please

@liupan1111 liupan1111 requested a review from yuyuyu101 Aug 20, 2017

@yuyuyu101 yuyuyu101 merged commit e7ec34b into ceph:master Aug 21, 2017

5 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.