Issue metadata
Sign in to add a comment
|
41.2%-610.8% regression in webrtc_perf_tests at 11713:11713 |
||||||||||||||||||||||
Issue descriptionThis must be related to the following change. Please verify that the degradation is intended. https://chromium.googlesource.com/external/webrtc/+/18fcbcf48 c190b6248cd16ef044d1edb79cba040
,
Mar 1 2016
,
Mar 1 2016
I would expect this patch to be more complex than before, but 600% is a little bit too much. I will look into it and report back.
,
Mar 1 2016
It seems to me that the increase in complexity for mac is reasonable, since the VAD started running on the render thread. But in Windows it seems to be too much and it also leaks to the capture thread complexity, which points towards a threading problem in the IE (which we are already aware of). What is your view on it, Per? I will continue looking into it, but I suspect this extreme regression should disappear once we fix the threading model of the IE.
,
Mar 2 2016
Please disregard my blaming of the threading model in #4. The actual issue is in the vectorisation of the fmaf function. A fix was uploaded here: https://codereview.webrtc.org/1755943002/
,
Mar 2 2016
aluebs@ - once https://codereview.webrtc.org/1755943002/ lands and the affected perf graphs look good, please let us know if you want that fix to be considered for an M50 merge. The WebRTC 50 branch was cut at r11765 (aka https://codereview.webrtc.org/1739713002), so it currently includes this regression.
,
Mar 2 2016
No, I don't think a merge makes sense, since this component isn't enabled anywhere.
,
Mar 2 2016
Moving to M51 and downgrading to P2 per #7. Thanks for the clarification!
,
Mar 4 2016
I am marking this as fixed, since the CL has landed and the performance bots returned to reasonable levels. Please note that they are not back to the original levels, since the CL that introduced this regression added a VAD to the render side, so a increase in complexity is expected. But at least there are no 600% regressions anymore. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by minyue@chromium.org
, Mar 1 2016