summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>2019-09-25 17:29:23 +0900
committerJens Axboe <axboe@kernel.dk>2019-09-25 03:12:04 -0600
commit0d0601479d593ca9e96e35f43e2c0586f8358f34 (patch)
tree764187b7573b060d36c48bda70593d2090115e29
parent3982ec03ab0c125d01b62876f5139b2c07082c7a (diff)
downloadfio-0d0601479d593ca9e96e35f43e2c0586f8358f34.tar.gz
fio-0d0601479d593ca9e96e35f43e2c0586f8358f34.tar.bz2
Revert "filesetup: honor the offset option"
This reverts commit ad46c5715ffc5d4e70d2e941225b1f504f23e409. The commit tried to fix a failure observed with --offset=90%, --size=1T and --ioengine=null options. However, it affected fio behavior not only for null ioengine but also for other all ioengines. It changed io_size, then I/O read or written bytes are not same as that specified with command line 'size' option. This caused t/zbd/test-zbd-support test script failures for zoned block devices: test cases #17, #28 and #38 failed. To prepare for other fix with limited ioengine impacts, revert the fix which affects all ioengines. Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com> Signed-off-by: Jens Axboe <axboe@kernel.dk>
-rw-r--r--filesetup.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/filesetup.c b/filesetup.c
index b8d1d838..7904d187 100644
--- a/filesetup.c
+++ b/filesetup.c
@@ -1047,7 +1047,7 @@ int setup_files(struct thread_data *td)
* doesn't divide nicely with the min blocksize,
* make the first files bigger.
*/
- f->io_size = fs - f->file_offset;
+ f->io_size = fs;
if (nr_fs_extra) {
nr_fs_extra--;
f->io_size += bs;