Fix bad setting of td->o.ddir_seq_nr in add_job()

This defaults to 1, but add_job() adds 1 again which means
that the random isn't always effective since every alternate
IO gets to bypass that logic.

Signed-off-by: Jens Axboe <jaxboe@fusionio.com>

diff --git a/init.c b/init.c
index cc4037a787e04c48383729c0afe58f8d65c45773..b84d0ad4f358ba6a5004d3443c9c93b06a355268 100644 (file)
--- a/init.c
+++ b/init.c
@@ -561,7 +561,7 @@ static int add_job(struct thread_data *td, const char *jobname, int job_add_num)
        td->ts.slat_stat[0].min_val = td->ts.slat_stat[1].min_val = ULONG_MAX;
        td->ts.lat_stat[0].min_val = td->ts.lat_stat[1].min_val = ULONG_MAX;
        td->ts.bw_stat[0].min_val = td->ts.bw_stat[1].min_val = ULONG_MAX;
        td->ts.slat_stat[0].min_val = td->ts.slat_stat[1].min_val = ULONG_MAX;
        td->ts.lat_stat[0].min_val = td->ts.lat_stat[1].min_val = ULONG_MAX;
        td->ts.bw_stat[0].min_val = td->ts.bw_stat[1].min_val = ULONG_MAX;

-       td->ddir_seq_nr = td->o.ddir_seq_nr + 1;
+       td->ddir_seq_nr = td->o.ddir_seq_nr;

 
        if ((td->o.stonewall || td->o.new_group) && prev_group_jobs) {
                prev_group_jobs = 0;
 
        if ((td->o.stonewall || td->o.new_group) && prev_group_jobs) {
                prev_group_jobs = 0;