[PATCH 1/1] test/py: use actual core count for parallel builds

When building U-Boot we should not blindly use make -j8 but consider the actual core count given by os.cpu_count().
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de --- test/py/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/py/conftest.py b/test/py/conftest.py index e3392ff6bc..30920474b3 100644 --- a/test/py/conftest.py +++ b/test/py/conftest.py @@ -156,7 +156,7 @@ def pytest_configure(config): o_opt = '' cmds = ( ['make', o_opt, '-s', board_type + '_defconfig'], - ['make', o_opt, '-s', '-j8'], + ['make', o_opt, '-s', '-j{}'.format(os.cpu_count())], ) name = 'make'
-- 2.26.2

On Sat, 30 May 2020 at 16:44, Heinrich Schuchardt xypron.glpk@gmx.de wrote:
When building U-Boot we should not blindly use make -j8 but consider the actual core count given by os.cpu_count().
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de
test/py/conftest.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
I did send a patch to make this use buildman, but people were not keen on the extra dependency.
diff --git a/test/py/conftest.py b/test/py/conftest.py index e3392ff6bc..30920474b3 100644 --- a/test/py/conftest.py +++ b/test/py/conftest.py @@ -156,7 +156,7 @@ def pytest_configure(config): o_opt = '' cmds = ( ['make', o_opt, '-s', board_type + '_defconfig'],
['make', o_opt, '-s', '-j8'],
['make', o_opt, '-s', '-j{}'.format(os.cpu_count())], ) name = 'make'
-- 2.26.2

On 5/30/20 4:44 PM, Heinrich Schuchardt wrote:
When building U-Boot we should not blindly use make -j8 but consider the actual core count given by os.cpu_count().
Acked-by: Stephen Warren swarren@nvidia.com

On Sun, May 31, 2020 at 12:44:24AM +0200, Heinrich Schuchardt wrote:
When building U-Boot we should not blindly use make -j8 but consider the actual core count given by os.cpu_count().
Signed-off-by: Heinrich Schuchardt xypron.glpk@gmx.de Reviewed-by: Simon Glass sjg@chromium.org Acked-by: Stephen Warren swarren@nvidia.com
Applied to u-boot/master, thanks!
participants (4)
-
Heinrich Schuchardt
-
Simon Glass
-
Stephen Warren
-
Tom Rini