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

Issue 1226403003: Support per-thread zones and stack resources. (Closed)

Created:
5 years, 5 months ago by koda
Modified:
5 years, 5 months ago
Reviewers:
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

Support per-thread zones and stack resources. This is needed to enable parallel marking/sweeping (and in the future, compilation) tasks to have their own zone allocations and to safely call code that relies on various scoped constructs (stack resources). The next step is to migrate the scopes like NoSafepointScope, NoHandleScope, etc. from isolate- to thread-based interfaces. NOTE: This is a copy of issue 1204303003. BUG= Committed: https://github.com/dart-lang/sdk/commit/7f890eeecf672f1e984c3515d63fe343643731d5

Patch Set 1 #

Patch Set 2 : Add thread_registry.h #

Unified diffs Side-by-side diffs Delta from patch set Stats (+489 lines, -137 lines) Patch
M runtime/vm/allocation.h View 2 chunks +40 lines, -19 lines 0 comments Download
M runtime/vm/base_isolate.h View 5 chunks +6 lines, -16 lines 0 comments Download
M runtime/vm/isolate.h View 5 chunks +32 lines, -5 lines 0 comments Download
M runtime/vm/isolate.cc View 5 chunks +7 lines, -5 lines 0 comments Download
M runtime/vm/os_thread.h View 2 chunks +2 lines, -2 lines 0 comments Download
M runtime/vm/stub_code_arm.cc View 9 chunks +13 lines, -13 lines 0 comments Download
M runtime/vm/stub_code_arm64.cc View 9 chunks +13 lines, -13 lines 0 comments Download
M runtime/vm/stub_code_ia32.cc View 9 chunks +13 lines, -13 lines 0 comments Download
M runtime/vm/stub_code_mips.cc View 9 chunks +13 lines, -13 lines 0 comments Download
M runtime/vm/stub_code_x64.cc View 10 chunks +13 lines, -13 lines 0 comments Download
M runtime/vm/thread.h View 6 chunks +43 lines, -6 lines 0 comments Download
M runtime/vm/thread.cc View 7 chunks +25 lines, -1 line 0 comments Download
A runtime/vm/thread_registry.h View 1 1 chunk +105 lines, -0 lines 0 comments Download
M runtime/vm/thread_test.cc View 2 chunks +140 lines, -0 lines 0 comments Download
M runtime/vm/vm_sources.gypi View 1 chunk +1 line, -0 lines 0 comments Download
M runtime/vm/zone.h View 2 chunks +23 lines, -17 lines 0 comments Download
M runtime/vm/zone.cc View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
koda
5 years, 5 months ago (2015-07-09 18:22:33 UTC) #1
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as
7f890eeecf672f1e984c3515d63fe343643731d5 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698