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

mds/MDLog.cc Fix perf counter type for jlat #13449

Merged
merged 1 commit into from Feb 17, 2017

Conversation

Projects
None yet
2 participants
@xiaoxichen
Copy link
Contributor

xiaoxichen commented Feb 16, 2017

jlat (journal latency) should be PERFCOUNTER_TIME | PERFCOUNTER_LONGRUNAVG
instead of PERFCOUNTER_U64

Current code with PERFCOUNTER_U64 make PerfCounters::tinc return
directly thus jlat will always be zero.

Signed-off-by: Xiaoxi Chen xiaoxchen@ebay.com

mds/MDLog.cc Fix perf counter type for jlat
jlat (journal latency) should be PERFCOUNTER_TIME | PERFCOUNTER_LONGRUNAVG
instead of PERFCOUNTER_U64

Current code with PERFCOUNTER_U64 make PerfCounters::tinc return
directly thus jlat will always be zero.

Signed-off-by: Xiaoxi Chen <xiaoxchen@ebay.com>

@xiaoxichen xiaoxichen requested review from jcsp and ukernel Feb 16, 2017

@xiaoxichen

This comment has been minimized.

Copy link
Contributor Author

xiaoxichen commented Feb 16, 2017

@jcsp wonder if any chance can squeeze into next jewel release in the corner(10.2.6)

@jcsp

jcsp approved these changes Feb 16, 2017

@xiaoxichen xiaoxichen merged commit 82a0260 into ceph:master Feb 17, 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
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.