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

Issue 100193002: Fix GN deadlock on error handling. (Closed)

Created:
7 years ago by brettw
Modified:
7 years ago
Reviewers:
scottmg
CC:
chromium-reviews
Visibility:
Public.

Description

Fix GN deadlock on error handling. This would deadlock in some error cases when the failing thread tried to acquire the lock to report an error, but the main thread was in the process of shutting down (due to a previous error) and blocking on the worker threads finishing while holding thc lock. This moves the worker pool shutdown to outside the lock, and adds some extra shutdown assertions. It also fixes a syntax error in the base Linux build file. BUG= R=scottmg@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238207

Patch Set 1 #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -5 lines) Patch
M tools/gn/scheduler.cc View 1 2 chunks +10 lines, -4 lines 0 comments Download
M tools/gn/secondary/base/BUILD.gn View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
brettw
7 years ago (2013-12-02 21:19:32 UTC) #1
scottmg
lgtm
7 years ago (2013-12-02 21:30:14 UTC) #2
brettw
7 years ago (2013-12-02 22:20:09 UTC) #3
Message was sent while issue was closed.
Committed patchset #2 manually as r238207 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698