summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJosef Cejka <jcejka@suse.com>2015-08-20 11:52:51 -0400
committerJens Axboe <axboe@fb.com>2015-08-20 08:58:08 -0700
commit2564a6029e55d6a0507ebb28a5a01f5dffb0bd56 (patch)
tree6795fe80baef6274e55616d02215fec79a928a77
parent88d38b4d0a5b33332fa80d7028a87c0717bb5d75 (diff)
downloadblktrace-2564a6029e55d6a0507ebb28a5a01f5dffb0bd56.tar.gz
blktrace-2564a6029e55d6a0507ebb28a5a01f5dffb0bd56.tar.bz2
btreplay: Fix typo in scaling up the dynamic cpu set size.
In get_ncpus, we default to using 4096 CPUs if _SC_NPROCESSORS_CONF isn't enabled. If that is insufficient, sched_getaffinity will fail and we retry after doubling the size of the cpu_set_t allocation. There's a typo in there that means we don't actually double the size and will loop forever allocating the same sized cpu_set_t instead. Signed-off-by: Josef Cejka <jcejka@suse.com> Signed-off-by: Jeff Mahoney <jeffm@suse.com> Signed-off-by: Jens Axboe <axboe@fb.com>
-rw-r--r--btreplay/btreplay.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/btreplay/btreplay.c b/btreplay/btreplay.c
index 5444010..f762279 100644
--- a/btreplay/btreplay.c
+++ b/btreplay/btreplay.c
@@ -516,7 +516,7 @@ realloc:
if (sched_getaffinity(getpid(), size, cpus)) {
if( errno == EINVAL && nrcpus < (4096<<4) ) {
CPU_FREE(cpus);
- nrcpus <= 1;
+ nrcpus <<= 1;
goto realloc;
}
fatal("sched_getaffinity", ERR_SYSCALL, "Can't get CPU info\n");