Closed Bug 1296996 Opened 3 years ago Closed 3 years ago

Return of crash in mozilla::gl::SharedSurface_EGLImage::ProducerReleaseImpl: "MOZ_RELEASE_ASSERT(false) (GFX: Non-recycleable should not Fence twice.)"

Categories

(Core :: Graphics, defect, critical)

51 Branch
ARM
Android
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox50 --- unaffected
firefox51 --- fixed

People

(Reporter: njn, Assigned: eflores)

References

Details

(Keywords: crash, regression, topcrash, Whiteboard: [webvr])

Crash Data

Attachments

(1 file)

Bug 1238541 fixed the previous incarnation of this about 7 months ago, but it's just spiked again, e.g.: https://crash-stats.mozilla.com/report/index/7b505988-d473-49f5-a6b1-099f92160821.

The first Nightly affected is 20160819080504. There have been 56 crashes since then, making it the #1 Fennec topcrash for the past 7 days.

jgilbert, you fixed the previous bug -- any ideas what might have happened? Thanks.
Flags: needinfo?(jgilbert)
Depends on: 1285692
Flags: needinfo?(jgilbert)
Jeff, did you expect this to be fixed now with bug 1285692? I still see reports as of today's Nightly.
Flags: needinfo?(jgilbert)
Crashes by build ID:
> 20160824030337 	7 	4.86 %
> 20160823072522 	2 	1.39 %
> 20160823030224 	2 	1.39 %
> 20160822064441 	12 	8.33 %
> 20160821030226 	69 	47.92 %
> 20160820030224 	34 	23.61 %
> 20160819080504 	14 	9.72 %
Thanks Edwin - can you look for the fix as well?
Assignee: nobody → edwin
Flags: needinfo?(edwin)
Version: 46 Branch → 51 Branch
This seems to happen on many Android devices, but FWIW, I can reproduce with recent Nightlies on my Nexus 10 by loading https://lantea.kairo.at/ (which uses WebGL).
Oh actually given the regression range and the fact that jgilbert made this depend on bug 1285692 (which is *in* the regression range), I assume he meant to make this bug *block* bug 1285692 as the source of the regression.
Blocks: 1285692
No longer blocks: 1287653
No longer depends on: 1285692
Attached patch 1296996.patchSplinter Review
Flags: needinfo?(jgilbert)
Flags: needinfo?(edwin)
Attachment #8790643 - Flags: review?(jgilbert)
Attachment #8790643 - Flags: review?(jgilbert) → review+
Pushed by eflores@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee1070c230bd
Use ProducerRead{Acquire/Release} in ReadBuffer::Create - r=jgilbert
https://hg.mozilla.org/mozilla-central/rev/ee1070c230bd
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.