The current size 4096, using telemetry we can see that we can do about image decoding at around 10000 KB/s. This means we can do about 10240 bytes per ms. Since we have 5ms to do the decoding that means we should be able to 51200 bytes in 5ms. I'd suggest we change to 16384 as that gives a healthy margin.
Created attachment 681209 [details] [diff] [review] Increase the default to 16384 Increase this limit to 16384. Note tenfour reverted the changes in 666352 for performance reasons.
Attachment #681209 - Flags: review?(joe)
Comment on attachment 681209 [details] [diff] [review] Increase the default to 16384 (this is joe honest)
Attachment #681209 - Flags: review?(joe) → review+
Assignee: nobody → jmuizelaar
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Does this have any impact on memory consumption? I have no idea how many of these chunks we have live at once.
Only temporarily in the decoder. These chunks are written from data buffers we are already holding on to.
(In reply to Joe Drew (:JOEDREW! \o/) from comment #6) > Only temporarily in the decoder. These chunks are written from data buffers > we are already holding on to. And likely not really even in the decoder, because it doesn't make a copy of the source data.
You need to log in before you can comment on or make changes to this bug.