Bug 1374499 - Increase number of shmems pre-allocated by PChromiumCDM to 6. r?gerald draft
authorChris Pearce <cpearce@mozilla.com>
Tue, 20 Jun 2017 13:47:33 +1200
changeset 597038 a7fe66c6164fb60b5bff4f19e66631dae3b969b3
parent 597032 cf97f3510fdb35c96881668e881841e5486dd070
child 634113 0f6d95b8d976e242de2a43c41780a752277c2c97
push id64804
push userbmo:cpearce@mozilla.com
push dateTue, 20 Jun 2017 01:48:04 +0000
reviewersgerald
bugs1374499
milestone56.0a1
Bug 1374499 - Increase number of shmems pre-allocated by PChromiumCDM to 6. r?gerald Currently we preallocate 4 shmems for ferrying video frames between the GMP and CDM procecss. Our telemetry indicates that 88% of PChromiumCDM instances need only 4. So 12% are finding that insufficient, and are taking the slow path and sending frames back to the content process via a non-shared path while they allocate more shmem. If we increased our pre-allocation to 6 shmems, we'd guess correctly for 97%, and thus we'd avoid the slow path more often. MozReview-Commit-ID: 9HcrlskiCPq
modules/libpref/init/all.js
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -408,17 +408,17 @@ pref("media.gmp.decoder.h264", 0);
 pref("media.raw.enabled", true);
 #endif
 pref("media.ogg.enabled", true);
 pref("media.opus.enabled", true);
 pref("media.wave.enabled", true);
 pref("media.webm.enabled", true);
 
 pref("media.eme.chromium-api.enabled", true);
-pref("media.eme.chromium-api.video-shmems", 4);
+pref("media.eme.chromium-api.video-shmems", 6);
 
 #ifdef MOZ_APPLEMEDIA
 #ifdef MOZ_WIDGET_UIKIT
 pref("media.mp3.enabled", true);
 #endif
 pref("media.apple.mp3.enabled", true);
 pref("media.apple.mp4.enabled", true);
 #endif