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
Fix clat-percentile accuracy in fio #1503
Fix clat-percentile accuracy in fio #1503
Conversation
For the record, the perf presubmit tests passed with the current commit of this PR: log . |
747d22e
to
0466819
Compare
3b65e0d
to
c6c3722
Compare
0466819
to
705a092
Compare
705a092
to
cbac061
Compare
334e5b6
to
00a6876
Compare
3d79660
to
abebb35
Compare
@raj-prince please review. |
This fix is to address internal bug#309563824. As recorded in this bug, fio by default supports clat percentile values to be calculated accurately upto only 2^(FIO_IO_U_PLAT_GROUP_NR + 5) ns = 14 ms (with default value of FIO_IO_U_PLAT_GROUP_NR = 29). This change increases it upto 32, to allow latencies upto 112 ms to be calculated accurately.
abebb35
to
b682575
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Please fix before merging.
- Linux tests are failing.
- We should run the perf-test to make sure, we are not breaking anything.
I have tested the install_fio.sh script locally, and it worked. I'll wait till perf presubmit is run and passing. |
The perf presubmit tests passed: log fetching results.. |
* Fix clat-percentile accuracy in fio This fix is to address internal bug#309563824. As recorded in this bug, fio by default supports clat percentile values to be calculated accurately upto only 2^(FIO_IO_U_PLAT_GROUP_NR + 5) ns = 14 ms (with default value of FIO_IO_U_PLAT_GROUP_NR = 29). This change increases it upto 32, to allow latencies upto 112 ms to be calculated accurately. * address comments
* Fix clat-percentile accuracy in fio This fix is to address internal bug#309563824. As recorded in this bug, fio by default supports clat percentile values to be calculated accurately upto only 2^(FIO_IO_U_PLAT_GROUP_NR + 5) ns = 14 ms (with default value of FIO_IO_U_PLAT_GROUP_NR = 29). This change increases it upto 32, to allow latencies upto 112 ms to be calculated accurately. * address comments
* Fix clat-percentile accuracy in fio This fix is to address internal bug#309563824. As recorded in this bug, fio by default supports clat percentile values to be calculated accurately upto only 2^(FIO_IO_U_PLAT_GROUP_NR + 5) ns = 14 ms (with default value of FIO_IO_U_PLAT_GROUP_NR = 29). This change increases it upto 32, to allow latencies upto 112 ms to be calculated accurately. * address comments
Description
This fix is to address internal bug#309563824.
As recorded in this bug, fio by default supports
clat percentile values to be calculated accurately upto only 2^(FIO_IO_U_PLAT_GROUP_NR + 5) ns = 14 ms
(with default value of FIO_IO_U_PLAT_GROUP_NR = 29). This change increases it upto 32, to allow latencies upto 112 ms to be calculated accurately.
This is dependent on #1506 . This fix is bsased on the suggestion in axboe/fio#1668 (comment) .
Link to the issue in case of a bug fix.
NA
Testing details