From: Heiko Carstens Date: Wed, 10 Aug 2011 08:34:29 +0000 (+0200) Subject: lscpu: fix fallback nthreads calculation X-Git-Tag: v2.20.1~52 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d94f62fe46feef1230b288bd51f1e9dbe1782cb6;p=thirdparty%2Futil-linux.git lscpu: fix fallback nthreads calculation The fallback calculation of nthreads did not consider books. In order to avoid division/multiply by/with zero make sure each number used is at least "1". Signed-off-by: Heiko Carstens --- diff --git a/sys-utils/lscpu.c b/sys-utils/lscpu.c index 210a4c2431..9755be50ae 100644 --- a/sys-utils/lscpu.c +++ b/sys-utils/lscpu.c @@ -649,17 +649,21 @@ read_topology(struct lscpu_desc *desc, int num) nthreads = CPU_COUNT_S(setsize, thread_siblings); /* cores within one socket */ ncores = CPU_COUNT_S(setsize, core_siblings) / nthreads; - /* number of sockets within one book */ - nsockets = desc->ncpus / nthreads / ncores; + /* number of sockets within one book. + * Because of odd / non-present cpu maps and to keep + * calculation easy we make sure that nsockets and + * nbooks is at least 1. + */ + nsockets = desc->ncpus / nthreads / ncores ?: 1; /* number of books */ - nbooks = desc->ncpus / nthreads / ncores / nsockets; + nbooks = desc->ncpus / nthreads / ncores / nsockets ?: 1; /* all threads, see also read_basicinfo() - * -- this is fallback for kernels where is not + * -- fallback for kernels without * /sys/devices/system/cpu/online. */ if (!desc->nthreads) - desc->nthreads = nsockets * ncores * nthreads; + desc->nthreads = nbooks * nsockets * ncores * nthreads; /* For each map we make sure that it can have up to ncpus * entries. This is because we cannot reliably calculate the * number of cores, sockets and books on all architectures.