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

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

[Heinrich Schuchardt]

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

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

[Simon Glass]

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
>

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

[Stephen Warren]

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>

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

[Tom Rini]

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!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200603/eec2c2df/attachment.sig>