Issue metadata
Sign in to add a comment
|
WebGL Performance terrible on AMD Hardware since Chromium 55
Reported by
hee...@viadrupsteen.nl,
Dec 16 2016
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36 Steps to reproduce the problem: 1. Go to http://radio.garden on a AMD GPU machine 2. Open Developer tools and turn on FPS Meter 3. See that the FPS is really low, while this isn't the case for Nvidia. (~~6 Frames per Second) What is the expected behavior? Similar performance to Firefox (~~40 Frames Per Second) What went wrong? Performance is severely low. My guess is that something somewhere went wrong when Direct3D11 was supported on AMD, since Chromium 55 doesn't limit the GPU I'm using anymore (AMD Radeon HD 5750 at work / MSI Radeon R9 390 at home) while this used to be the case. Both work on different drivers. In Firefox performance is fine. Did this work before? Yes Chrome 54 Chrome version: 55.0.2883.87 Channel: stable OS Version: 10.0 Flash Version: Shockwave Flash 24.0 r0 Previous version of chrome limited AMD Gpu's to DirectX9
,
Jan 19 2017
,
Jan 19 2017
,
Jan 19 2017
Geoff, do you have an AMD GPU in the office and if so could you triage this quickly? It sounds like a potential problem with ANGLE. Let's also test Canary in case it's a problem that's already been fixed. Thanks.
,
Jan 19 2017
Was able to reproduce on an AMD system. Got about 30 FPS on bad Chrome versions and about 60 on good. It's fixed in Canary so I did a bisect-builds and narrowed down the fix to this ANGLE roll: https://chromium.googlesource.com/chromium/src/+/b5de482ad9cc882bf3ef9a73f3d75953a3350c44 which contains this CL: https://chromium.googlesource.com/angle/angle.git/+/408293f8a4dfa129eb47df996cc6881cefb9c9c2 which is a fix for an AMD workaround. We could try to merge this back to 56.
,
Jan 19 2017
,
Jan 20 2017
Thanks Geoff for tracking that down. Let's request a merge back. Since that bug didn't have a Chromium issue, not sure exactly how this works for tracking purposes.
,
Jan 20 2017
This bug requires manual review: We are only 10 days from stable. Please contact the milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), gkihumba@(cros), bustamante@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 20 2017
Marking fixed as per the merge request guidelines.
,
Jan 20 2017
We're too close to stable for this change to make sense, typically we're only taking high severity security/crash fixes right now, and this CL is pretty large. I'd like to wait until M-57, if you think this should still qualify. Please add the "Merge-Review-56" label with a justification and we'll review it.
,
Jun 20 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by brajkumar@chromium.org
, Dec 19 2016Labels: TE-NeedsTriageFromMTV