Commit 6525dd18 authored by Ben Smith's avatar Ben Smith Committed by Commit Bot

[Atomics] Workaround for d8 worker limit

The previous change to futex.js will spawn too many workers in d8 in
some test configurations, which will throw an error.

This CL works around that by spawning fewer workers for the
Atomics.notify tests.

TBR=adamk@chromium.org

Bug: v8:7338
Change-Id: I0f3583781e5352b4d6672d43a087dc56d920122d
Reviewed-on: https://chromium-review.googlesource.com/1142895Reviewed-by: 's avatarBen Smith <binji@chromium.org>
Commit-Queue: Ben Smith <binji@chromium.org>
Cr-Commit-Position: refs/heads/master@{#54536}
parent 8043f283
......@@ -244,5 +244,7 @@ if (this.Worker) {
};
TestWakeMulti(Atomics.wake);
TestWakeMulti(Atomics.notify);
// TODO(binji): This is hitting d8's max worker count when run with multiple
// isolates. Re-enable when workers are cleaned up after termination.
// TestWakeMulti(Atomics.notify);
}
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment