• Gabriel Charette's avatar
    Revert "v8::ItemParallelJob : Do not launch more Tasks than there are Items to process." · 2407b2bd
    Gabriel Charette authored
    This reverts commit 8a27c7d3.
    
    Reason for revert: 
    
    Having more tasks then work items is intentional in some use cases, i.e. Scavenging where RunInParallel() does parallel processing on a dynamic workload *after* the initial set of work items:
    
        {
          barrier_->Start();
          TimedScope scope(&scavenging_time);
          PageScavengingItem* item = nullptr;
          while ((item = GetItem<PageScavengingItem>()) != nullptr) {
            item->Process(scavenger_);
            item->MarkFinished();
          }
          do {
            scavenger_->Process(barrier_);
          } while (!barrier_->Wait());
          scavenger_->Process();
        }
    
    Original change's description:
    > v8::ItemParallelJob : Do not launch more Tasks than there are Items to process.
    > 
    > Except when there are 0 items. For some reason I don't quite understand yet, not
    > calling Run() on tasks_[0] when there are 0 items results in DCHECKs...
    > 
    > Bug: chromium:806237
    > Change-Id: I38c8fffde64a42f93f4efda492832651137eebd7
    > Reviewed-on: https://chromium-review.googlesource.com/888704
    > Commit-Queue: Gabriel Charette <gab@chromium.org>
    > Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#50924}
    
    TBR=gab@chromium.org,mlippautz@chromium.org
    
    Change-Id: Iad2ab16bb41f339de8e3fbca1c08c5d26b8a0111
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:806237
    Reviewed-on: https://chromium-review.googlesource.com/891186Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Commit-Queue: Gabriel Charette <gab@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#50928}
    2407b2bd
Name
Last commit
Last update
..
OWNERS Loading commit data...
array-buffer-collector.cc Loading commit data...
array-buffer-collector.h Loading commit data...
array-buffer-tracker-inl.h Loading commit data...
array-buffer-tracker.cc Loading commit data...
array-buffer-tracker.h Loading commit data...
barrier.h Loading commit data...
code-stats.cc Loading commit data...
code-stats.h Loading commit data...
concurrent-marking.cc Loading commit data...
concurrent-marking.h Loading commit data...
embedder-tracing.cc Loading commit data...
embedder-tracing.h Loading commit data...
gc-idle-time-handler.cc Loading commit data...
gc-idle-time-handler.h Loading commit data...
gc-tracer.cc Loading commit data...
gc-tracer.h Loading commit data...
heap-inl.h Loading commit data...
heap.cc Loading commit data...
heap.h Loading commit data...
incremental-marking-inl.h Loading commit data...
incremental-marking-job.cc Loading commit data...
incremental-marking-job.h Loading commit data...
incremental-marking.cc Loading commit data...
incremental-marking.h Loading commit data...
invalidated-slots-inl.h Loading commit data...
invalidated-slots.cc Loading commit data...
invalidated-slots.h Loading commit data...
item-parallel-job.h Loading commit data...
local-allocator.h Loading commit data...
mark-compact-inl.h Loading commit data...
mark-compact.cc Loading commit data...
mark-compact.h Loading commit data...
marking.cc Loading commit data...
marking.h Loading commit data...
memory-reducer.cc Loading commit data...
memory-reducer.h Loading commit data...
object-stats.cc Loading commit data...
object-stats.h Loading commit data...
objects-visiting-inl.h Loading commit data...
objects-visiting.cc Loading commit data...
objects-visiting.h Loading commit data...
remembered-set.h Loading commit data...
scavenge-job.cc Loading commit data...
scavenge-job.h Loading commit data...
scavenger-inl.h Loading commit data...
scavenger.cc Loading commit data...
scavenger.h Loading commit data...
setup-heap-internal.cc Loading commit data...
slot-set.h Loading commit data...
spaces-inl.h Loading commit data...
spaces.cc Loading commit data...
spaces.h Loading commit data...
store-buffer.cc Loading commit data...
store-buffer.h Loading commit data...
stress-marking-observer.cc Loading commit data...
stress-marking-observer.h Loading commit data...
stress-scavenge-observer.cc Loading commit data...
stress-scavenge-observer.h Loading commit data...
sweeper.cc Loading commit data...
sweeper.h Loading commit data...
worklist.h Loading commit data...