Fix off-by-one in cpus_allowed
authorJens Axboe <axboe@fb.com>
Tue, 21 Apr 2015 15:39:32 +0000 (09:39 -0600)
committerJens Axboe <axboe@fb.com>
Tue, 21 Apr 2015 15:39:32 +0000 (09:39 -0600)
Similar to commit b84113993b3a, fix it for cpus_allowed as well.

Signed-off-by: Jens Axboe <axboe@fb.com>
options.c

index b5195c35c7e4bae7991d66b8c83b89c1f644bf58..5fbad317e47a1e9809006e226cf4e29f15c67630 100644 (file)
--- a/options.c
+++ b/options.c
@@ -503,9 +503,9 @@ static int set_cpus_allowed(struct thread_data *td, os_cpu_mask_t *mask,
                                ret = 1;
                                break;
                        }
-                       if (icpu > max_cpu) {
+                       if (icpu >= max_cpu) {
                                log_err("fio: CPU %d too large (max=%ld)\n",
-                                                       icpu, max_cpu);
+                                                       icpu, max_cpu - 1);
                                ret = 1;
                                break;
                        }