Closed Bug 1271166 Opened 8 years ago Closed 8 years ago

Reduce media threadpool size

Categories

(Core :: Audio/Video: Playback, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: jya, Unassigned)

Details

Attachments

(1 file)

The total threadpool size is 24 ; we no longer require such high value. it can be reduced.
A thread pool of 8 threads is already more than enough; especially considering that the media decoder use their own threading model.

Review commit: https://reviewboard.mozilla.org/r/51307/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/51307/
Attachment #8750144 - Flags: review?(cpearce)
Comment on attachment 8750144 [details]
MozReview Request: Bug 1271166: Reduce media thread pool size. r?cpearce

https://reviewboard.mozilla.org/r/51307/#review47971

Your commit comment says you're reducing the number of threads to 8, but the code says 4. Which is it?
Attachment #8750144 - Flags: review?(cpearce) → review+
https://reviewboard.mozilla.org/r/51307/#review47971

There's 2 threadpool type used; decoder and playback, 4 for each. so total is 8.
https://hg.mozilla.org/mozilla-central/rev/b5091adfb4ed
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: