Commit ae7b5890 authored by Darek Stojaczyk's avatar Darek Stojaczyk Committed by Ben Walker
Browse files

githooks: limit the number of threads for pre-push hook



The script probably meant to execute make with $(nproc),
but executed it with ${nproc} instead. "nproc" was not
defined, evaluated to nothing, and -j without integer
spawned unlimited number of processes, which was slow.

While here, also use sysctl -a | grep -E -i 'hw.ncpu'
to get the number of cores on BSD. That's what we already
do in autobuild.sh.

Fixes #881

Change-Id: I8b07a2c28c4834b5dfb1c1bfa66d2b696d85720f
Reported-by: default avatarJan Kryl <jan.kryl@mayadata.io>
Signed-off-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/463179


Reviewed-by: default avatarJan Kryl <jan.kryl@mayadata.io>
Reviewed-by: default avatarPaul Luse <paul.e.luse@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 4640f324
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -20,10 +20,10 @@ SYSTEM=`uname -s`
exec 1>&2

if [ "$SYSTEM" = "FreeBSD" ]; then
	MAKE="gmake MAKE=gmake -j ${nproc}"
	MAKE="gmake MAKE=gmake -j $(sysctl -a | grep -E -i 'hw.ncpu' | awk '{print $2}')"
	COMP="clang"
else
	MAKE="make -j ${nproc}"
	MAKE="make -j $(nproc)"
	COMP="gcc"
fi