Refer td->loops instead of td->o.loops to fix loop count issue
authorShin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Fri, 1 Oct 2021 10:32:57 +0000 (19:32 +0900)
committerJens Axboe <axboe@kernel.dk>
Fri, 1 Oct 2021 17:17:13 +0000 (11:17 -0600)
commit336e3bfbf01657e3814a6a15596c15574b4335ee
tree54b40206565acdcffaf0b41325f559887c1173ac
parent2d44f2c6b45fbffb9ad8570c25ad3b8de5c47547
Refer td->loops instead of td->o.loops to fix loop count issue

In the github issues #1093 and #1278, it was reported that the loops
option does not work as expected when do_verify=0 option is specified.
Per analysis by Sowmya Ravi, the cause was as follows:

1) keep_running() decrements td->o.loops at job repetition, then
   td->o.loops has zero value when the last loop is executed.
2) clear_io_state() is called at the beginning of the thread_main loop
   for each repetition for loops option.
3) clear_io_state() calls reset_io_counters() which resets
   td->nr_done_files to zero when td->o.loops is non-zero.
4) For the last loop of loops option, clear_io_state() call does not
   clear td->nr_done_files since td->l.loops is zero. This results in a
   setup error in do_io().

To fix the issue, modify reset_io_counters() to refer td->loops instead
of td->o.loops. td->o.loops is not a good reference since it is updated
in keep_running(). td->loops is not updated during fio run, and safe to
refer.

Signed-off-by: Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>
Link: https://lore.kernel.org/r/20211001103257.4130231-3-shinichiro.kawasaki@wdc.com
Signed-off-by: Jens Axboe <axboe@kernel.dk>
libfio.c