From: Dave <dglen@micron.com>
Date: Tue, 18 Aug 2015 16:39:11 +0000 (-0600)
Subject: Correct handleing of rate_iops_min and ratemin
X-Git-Tag: fio-2.2.10~18^2
X-Git-Url: https://git.kernel.dk/?p=fio.git;a=commitdiff_plain;h=20e37e4a63ea7edb528c8a443b738a0a67c2406e

Correct handleing of rate_iops_min and ratemin

change to allow ratemin to be set without rate disabled ratemin checking in __check_min_rate.

Separate from the change to ratemin parsing, rate_iops_min check did not result in "return 1;" so jobs would not terminate on a failure to maintain rate_iops_min.
---

diff --git a/backend.c b/backend.c
index d5b260a8..094cf710 100644
--- a/backend.c
+++ b/backend.c
@@ -179,7 +179,7 @@ static int __check_min_rate(struct thread_data *td, struct timeval *now,
 		if (spent < td->o.ratecycle)
 			return 0;
 
-		if (td->o.rate[ddir]) {
+		if (td->o.rate[ddir] || td->o.ratemin[ddir]) {
 			/*
 			 * check bandwidth specified rate
 			 */
@@ -220,6 +220,7 @@ static int __check_min_rate(struct thread_data *td, struct timeval *now,
 					log_err("%s: min iops rate %u not met,"
 						" got %lu\n", td->o.name,
 							rate_iops_min, rate);
+					return 1;
 				}
 			}
 		}