Closed
Bug 1031553
Opened 11 years ago
Closed 11 years ago
b2g crashes during enabling data connection
Categories
(Core :: Graphics: Layers, defect, P1)
Tracking
()
People
(Reporter: tkundu, Assigned: sotaro)
References
Details
(Keywords: crash, Whiteboard: [caf-crash 240][caf priority: p1][CR 686673][b2g-crash])
Attachments
(3 files, 1 obsolete file)
Test steps:
1. Run music, send sms, call another phone from test device in automation.
2. Enable data connection on the device.
3. Device is rebooting and mini dumps are generated on the phone.
STR will be very difficult to follow but if you can give us right direction then we can enable additional logs which may help debugging this crash
Reporter | ||
Updated•11 years ago
|
blocking-b2g: --- → 2.0?
Flags: needinfo?(msreckovic)
Reporter | ||
Comment 1•11 years ago
|
||
This crash is happening multiple times on FFOS 2.0, msm8610 256MB device.
gaia: https://www.codeaurora.org/cgit/quic/lf/b2g/mozilla/gaia/tag/?h=0a25eff60fcc699687e45ba2ac8b9a3ab3782672&id=AU_LINUX_GECKO_B2G_KK_3.6.01.04.00.000.016
gecko: https://www.codeaurora.org/cgit/quic/lf/b2g/mozilla/gecko/tag/?h=0b83d5e6a54df9bfccdbb5ba662527bf657f5381&id=AU_LINUX_GECKO_B2G_KK_3.6.01.04.00.000.016
Reporter | ||
Updated•11 years ago
|
Whiteboard: [CR 686673]
Updated•11 years ago
|
Whiteboard: [CR 686673] → [caf priority: p1][CR 686673]
Comment 2•11 years ago
|
||
Crash observed on:
Device: msm8610
Gonk Version: AU_LINUX_GECKO_B2G_KK_3.6.01.04.00.000.016
Moz BuildID: 20140623000201
B2G Version: 2.0
Gecko Version: 32.0a2
Gaia: http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=729f214b887ce8efe7d870145d31acb2c6427817
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=f3721e7a9961f09722975d0d76af31130f8847ef
Updated•11 years ago
|
Component: General → Graphics: Layers
Keywords: crash
Product: Firefox OS → Core
Whiteboard: [caf priority: p1][CR 686673] → [caf priority: p1][CR 686673][b2g-crash]
Version: unspecified → 32 Branch
Comment 3•11 years ago
|
||
This also reproduces on v2.0 7x27a builds.
Comment 4•11 years ago
|
||
(Seen 90 times in the last week on v2.0 across all devices)
blocking-b2g: 2.0? → 2.0+
Comment 5•11 years ago
|
||
Observed on:
Device: msm8610
Gonk Version: AU_LINUX_GECKO_B2G_KK_3.6.01.04.00.000.020
Moz BuildID: 20140628000201
B2G Version: 2.0
Gecko Version: 32.0a2
Gaia: http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=91958625774ebe6c425adf322b09f1edf906578d
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=eab4f8ff82758f08f049e14d305e29fcc50cb292
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Tapas Kumar Kundu from comment #0)
> Created attachment 8447427 [details]
> stack trace
The crash happens withing egl library. We do not have a source code of egl library :-(
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → sotaro.ikeda.g
Assignee | ||
Comment 7•11 years ago
|
||
Bug 1029856 might affect to this bug.
Updated•11 years ago
|
Flags: needinfo?(msreckovic) → needinfo?(milan)
![]() |
||
Updated•11 years ago
|
status-b2g-v2.0:
--- → affected
Assignee | ||
Comment 8•11 years ago
|
||
It seems very difficult to reproduce the problem. one possibility is "gralloc buffer is null". As another possible cause, Bug 1029856 or Bug 1031527 might related to this bug.
Reporter | ||
Comment 9•11 years ago
|
||
(In reply to Sotaro Ikeda [:sotaro] from comment #6)
> (In reply to Tapas Kumar Kundu from comment #0)
> > Created attachment 8447427 [details]
> > stack trace
>
> The crash happens withing egl library. We do not have a source code of egl
> library :-(
I also agree :). I am wondering whether any operation of gecko can lead to this crash in our code or not.
I will try with test patch from bug 1029856 and bug 1031527 to see whether that is the root cause of this issue or not.
Assignee | ||
Comment 10•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #8448239 -
Flags: review?(jmuizelaar)
Comment 11•11 years ago
|
||
Comment on attachment 8448239 [details] [diff] [review]
patch - Add more validity checks to GrallocTextureSourceOGL
Review of attachment 8448239 [details] [diff] [review]:
-----------------------------------------------------------------
This patch worries me. I'm sort of worried that this patch is covering up a more serious issue and will make it harder to find. Do we have any idea why the GrallocTextureHost is no longer valid in these cases?
Assignee | ||
Comment 12•11 years ago
|
||
(In reply to Jeff Muizelaar [:jrmuizel] from comment #11)
> Comment on attachment 8448239 [details] [diff] [review]
> patch - Add more validity checks to GrallocTextureSourceOGL
>
> Review of attachment 8448239 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> This patch worries me. I'm sort of worried that this patch is covering up a
> more serious issue and will make it harder to find. Do we have any idea why
> the GrallocTextureHost is no longer valid in these cases?
There are two possibliteis.
- TextureHost does not have gralloc buffer since it's construction.
- TextureClient/TextureParent is destroyed and TextureClient had TextureFlags::DEALLOCATE_CLIENT flag.
Assignee | ||
Comment 13•11 years ago
|
||
Comment on attachment 8448239 [details] [diff] [review]
patch - Add more validity checks to GrallocTextureSourceOGL
Clear review request. I am going to address the concern.
Attachment #8448239 -
Flags: review?(jmuizelaar)
Comment 14•11 years ago
|
||
Observed on:
Device: msm8610
Gonk Version: AU_LINUX_GECKO_B2G_KK_3.6.01.04.00.000.017
Moz BuildID: 20140624000201
B2G Version: 2.0
Gecko Version: 32.0a2
Gaia: http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=9d2f7bd16a8dc0c74c97c5a40d2f0731f3dfff4b
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=32c226e5a7adbb95e9c4ee003dc9e64699da03e1
Comment 15•11 years ago
|
||
Observed on:
Device: msm8610
Gonk Version: AU_LINUX_GECKO_B2G_KK_3.6.01.04.00.000.021
Moz BuildID: 20140630000201
B2G Version: 2.0
Gecko Version: 32.0a2
Gaia: http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=c0c8ad187c0466285f2580531e09f8322996f561
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=ef0781044f85ee7f364236c074ab0b7644c38ff8
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 16•11 years ago
|
||
Change the patch based on the comment. Did the following changes.
- TextureClient create TextureHost only when the TextureClient allocated buffer.
- Create GrallocTextureSourceOGL only when GrallocTextureHostOGL has a gralloc buffer
- GrallocTextureHostOGL forgets GrallocTextureSourceOGL when GrallocTextureHostOGL forgets the gralloc buffer.
- Add pointer checks to GrallocTextureHostOGL
Attachment #8448239 -
Attachment is obsolete: true
Assignee | ||
Updated•11 years ago
|
Attachment #8449671 -
Flags: review?(nical.bugzilla)
Attachment #8449671 -
Flags: review?(jmuizelaar)
Updated•11 years ago
|
Flags: needinfo?(milan)
Updated•11 years ago
|
Attachment #8449671 -
Flags: review?(nical.bugzilla) → review+
Assignee | ||
Comment 17•11 years ago
|
||
Assignee | ||
Comment 18•11 years ago
|
||
Comment on attachment 8449671 [details] [diff] [review]
patch - Create GrallocTextureSourceOGL only when GrallocTextureHostOGL has a gralloc buffer
Cancel jeff's review. nical's review+ is enough.
Attachment #8449671 -
Flags: review?(jmuizelaar)
Assignee | ||
Comment 19•11 years ago
|
||
Comment 20•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Comment 21•11 years ago
|
||
status-b2g-v2.1:
--- → fixed
status-firefox31:
--- → wontfix
status-firefox32:
--- → fixed
status-firefox33:
--- → fixed
Updated•11 years ago
|
Whiteboard: [caf priority: p1][CR 686673][b2g-crash] → [caf-crash 240][caf priority: p1][CR 686673][b2g-crash]
Comment 22•11 years ago
|
||
New test case needs to be added. There is no existing test case.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Flags: in-moztrap?(rmead)
Comment 23•11 years ago
|
||
This is done with Automation. No test case added to moztrap.
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Flags: in-moztrap?(rmead)
Flags: in-moztrap-
Comment 24•10 years ago
|
||
Unable to verify this issue because it is an automation crash.
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-verify-][QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-verify-][QAnalyst-Triage?] → [QAnalyst-verify-][QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in
before you can comment on or make changes to this bug.
Description
•