Closed Bug 933354 Opened 11 years ago Closed 11 years ago

Talos SVG-ASAP (tsvg) regression from bug 923193 (transform-origin in SVG)

Categories

(Core :: SVG, defect)

27 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla28
Tracking Status
firefox26 --- unaffected
firefox27 + fixed
firefox28 --- fixed

People

(Reporter: mbrubeck, Assigned: roc)

References

Details

(Keywords: perf, regression, Whiteboard: [qa-])

Attachments

(1 file)

Bug 923193 caused a regression in the Talos tsvg benchmark on all platforms: https://groups.google.com/d/topic/mozilla.dev.tree-management/9KnYIRIZEnQ/discussion Regression: Mozilla-Inbound-Non-PGO - SVG-ASAP - WINNT 6.1 (ix) - 25% increase ------------------------------------------------------------------------------ Previous: avg 312.057 stddev 7.814 of 12 runs up to revision 9c8ab7e9ae41 New : avg 389.992 stddev 3.334 of 12 runs since revision 45d9e6cd3473 Change : +77.936 (25% / z=9.974) Graph : http://mzl.la/1bCTFyV This was partially fixed by bug 929021: https://groups.google.com/d/topic/mozilla.dev.tree-management/1t2NtGGMNqQ/discussion Improvement: Mozilla-Inbound-Non-PGO - SVG-ASAP - WINNT 6.1 (ix) - 7.7% decrease -------------------------------------------------------------------------------- Previous: avg 393.485 stddev 2.711 of 12 runs up to revision c79088ab4e0c New : avg 363.197 stddev 3.229 of 12 runs since revision c1578a4fc86d Change : -30.288 (7.7% / z=11.171) Graph : http://mzl.la/1acLZGt Filing this bug to track the remaining regression. See bug 929021 comment 11 and 12 for some relevant discussion.
Roc, should we backout bug 923193 or track and try to fix in FF27? Any reason to believe this isn't an important perf regression?
Flags: needinfo?(roc)
Attached patch fix (deleted) — Splinter Review
This should fix the rest of the regression.
Assignee: nobody → roc
Attachment #827130 - Flags: review?(cam)
Flags: needinfo?(roc)
Attachment #827130 - Flags: review?(cam) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Comment on attachment 827130 [details] [diff] [review] fix [Approval Request Comment] Bug caused by (feature/regressing bug #): 923193 User impact if declined: slight performance regression for SVG elements with transforms Testing completed (on m-c, etc.): just landed Risk to taking this patch (and alternatives if risky): seems low risk String or IDL/UUID changes made by this patch: none
Attachment #827130 - Flags: approval-mozilla-aurora?
Attachment #827130 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
It's hard to tell whether this regression is fixed or not since bug 922942 regressed Tsvg more in the meantime. I think I will back out bug 923193, this fix, and the fix in bug 929021. Then I will fold those patches together and reland them to see if there is still a regression.
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: