Adar Dembo has submitted this change and it was merged.
Change subject: KUDU-2052: use XFS_IOC_UNRESVSP64 ioctl to punch holes on xfs
......................................................................
KUDU-2052: use XFS_IOC_UNRESVSP64 ioctl to punch holes on xfs
There's not much to say beyond what's already in the bug report.
To test, I created a 128 GB loopback-mounted xfs filesystem and ran the new
benchmark on it. I included the results below. I also ran the entire Kudu
test suite on that filesystem.
$ TEST_TMPDIR=/data/8/adar/xfs/ bin/env-test \
--gtest_filter=*Benchmark* \
--never_fsync=false \
--unlock_unsafe_flags
Time spent writing 1073741824 bytes to file: real 0.538s user 0.006s
sys 0.533s
Time spent syncing file: real 8.142s user 0.002s sys 0.193s
Time spent punching first hole of size 10485760: real 0.131s user 0.000s
sys 0.130s
Time spent syncing file: real 0.000s user 0.000s sys 0.000s
Time spent repunching 1000 holes of size 10485760: real 0.007s user
0.004s sys 0.003s
$ TEST_TMPDIR=/data/8/adar/xfs/ bin/env-test \
--gtest_filter=*Benchmark* \
--never_fsync=false \
--unlock_unsafe_flags \
--unlock_experimental_flags \
--env_use_ioctl_hole_punch_on_xfs=false
Time spent writing 1073741824 bytes to file: real 0.555s user 0.007s
sys 0.550s
Time spent syncing file: real 8.241s user 0.000s sys 0.219s
Time spent punching first hole of size 10485760: real 0.175s user 0.000s
sys 0.148s
Time spent syncing file: real 0.000s user 0.000s sys 0.000s
Time spent repunching 1000 holes of size 10485760: real 27.719s user
0.026s sys 0.029s
Change-Id: Ia3f5478729594f2a7e4e441905995b6f5d207d51
Reviewed-on: http://gerrit.cloudera.org:8080/7269
Reviewed-by: Todd Lipcon <[email protected]>
Tested-by: Kudu Jenkins
---
M src/kudu/util/env-test.cc
M src/kudu/util/env_posix.cc
2 files changed, 126 insertions(+), 17 deletions(-)
Approvals:
Todd Lipcon: Looks good to me, approved
Kudu Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/7269
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia3f5478729594f2a7e4e441905995b6f5d207d51
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
