tree 1b819e40a12c0bd5f258230936116cac3e5ad766
parent 8cba879f46cb6626f10c1b570f0cf826e7ffd7a6
author Martin Kustermann <kustermann@google.com> 1606932858 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> 1606932858 +0000

[vm/concurrency] Add patchable_call_mutex and take it in miss handlers

We serialize all changes to patchable call state to ensure we always
make progress (e.g. transition from Monomorphic -> Polymorphic).

Only if the miss handler actually needs to patch the patchable call do
we stop mutators - if it only updates ICData we don't have to do that.
=> In the future we might introduce a more clever mechanism to avoid
   stopping mutators.

The CL also moves some locks from Isolate to IsolateGroup to ensure
all mutators use the same lock, since they operate on shared data
structures.

Issue https://github.com/dart-lang/sdk/issues/36097

TEST=Existing isolate tests that --enable-isolate-groups.

Change-Id: I4a11a8b8bb65581c6aba11cdf73acdea0f24a502
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/174469
Reviewed-by: Alexander Aprelev <aam@google.com>
Commit-Queue: Martin Kustermann <kustermann@google.com>