Remove V8Platform::NumberOfAvailableBackgroundThreads() |
||
Issue descriptionHistorically, V8Platform::CallOnBackgroundThread() posted tasks to base::WorkerPool. When a task is posted to base::WorkerPool, it is scheduled on an idle thread (if any) or on a new thread. That means that an unbounded number of threads can be created as a result of calling V8Platform::CallOnBackgroundThread(). To avoid this explosion, v8 limits the number of pending V8Platform::CallOnBackgroundThread() tasks to V8Platform::NumberOfAvailableBackgroundThreads(). Now that V8Platform::CallOnBackgroundThread() posts tasks to base::TaskScheduler, which has a configurable maximum number of threads, V8Platform::NumberOfAvailableBackgroundThreads() could be removed.
,
May 25 2017
,
Nov 8 2017
Should this be assigned to V8 folks so they stop calling V8Platform::NumberOfAvailableBackgroundThreads(), or is the plan to make NumberOfAvailableBackgroundThreads() just always return a dummy value like 1? |
||
►
Sign in to add a comment |
||
Comment 1 by robliao@chromium.org
, Mar 29 2017