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: accurate the latency perf counter of queue latency #17435

Merged
merged 4 commits into from Sep 4, 2017
Merged

bluestore/NVMEDevice: accurate the latency perf counter of queue latency #17435

merged 4 commits into from Sep 4, 2017

Conversation

liupan1111
Copy link
Contributor

@liupan1111 liupan1111 commented Sep 4, 2017

accurate the latency perf counter of queue latency:
In old code, start is the beginning of _aio_thread. Indeed, it should be the start time of each Task.

Also contains several other small changes.

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

Pan Liu added 4 commits September 4, 2017 10:49
nvme-device-manager, to avoid name duplication with main thread
"ceph-osd"

Signed-off-by: Pan Liu <wanjun.lp@alibaba-inc.com>
Signed-off-by: Ziye Yang <optimistyzy@gmail.com>
because it will also statistic the latency of sync write

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

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

@optimistyzy @yuyuyu101 please help take a look. Thanks.

@liupan1111 liupan1111 changed the title accurate the latency perf counter of queue latency bluestore/NVMEDevice: accurate the latency perf counter of queue latency Sep 4, 2017
@yuyuyu101 yuyuyu101 merged commit 0956100 into ceph:master Sep 4, 2017
@liupan1111
Copy link
Contributor Author

@yuyuyu101 thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants