Summary: | WorkQueue::concurrentApply() and ParallelJobs do the same thing | ||
---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
Component: | Web Template Framework | Assignee: | Nobody <webkit-unassigned> |
Status: | NEW --- | ||
Severity: | Normal | CC: | sam, simon.fraser, thorton |
Priority: | P2 | ||
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Description
Simon Fraser (smfr)
2017-11-26 17:43:48 PST
My preference is to remove ParallelJobs and keep WorkQueue::concurrentApply(). Sam says:
> > Source/WebCore/platform/graphics/filters/FEMorphology.cpp:195
> > ParallelJobs<PlatformApplyParameters> parallelJobs(&WebCore::FEMorphology::platformApplyWorker, optimalThreadNumber);
>
> No strictly related to this patch, but I checked what we were using for
> ParallelJobs, saw that ParallelJobsLibdispatch (which I assume we are using)
> uses dispatch_apply with a global queue. I believe using DISPATCH_APPLY_AUTO
> for the queue will work better.
so that's worth testing.
|