Commit 0fe133cc authored by Omer Katz's avatar Omer Katz Committed by Commit Bot

[heap] Use less tasks for concurrrent marking

Use 1 less task for concurrent marking to allow Oilpan's concurrent
marking a chance to run as well.

Change-Id: I236f78925324dc4d21a38233d75dc101cc477930
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2062389
Commit-Queue: Omer Katz <omerkatz@chromium.org>
Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
Auto-Submit: Omer Katz <omerkatz@chromium.org>
Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#66311}
parent 2a936850
...@@ -524,7 +524,7 @@ void ConcurrentMarking::ScheduleTasks() { ...@@ -524,7 +524,7 @@ void ConcurrentMarking::ScheduleTasks() {
#else // defined(OS_MACOSX) #else // defined(OS_MACOSX)
// On other platforms use all logical cores, leaving one for the main // On other platforms use all logical cores, leaving one for the main
// thread. // thread.
total_task_count_ = Max(1, Min(kMaxTasks, num_cores - 1)); total_task_count_ = Max(1, Min(kMaxTasks, num_cores - 2));
#endif // defined(OS_MACOSX) #endif // defined(OS_MACOSX)
DCHECK_LE(total_task_count_, kMaxTasks); DCHECK_LE(total_task_count_, kMaxTasks);
// One task is for the main thread. // One task is for the main thread.
......
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