Closed
Bug 1329309
Opened 7 years ago
Closed 6 years ago
Crash in sk_sse41_new::loop<T>
Categories
(Core :: Graphics, defect, P3)
Tracking
()
RESOLVED
FIXED
People
(Reporter: milan, Assigned: lsalzman)
References
Details
(Keywords: crash, stale-bug, Whiteboard: [gfx-noted])
Crash Data
Attachments
(1 file)
8.79 KB,
patch
|
mchang
:
review+
lizzard
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is report bp-1ba24e60-369f-426f-9ef8-952482170106. ============================================================= Common crash, but stops in 51. It could be Skia update, some fix in 52, or different set of people running release and beta. If we don't know what fixed it, we should track and see if it shows up in 52 when that hits beta.
Reporter | ||
Comment 1•7 years ago
|
||
#2 "DrawTargetSkia" crash.
Assignee | ||
Updated•7 years ago
|
OS: Windows 7 → All
Priority: -- → P1
Whiteboard: [gfx-noted]
Assignee | ||
Comment 2•7 years ago
|
||
This is just a backport of this upstream fix: https://skia.googlesource.com/skia/+/6e472093009bf2fc4a8e53010b51040efcb71213
Updated•7 years ago
|
Attachment #8824539 -
Flags: review?(mchang) → review+
Assignee | ||
Comment 3•7 years ago
|
||
Comment on attachment 8824539 [details] [diff] [review] remove deprecated SkOpts_sse41 blits Approval Request Comment [Feature/Bug causing the regression]: bug 1246756, so 47+ [User impact if declined]: Possible crashes when rendering on CPUs with SSE 4.1. [Is this code covered by automated tests?]: Yes, pretty much any test that does any rendering at all. [Has the fix been verified in Nightly?]: Yes, bug 1299435 which updated us to Skia milestone55 in 52 already has this patch rolled in and we've seen no reports of these crashes on 52+. [Needs manual test from QE? If yes, steps to reproduce]: no [List of other uplifts needed for the feature/fix]: beta [Is the change risky?]: No [Why is the change risky/not risky?]: Fix is already incorporated upstream and in fixes we made in 52. It just causes us to use the same SSE2 routines that are used for everyone else instead of the specialized SSE4.1 routines. [String changes made/needed]: None
Attachment #8824539 -
Flags: approval-mozilla-beta?
Comment 4•7 years ago
|
||
Comment on attachment 8824539 [details] [diff] [review] remove deprecated SkOpts_sse41 blits Crash fix for beta only. This should make it into the 51 beta 13 build today.
Attachment #8824539 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 5•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/ce55e4d27603
status-firefox51:
--- → fixed
Comment 6•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/3218ea763746
Comment 7•7 years ago
|
||
Crash volume for signature 'sk_sse41_new::loop<T>': - nightly (version 53): 0 crashes from 2016-11-14. - aurora (version 52): 0 crashes from 2016-11-14. - beta (version 51): 236 crashes from 2016-11-14. - release (version 50): 3315 crashes from 2016-11-01. - esr (version 45): 0 crashes from 2016-07-13. Crash volume on the last weeks (Week N is from 01-09 to 01-15): W. N-1 W. N-2 W. N-3 W. N-4 W. N-5 W. N-6 W. N-7 - nightly 0 0 0 0 0 0 0 - aurora 0 0 0 0 0 0 0 - beta 57 45 26 9 19 36 24 - release 658 546 461 343 339 399 280 - esr 0 0 0 0 0 0 0 Affected platform: Windows Crash rank on the last 7 days: Browser Content Plugin - nightly - aurora - beta #441 #574 - release #167 #172 - esr
status-firefox50:
--- → affected
This is an assigned P1 bug without activity in two weeks. If you intend to continue working on this bug for the current release/iteration/sprint, remove the 'stale-bug' keyword. Otherwise we'll reset the priority of the bug back to '--' on Monday, August 28th.
Keywords: stale-bug
Assignee | ||
Comment 10•6 years ago
|
||
This was essentially already fixed prior even to 52 ESR with the patches already in this bug. All "new" reports are only on unsupported released from before then. I'm just going to close this out as fixed since it should be marked so.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•