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

Issue 1557033002: Fixes for OSThread creation shutdown race. (Closed)

Created:
4 years, 11 months ago by zra
Modified:
4 years, 11 months ago
Reviewers:
siva
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fixes for OSThread creation shutdown race. Also, backs out bad change to the thread pool. The real fix is to move the disable of OSThread creation until after the thread pool shuts down. R=asiva@google.com Committed: https://github.com/dart-lang/sdk/commit/18929970a03d86cc6b97c3524809dc3cb095f1c8

Patch Set 1 #

Total comments: 2

Patch Set 2 : Address comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+25 lines, -23 lines) Patch
M runtime/vm/dart.cc View 1 1 chunk +7 lines, -4 lines 0 comments Download
M runtime/vm/os_thread.h View 1 chunk +6 lines, -1 line 0 comments Download
M runtime/vm/os_thread.cc View 1 3 chunks +10 lines, -10 lines 0 comments Download
M runtime/vm/thread_pool.cc View 1 chunk +2 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
zra
4 years, 11 months ago (2016-01-04 18:37:51 UTC) #2
siva
lgtm https://codereview.chromium.org/1557033002/diff/1/runtime/vm/os_thread.cc File runtime/vm/os_thread.cc (right): https://codereview.chromium.org/1557033002/diff/1/runtime/vm/os_thread.cc#newcode131 runtime/vm/os_thread.cc:131: } Since everything inside the 'if statement' is ...
4 years, 11 months ago (2016-01-04 23:07:55 UTC) #3
zra
https://codereview.chromium.org/1557033002/diff/1/runtime/vm/os_thread.cc File runtime/vm/os_thread.cc (right): https://codereview.chromium.org/1557033002/diff/1/runtime/vm/os_thread.cc#newcode131 runtime/vm/os_thread.cc:131: } On 2016/01/04 23:07:55, siva wrote: > Since everything ...
4 years, 11 months ago (2016-01-04 23:21:53 UTC) #4
zra
4 years, 11 months ago (2016-01-04 23:22:06 UTC) #6
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
18929970a03d86cc6b97c3524809dc3cb095f1c8 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698