Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2)

Issue 9402039: Prevent a possible deadlock in the linker script when using goma. (Closed)

Created:
8 years, 10 months ago by Nico
Modified:
8 years, 10 months ago
Reviewers:
Lei Zhang, Evan Martin
CC:
chromium-reviews
Visibility:
Public.

Description

Prevent a possible deadlock in the linker script when using goma. Make (and now ninja) run ld under flock. Goma allows only a limited number N of local gcc processes, which it does by running a custom gcc. If N gcc processes are trying to run a link, N-1 will be waiting on flock. The Nth process then tries to call an N+1th gcc process to determine the bitness of the system, which blocks. As a workaround, use getconf to determine bitness instead. Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=122292

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M ld View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Nico
8 years, 10 months ago (2012-02-16 02:53:10 UTC) #1
Lei Zhang
I don't have a 64-bit kernel / 32-bit userland configuration, so I'm not sure if ...
8 years, 10 months ago (2012-02-16 03:23:45 UTC) #2
Nico
On 2012/02/16 03:23:45, Lei Zhang wrote: > I don't have a 64-bit kernel / 32-bit ...
8 years, 10 months ago (2012-02-16 03:26:21 UTC) #3
Lei Zhang
On 2012/02/16 03:26:21, Nico wrote: > On 2012/02/16 03:23:45, Lei Zhang wrote: > > I ...
8 years, 10 months ago (2012-02-16 03:40:00 UTC) #4
Evan Martin
8 years, 10 months ago (2012-02-16 16:12:19 UTC) #5
LGTM if you're willing to deal with bot fallout ;)

Powered by Google App Engine
This is Rietveld 408576698