seastar icon indicating copy to clipboard operation
seastar copied to clipboard

Fix instruction count for start_measuring_time

Open travisdowns opened this issue 3 years ago • 2 comments

Currently perf_tests::start_measuring_time (and the corresponding stop call) doesn't correctly measure the instruction count. It always reports 0 instructions if any of these methods is used in a perf test.

To fix this, we capture the linux_perf_event pointer in the time_measurement class.

travisdowns avatar Aug 08 '22 23:08 travisdowns

@bhalevy this may also be in your wheelhouse.

travisdowns avatar Aug 08 '22 23:08 travisdowns

Thanks for the review @bhalevy, much appreciated.

travisdowns avatar Aug 10 '22 04:08 travisdowns