From: Jens Axboe Date: Wed, 7 Jun 2017 14:48:05 +0000 (-0600) Subject: fileset: fix double addition of file offset X-Git-Tag: fio-2.21~21 X-Git-Url: https://git.kernel.dk/?p=fio.git;a=commitdiff_plain;h=f0d1393544bf5e2bfdb26d09f5153533f63d7375 fileset: fix double addition of file offset The offset-as-percentage patch inadvertently screwed up the regular offset path, fix that up. Fixes: 89978a6b26f8 ("allow a percent value for the offset parameter") Fixes: https://github.com/axboe/fio/issues/382 Signed-off-by: Jens Axboe --- diff --git a/filesetup.c b/filesetup.c index e548d21d..683a5c55 100644 --- a/filesetup.c +++ b/filesetup.c @@ -840,7 +840,6 @@ uint64_t get_start_offset(struct thread_data *td, struct fio_file *f) return f->real_file_size; if (o->start_offset_percent > 0) { - /* if blockalign is provided, find the min across read, write, and trim */ if (fio_option_is_set(o, ba)) { align_bs = (unsigned long long) min(o->ba[DDIR_READ], o->ba[DDIR_WRITE]); @@ -858,8 +857,8 @@ uint64_t get_start_offset(struct thread_data *td, struct fio_file *f) offset = (offset / align_bs + (offset % align_bs != 0)) * align_bs; } else { /* start_offset_percent not set */ - offset = o->start_offset + o->start_offset + - td->subjob_number * o->offset_increment; + offset = o->start_offset + + td->subjob_number * o->offset_increment; } return offset; }