All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robert Yang <liezhi.yang@windriver.com>
To: Richard Purdie <richard.purdie@linuxfoundation.org>,
	Alexander Kanavin <alex.kanavin@gmail.com>
Cc: Konrad Weihmann <kweihmann@outlook.com>,
	OE-core <openembedded-core@lists.openembedded.org>
Subject: Re: [OE-core] [PATCH 1/1] bitbake.conf: Set ZSTD_THREADS to half of cpu number
Date: Thu, 11 Nov 2021 12:09:34 +0800	[thread overview]
Message-ID: <505921f1-c7e7-1ace-2b91-a0248f47b31d@windriver.com> (raw)
In-Reply-To: <4cea3606226d54d229d0103281160343b041eb3f.camel@linuxfoundation.org>

Hi RP,

On 11/9/21 6:26 PM, Richard Purdie wrote:
> On Tue, 2021-11-09 at 10:56 +0100, Alexander Kanavin wrote:
>> On Tue, 9 Nov 2021 at 10:51, Robert Yang <liezhi.yang@windriver.com> wrote:
>>>> Maybe, but once we start doing tweaks like this, where do they stop? Hey,
>>> I'd
>>>
>>> I think that it's not only a performance tweak, but this should be
>>> considered
>>> as
>>> a bug since it would be failed to build on powerful machine, which isn't
>>> good
>>> for oe-core's user experience.
>>>
>>
>>
>> I managed to bring down a powerful machine just yesterday by building webkit
>> and llvm together at the same time. They together ate all the RAM, and someone
>> in the office had to go and reset the box.
>> Do I see it as a bug? Absolutely not: it's in fact my fault for not ensuring
>> system resource consumption doesn't overwhelm the machine. This is the same:
>> if you do not have RAM to match the cores on your specific rig, add RAM or
>> limit the threads.
> 
> I can see both sides of this. If we had a pool for all the threads and the
> pieces of the system all used the same pool, it would work well as implemented.
> Sadly the bitbake threads, the make processes and now the compression threads
> are all separate.
> 
> We do see the autobuilder break due to load issues too and on that we have
> scaled back some of the defaults but not ZSTD as yet that I recall.
> 
> As such I think limiting the upper value on number of threads may be a good
> compromise, both for compression and maybe for parallel make too.

What should we do next, please? Limit bb thread + parallel make + zstd thread?
Or just limit zstd's thread since other people may concern about the
performance.

// Robert

> 
> Cheers,
> 
> Richard
> 


  reply	other threads:[~2021-11-11  4:09 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-11-09  8:48 [PATCH 0/1] bitbake.conf: Set ZSTD_THREADS to half of cpu number Robert Yang
2021-11-09  8:48 ` [PATCH 1/1] " Robert Yang
2021-11-09  8:21   ` [OE-core] " Alexander Kanavin
2021-11-09  9:45     ` Robert Yang
2021-11-09  8:41   ` Konrad Weihmann
2021-11-09  9:40     ` Robert Yang
2021-11-09  9:44       ` Alexander Kanavin
2021-11-09  9:51         ` Robert Yang
2021-11-09  9:56           ` Alexander Kanavin
2021-11-09 10:26             ` Richard Purdie
2021-11-11  4:09               ` Robert Yang [this message]
2021-11-09  8:45   ` Jose Quaresma

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=505921f1-c7e7-1ace-2b91-a0248f47b31d@windriver.com \
    --to=liezhi.yang@windriver.com \
    --cc=alex.kanavin@gmail.com \
    --cc=kweihmann@outlook.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=richard.purdie@linuxfoundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.