Re: [libvirt-ci PATCH] lcitool: Set make parallelism dynamically

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, May 07, 2020 at 06:53:12PM +0200, Andrea Bolognani wrote:
> We're about to make some changes to lcitool that would make it
> more annoying to get access to the number of install-time vCPUs
> from 'lcitool update', so let's use getconf instead.
> 
> As a side effect, this also copes better with the situation where
> the builder was installed with a certain number of vCPUs but the
> amount has since been modified.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  guests/playbooks/update/templates/bashrc.j2 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/guests/playbooks/update/templates/bashrc.j2 b/guests/playbooks/update/templates/bashrc.j2
> index 8775864..05a86a1 100644
> --- a/guests/playbooks/update/templates/bashrc.j2
> +++ b/guests/playbooks/update/templates/bashrc.j2
> @@ -4,7 +4,7 @@ export MAKE="{{ paths.make }}"
>  export NINJA="{{ paths.ninja }}"
>  export PYTHON="{{ paths.python }}"
>  
> -export MAKEFLAGS="-j{{ install_vcpus|int + 1 }}"
> +export MAKEFLAGS="-j$(getconf _NPROCESSORS_ONLN)"

By the way, do you happen to know where the _NPROCESSORS_ONLN came from? I mean
the only trace of it was the system config constant _SC_NPROCESSORS_ONLN which
the getconf one will be translated to, I was just confused where shorter name
came from, since it's not documented in any of the man pages (or it may be, but
I don't have that specific man page installed).

Erik




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux