diff options
| author | Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com> | 2019-09-25 17:29:23 +0900 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2019-09-25 03:12:04 -0600 |
| commit | 0d0601479d593ca9e96e35f43e2c0586f8358f34 (patch) | |
| tree | 764187b7573b060d36c48bda70593d2090115e29 | |
| parent | 3982ec03ab0c125d01b62876f5139b2c07082c7a (diff) | |
| download | fio-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.c | 2 |
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; |