From 30cb4c650a0a4c7ca79d4336ddb1fe3f988a4b2a Mon Sep 17 00:00:00 2001 From: Jens Axboe Date: Fri, 28 Feb 2014 08:35:40 -0800 Subject: [PATCH] Fix off-by-one bug in CPU split mask handling The thread number is not zero based, it starts from 1. Signed-off-by: Jens Axboe --- backend.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backend.c b/backend.c index 12c76d85..72d9d6d2 100644 --- a/backend.c +++ b/backend.c @@ -1279,7 +1279,7 @@ static void *thread_main(void *data) */ if (o->cpumask_set) { if (o->cpus_allowed_policy == FIO_CPUS_SPLIT) { - ret = fio_cpus_split(&o->cpumask, td->thread_number); + ret = fio_cpus_split(&o->cpumask, td->thread_number - 1); if (!ret) { log_err("fio: no CPUs set\n"); log_err("fio: Try increasing number of available CPUs\n"); -- 2.25.1