Give more RAM per ld process.

Give more RAM per ld process.

It has been observed that occasionally official buildbots
would crash with OOM, when building Chrome with CFI
(and thus with LTO). This CL reduces the number of
concurrent link processes to avoid this problem.

BUG=464797
Committed: https://crrev.com/2d29f7386050b401838cfbb722a0f53fd1230fce
Cr-Commit-Position: refs/heads/master@{#364173}

[krasin, 2015-12-09 04:31:32]

krasin@google.com changed reviewers:
+ thakis@chromium.org

[Nico, 2015-12-09 15:53:36]

This only controls how many ld processes ninja will allow to run in parallel at
most. Why does this help with OOM for a single ld process? Due to linux's OOM
killer?

[imkrasin, 2015-12-09 16:53:10]

On 2015/12/09 15:53:36, Nico wrote:
> This only controls how many ld processes ninja will allow to run in parallel
at
> most. Why does this help with OOM for a single ld process? Due to linux's OOM
> killer?

I am sorry, if the CL description is not clear. I am trying to solve the
parallel case. The CL reserves more RAM for each of the ld processes running in
parallel.

[krasin, 2015-12-09 21:24:55]

Nico,

please, take another look. I have decided to be even more conservative and make
a 20 GB reserve for spikes.

This will gives us 1 link process for the machines with less than 60 GB RAM, 2
link processes for 60 / 64 GB RAM, and 5 link processes for 128 GB. May be we'll
relax it a bit later.

[Nico, 2015-12-09 21:28:31]

lgtm

[krasin, 2015-12-09 21:40:53]

The CQ bit was checked by krasin@google.com

[commit-bot: I haz the power, 2015-12-09 21:46:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1509733004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1509733004/20001

[commit-bot: I haz the power, 2015-12-09 22:15:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-12-09 22:16:51]

Description was changed from

==========
Give more RAM per ld process.

It has been observed that occasionally official buildbots
would crash with OOM, when building Chrome with CFI
(and thus with LTO). This CL reduces the number of
concurrent link processes to avoid this problem.

BUG=464797
==========

to

==========
Give more RAM per ld process.

It has been observed that occasionally official buildbots
would crash with OOM, when building Chrome with CFI
(and thus with LTO). This CL reduces the number of
concurrent link processes to avoid this problem.

BUG=464797

Committed: https://crrev.com/2d29f7386050b401838cfbb722a0f53fd1230fce
Cr-Commit-Position: refs/heads/master@{#364173}
==========

[commit-bot: I haz the power, 2015-12-09 22:16:52]

Patchset 2 (id:??) landed as
https://crrev.com/2d29f7386050b401838cfbb722a0f53fd1230fce
Cr-Commit-Position: refs/heads/master@{#364173}