Closed Bug 1253202 Opened 6 years ago Closed 5 years ago

EXCEPTION_ILLEGAL_INSTRUCTION crashes in _ZN6thread7Builder3new20h78612e3b213f1cf4IvbE (Rust MP4)

Categories

(Core :: Audio/Video: Playback, defect)

x86
Windows NT
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Tracking Status
firefox46 --- unaffected
firefox47 + fixed

People

(Reporter: dbaron, Assigned: rillian)

References

Details

(Keywords: crash, topcrash)

Crash Data

This bug was filed from the Socorro interface and is 
report bp-4ef38e6e-2f20-4275-8ec8-e15c32160302.
=============================================================

Following the fix for bug 1252041, it appears to have been replaced by a different topcrash with similar characteristics.

See the full set of such crashes at:
https://crash-stats.mozilla.com/signature/?product=Firefox&release_channel=nightly&date=%3E%3D2016-03-01&signature=_ZN6thread7Builder3new20h78612e3b213f1cf4IvbE

CPUs observed so far:
Rank 	Cpu info 	  	  	  	  	Count 	%
1 	AuthenticAMD family 6 model 8 stepping 1 | 1 	29 	80.56 %
2 	AuthenticAMD family 6 model 10 stepping 0 | 1 	7 	19.44 %
Flags: needinfo?(giles)
via c++filt, this is thread::Builder::new::h78612e3b213f1cf4Ivb.

I'm guessing this the same bug but in the pre-built libstd we're using?
Hi Xidorn, since you fixed bug 1252014, I thought I'd start with you. Could you please help investigate since this is a top crash? Thanks!
Flags: needinfo?(quanxunzhen)
That was fixed by rillian, not me :)
Flags: needinfo?(quanxunzhen)
Hmph. I was just wondering about that. Sounds like we need to do a custom build of the standard library with sse2 generation turned off. :(
Assignee: nobody → giles
Flags: needinfo?(giles)
Depends on: 1253456
Alex, what do you think about adding an i585-pc-window-msvc (or i686+nosse2) rust-std target? This is another thing moving us away from being able to use rust-lang builds.
Flags: needinfo?(acrichton)
Seems reasonable to me, we recently added an i586-unknown-linux-gnu target (https://github.com/rust-lang/rust/pull/31629) with the expectation that we'd add more. I'll work on getting some nightlies produced for this.
Flags: needinfo?(acrichton)
Excellent, thanks!
Alex's change is now landed, but no builds yet and I'm away next week. Exploring runtime switching based on cpuid as a workaround in the meantime.
Depends on: 1255869
Depends on: 1258796
No crashes with a build id more recent than March 2 since re-enabling rust, so it looks like the runtime check in bug 1255869 is working. Leaving this open until we have the more robust i586-pc-windows-msvc standard library in place.
Depends on: 1261503
Depends on: 1262637
No return of this signature, and I don't see anything related in the top-crashers list, so confirming this is properly fixed now.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.