Closed Bug 1235839 Opened 8 years ago Closed 8 years ago

Categories

(Core :: Graphics, defect)

46 Branch
x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox46 --- affected

People

(Reporter: marco, Unassigned)

References

Details

(Keywords: perf, Whiteboard: gfx-noted)

Attachments

(1 file)

717.48 KB, application/x-7z-compressed
Details
The animation in "tag cloud" on the right causes scrolling to be really slow.
Attached file profile.7z
Flags: needinfo?(mchang)
Keywords: perf
Whiteboard: gfx-noted
This profile seems to be in a .bin format? What kind of profiler were you using with the attached profile? Thanks!
Flags: needinfo?(mchang) → needinfo?(mcastelluccio)
The Gecko Profiler add-on.
Flags: needinfo?(mcastelluccio)
From the profiles, I see super long 500+ms times to do XRender::Sync, _XSEND and XUnlockDisplay. I know we're planning to reduce our dependence on XRender, which should help this test case.
Depends on: 1241832
It's still slow, I can grab another profile (although bug 1241832 caused bug 1247935, so I don't know if it'll stick).
This is now fixed (both with gfx.xrender.enabled set to true or false), I wonder if this was fixed by the same fix for bug 1247935.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
(In reply to Marco Castelluccio [:marco] from comment #6)
> This is now fixed (both with gfx.xrender.enabled set to true or false), I
> wonder if this was fixed by the same fix for bug 1247935.

That makes sense, yes. As Mason noted, a lot of the profile was spent in XSync waiting for stuff to be drawn. So if there was somehow contention for the connection to the X display, the changes I made in bug 1247935 would massively reduce that contention, since it no longer uses the XSync mechanism at all.
No longer depends on: 1263222
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: