1015852 - [dolphin][monkey test] monkey test crash at libxul.so!mozilla::gl::GLContextProviderEGL::CreateForWindow(nsIWidget*)

Status: RESOLVED WORKSFORME

(Core :: Graphics: Layers, defect)

Description

[Peipei Cheng (needinfo if you need my action)]

Operating system: Android
                  0.0.0 Linux 3.10.17 #1 PREEMPT Tue May 13 06:17:35 CST 2014 armv7l Spreadtrum/scx15_sp7715gaplus/scx15_sp7715ga:4.4.2/KOT49H/107:userdebug/test-keys
CPU: arm
     1 CPU

Crash reason:  SIGSEGV
Crash address: 0x0

Thread 19 (crashed)
 0  libxul.so!mozilla::gl::GLContextProviderEGL::CreateForWindow(nsIWidget*) [GLContextProviderEGL.cpp : 685 + 0x2]
     r4 = 0x00000018    r5 = 0xb6539044    r6 = 0xb6ea2431    r7 = 0xb1c017a4
     r8 = 0xb64c97f4    r9 = 0x00000000   r10 = 0xb20f02f0    fp = 0xb1c018e4
     sp = 0xb1c01790    lr = 0xb6ea049d    pc = 0xb517ae80
    Found by: given as instruction pointer in context
 1  libxul.so!mozilla::layers::CompositorOGL::CreateContext() [CompositorOGL.cpp : 178 + 0x3]
     r4 = 0x00000000    r5 = 0xb1c019a4    r6 = 0xafc51b40    r7 = 0xb002da10
     r8 = 0xb1c01b00    r9 = 0x00000000   r10 = 0x00000001    fp = 0xb5f3b9a5
     sp = 0xb1c018e0    pc = 0xb51dde53
    Found by: call frame info
 2  libxul.so!mozilla::layers::CompositorOGL::Initialize() [CompositorOGL.cpp : 247 + 0x7]
     r4 = 0xb002da10    r5 = 0xb1c019a4    r6 = 0xafc51b40    r7 = 0xb002da10
     r8 = 0xb1c01b00    r9 = 0x00000000   r10 = 0x00000001    fp = 0xb5f3b9a5
     sp = 0xb1c018f8    pc = 0xb51df97b
    Found by: call frame info

Comment 1

[Peipei Cheng (needinfo if you need my action)]

Since the log file is big. I uploaded it here: https://www.dropbox.com/s/vmhxjm1rsd8hqx8/1015852.tar.bz2

Comment 2

[Ivan Tsay (:ITsay)]

Hi Peter,

Please help to take a look this crash. assign to someone else if you are overloaded.

Comment 3

[Peter Chang[:pchang]]

Jerry, please help to check it.

Comment 4

[Jerry Shih[:jerry] (UTC+8) (inactive)]

We got MOZ_CRASH() at:
https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/file/334e9712f079/gfx/gl/GLContextProviderEGL.cpp#l685

I will update a patch to print more log for debugging.

Comment 5

[Preeti Raghunath(:Preeti)]

Milan

Is this actionable? DO we have enough info to carry on?

Comment 6

[Milan Sreckovic [:milan] (needinfo for best results)]

Well, a way to reproduce it would be nice :)  Peter, which file in the archive from comment 1 is the logcat of the run that crashed?

Comment 7

[Ivan Tsay (:ITsay)]

(In reply to Jerry Shih[:jerry] from comment #4)
> We got MOZ_CRASH() at:
> https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/file/334e9712f079/gfx/gl/
> GLContextProviderEGL.cpp#l685
> 
> I will update a patch to print more log for debugging.

Hi Jerry,

When will the patch be available? We can have our QA run your patch after you provide it.

Comment 8

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Attachment: test_egl_config.patch

add log to check egl config error

pcheng, could you please apply this gecko debug patch and run the monkey test again?
Then, please provide the adb log and the mini dump info.

Comment 9

[Jerry Shih[:jerry] (UTC+8) (inactive)]

attachment 8434054 [details] [diff] [review] is for gecko v1.4

Comment 10

[Peipei Cheng (needinfo if you need my action)]

james, could you please help?

Comment 11

[James Zhang (Spreadtrum)]

Sam, land this patch for debugging.

Comment 12

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Hi Sam,
Please provide the adb log and dump data if this issus occurs.

Comment 13

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Hi James,
What's the current status of this issue? Do we have additional info about this bug?

Comment 14

[James Zhang (Spreadtrum)]

Loop Sam.
Sam, do you see this crash with this patch?

Comment 15

[Milan Sreckovic [:milan] (needinfo for best results)]

(In reply to James Zhang (Spreadtrum) from comment #14)
> Loop Sam.
> Sam, do you see this crash with this patch?

Just to be clear, we don't expect this patch to fix the problem. If the problem is gone, it is because of some other reason. This patch only adds more debugging information so that we can try and figure out the fix without being able to reproduce it internally.  So, we'd want the logs from the crash (if still there) with this patch applied, as the log would have more information in it.

Comment 16

[James Zhang (Spreadtrum)]

We haven't met this crash with log patch. Let's wait for reproducing.

Comment 17

[Preeti Raghunath(:Preeti)]

Moving this to nom in case we see this issue. If the issue is not seen soon I'll backlog it.

Comment 18

[Wayne Chang [:wchang]]

Minusing this as no reproduction has seen or been informed about since last triage.

Comment 19

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Has this reproduced?  If not should we close this as WFM for now?

Comment 20

[Peipei Cheng (needinfo if you need my action)]

james, I would like to close this bug as works for me since it hasn't been reproduced for about one month. We can reopen it if it happens again.