Fix another stupid bug introduced by 09a1be3

Author: sfan5

src/bench-targets.c

@@ -57,5 +57,9 @@ int main(int argc, char *argv[])
 	printf("time %.2fs\n", secs);
 	printf("speed %ldk/s\n", (long)(naddr / secs) / 1000);
 
+	if(target_gen_progress() <= 0.999f) {
+		printf("DID NOT FINISH?!\n");
+		return 1;
+	}
 	return 0;
 }

src/target-gen.c

@@ -401,8 +401,9 @@ static void fill_cache(void)
 		iter_min = (targets_last_i + 1) % targets_i;
 	do {
 		const unsigned int iter_max = unfinished_max;
-		// update unfinished_max along the way, for the next iteration
-		unfinished_max = 0;
+		// update unfinished_max along the way, for the next iteration.
+		// take the targets we haven't looked at into account when iterating partially.
+		unfinished_max = iter_min;
 		assert(iter_min < iter_max);
 		for(unsigned int i = iter_min; i < iter_max; i++) {
 			if(targets[i].done)
@@ -423,6 +424,12 @@ static void fill_cache(void)
 		}
 		iter_min = 0;
 	} while(unfinished_max > 0);
+
+	// if we get here, we must be done
+	for(unsigned int i = 0; i < targets_i; i++) {
+		assert(targets[i].done);
+		assert(targets[i].delayed_start == 0);
+	}
 }
 
 static inline uint64_t read_be64(const void *ptr)

util/ci-test4.sh

@@ -112,6 +112,22 @@ done
 
 ##
 
+# differing sizes and bigger than the internal buffer
+printf '%s\n' >in.txt \
+	f00::x f01::xx f02::xxx f03::xxxx
+
+for opt in 1 0; do
+
+	notee=1 try --randomize-hosts $opt --print-hosts @in.txt
+	check_out "^f00::2"
+	check_out "^f01::24"
+	check_out "^f02::246"
+	check_out "^f03::2468"
+
+done
+
+##
+
 try --icmp 3ffe::/48
 check_out "tremendous amount of time"