Open
Bug 1445422
Opened 6 years ago
Updated 2 years ago
live maps at weather.com are slow (with profile)
Categories
(Core :: CSS Parsing and Computation, defect, P3)
Core
CSS Parsing and Computation
Tracking
()
NEW
People
(Reporter: asa, Unassigned)
References
Details
A user reported extremely slow live maps at weather.com on Firefox 58 and 59. He's got a powerful system (win 10 with an AMD Threadripper 1950x and 64gb of ddr4 3200 driving a 4K display) and this is his profile. https://t.co/ExkAi5nPVF for https://weather.com/weather/radar/interactive/l/USOH1375:1:US?layer=radar&overlays=currtrafficspeed
:mconley, thank you!
Flags: needinfo?(mconley)
Comment 2•6 years ago
|
||
There are a few things going wrong here at first glance, but the big one appears to involve Stylo / Rayon: https://perfht.ml/2r25HBM Content Process 5 (which is hosting at least one weather.com tab) is going absolutely bonkers. Hey emilio, got any insight?
Component: General → CSS Parsing and Computation
Flags: needinfo?(mconley) → needinfo?(emilio)
Comment 3•6 years ago
|
||
So, my profile of that page looks fairly different, fwiw, mostly spent in JS. https://perf-html.io/from-addon/calltree/?range=19.5583_21.8275&thread=2&threadOrder=0-2-1&v=3 That machine has 16 physical CPUs, which means that we probably choose too many threads for the thread-pool. It's kinda fishy that in that profile there are no restyle markers, but the profile is 1 month old. Seeing what's going on in the StyleThreads would be useful, but I suspect it's just Rayon scaling not-so-well, and the threads starving and consuming CPU. Bobby, we had a bug on file for Bas's machine (can't find it). Should we just clamp the number of style threads?
Flags: needinfo?(emilio) → needinfo?(bobbyholley)
Comment 4•6 years ago
|
||
That discussion was bug 1431285 comment 13. Still on my todo list, but haven't gotten to it, and likely won't get to it before heading out on PTO. I do think we should probably clamp, but I'd like us to measure per the aforementioned comment. I'd also like to understand that user's profile better. Could we get her/him to add StyleThread to the list of profiled threads?
Flags: needinfo?(bobbyholley)
Reporter | ||
Comment 6•6 years ago
|
||
(In reply to Emilio Cobos Álvarez [:emilio] (Away-ish from 27/04 to 09/05) from comment #5) > Asa, any chance? Thanks! I'm sorry. I lost track of the reporter. I'll try to find him.
Flags: needinfo?(asa)
Reporter | ||
Comment 7•6 years ago
|
||
(In reply to Asa Dotzler [:asa] from comment #6) > (In reply to Emilio Cobos Álvarez [:emilio] (Away-ish from 27/04 to 09/05) > from comment #5) > > Asa, any chance? Thanks! > > I'm sorry. I lost track of the reporter. I'll try to find him. Found him and asked for a new profile with StyleThread added. https://twitter.com/asadotzler/status/992309975698427904 for future reference.
Updated•6 years ago
|
Priority: -- → P3
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•