New issue
Advanced search Search tips

Issue 687264 link

Starred by 3 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocked on:
issue 659191



Sign in to add a comment

Remove V8Platform::NumberOfAvailableBackgroundThreads()

Project Member Reported by fdoray@chromium.org, Jan 31 2017

Issue description

Historically, 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.
 
Status: Assigned (was: Untriaged)

Comment 2 by fdoray@chromium.org, May 25 2017

Owner: ----
Status: Untriaged (was: Assigned)
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