io_u: wrap to beginning when end-of-file is reached for time_based
authorJens Axboe <axboe@kernel.dk>
Wed, 1 Nov 2017 17:02:30 +0000 (11:02 -0600)
committerJens Axboe <axboe@kernel.dk>
Wed, 1 Nov 2017 17:02:30 +0000 (11:02 -0600)
The logic around using io_size isn't correct.

Signed-off-by: Jens Axboe <axboe@kernel.dk>
io_u.c

diff --git a/io_u.c b/io_u.c
index 4246edff0b2ffe19242dace14d4271ba851c7856..5e4209d3533252a965b7be6016d33300e3106db7 100644 (file)
--- a/io_u.c
+++ b/io_u.c
@@ -361,16 +361,13 @@ static int get_next_seq_offset(struct thread_data *td, struct fio_file *f,
 
        assert(ddir_rw(ddir));
 
+       /*
+        * If we reach the end for a time based run, reset us back to 0
+        * and invalidate the cache, if we need to.
+        */
        if (f->last_pos[ddir] >= f->io_size + get_start_offset(td, f) &&
            o->time_based) {
-               struct thread_options *o = &td->o;
-               uint64_t io_size = f->io_size + (f->io_size % o->min_bs[ddir]);
-
-               if (io_size > f->last_pos[ddir])
-                       f->last_pos[ddir] = 0;
-               else
-                       f->last_pos[ddir] = f->last_pos[ddir] - io_size;
-
+               f->last_pos[ddir] = 0;
                loop_cache_invalidate(td, f);
        }