Revert "filesetup: honor the offset option"
authorShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Wed, 25 Sep 2019 08:29:23 +0000 (17:29 +0900)
committerJens Axboe <axboe@kernel.dk>
Wed, 25 Sep 2019 09:12:04 +0000 (03:12 -0600)
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>
filesetup.c

index b8d1d83899cea588a40af2ed126adf150f26fa4c..7904d187e5f3ce394ce8707fee05d02562a02530 100644 (file)
@@ -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;