Issue metadata
Sign in to add a comment
|
2.7%-13.1% regression in webrtc_perf_tests at 19308:19309 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Aug 11 2017
Increased memory usage due to https://codereview.webrtc.org/2994093002. philipel: Please close if the increase is within the expectations :)
,
Aug 11 2017
It's a bit unfortunate that it has such a big memory impact on large meetings (10% mem increase for 15 person meetings). It seems like it would be pretty safe to reduce from 512 to, say, 128 and still not hit the bug. WDYT? sprang@, do you know how common frames larger than 100 packets are?
,
Aug 11 2017
In practice the change in memory usage is not that big. A buffer of 512 packets has the size of ~720 kB, and in the context of running webrtc in chrome it doesn't matter much. Also, this is a temporary workaround, so it should be gone in M63. Even if we wanted to reduce the start size there is a risk of frames being larger than 128 packets when doing screensharing.
,
Aug 11 2017
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Aug 11 2017