1154445 - [Camera] Crash while stability testing, maybe in camera

Status: RESOLVED DUPLICATE of bug 1152407

(Core :: DOM: Core & HTML, defect)

Description

[Greg Grisco]

Saw the folloowing crash signature while running stability test:

[@ xpcObjectHelper::xpcObjectHelper | mozilla::dom::WrapNativeISupportsParent | mozilla::dom::PromiseBinding::Wrap | mozilla::dom::quota::QuotaManager::AbortCloseStoragesForProcess ]

From the logs, looks like camera was being used at time of crash.  cafbot will upload the logs.

Comment 1

[cafbot (PoC: ggrisco)]

Attachment: EXTRA file attachment -

Comment 2

[cafbot (PoC: ggrisco)]

Attachment: decoded minidump -

Comment 3

[cafbot (PoC: ggrisco)]

Observed on: 

Device: msm8909
Gonk Version: AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.129
Moz BuildID: 20150409002503
Manifest: https://www.codeaurora.org/cgit/quic/lf/b2g/manifest/tree/caf_AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.129.xml?h=release
Gecko Version: 37.0
Gaia:  http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=ea735c21bfb0d78333213ff0376fce1eac89ead6
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=ac12d858a58574b2db482b5eed61750a3e210040
Patches: bug 1150923, bug 1150924, bug 1133147, bug 1150271, bug 1142806, bug 1148641

Comment 4

[cafbot (PoC: ggrisco)]

Attachment: EXTRA file attachment - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.129

Comment 5

[cafbot (PoC: ggrisco)]

Attachment: decoded minidump - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.129

Comment 6

[Mike Habicher [:mikeh] (high bugzilla latency)]

Crash site:
http://mxr.mozilla.org/mozilla-b2g37_v2_2/source/xpcom/glue/nsISupportsUtils.h#141

Comment 7

[Mike Habicher [:mikeh] (high bugzilla latency)]

Looks like this starts here:
http://mxr.mozilla.org/mozilla-b2g37_v2_2/source/dom/quota/QuotaManager.cpp#1767

It looks like that particular line has been in the tree for a while. Dave, do you know anything about the QuotaManager? Any idea what might be going on here?

Comment 8

[Keven Kuo [:kkuo]]

Hi! Becker,

Please also help to take a look. Thanks

--
Keven

Comment 9

[Dave Hylands [:dhylands]]

I'm not familiar with the QuotaManager, but it looks like ContentParent::ShutDownProcess calls it explicitly:

https://dxr.mozilla.org/mozilla-central/source/dom/ipc/ContentParent.cpp#1636

Because this is failing during a stability test, I'm going to guess that there is some type of race happening where the quota manage is trying to close a storage that hasn't been completely opened or something like that.

Comment 10

[becker hsieh{:behsieh}]

I can not comment whether it was relative to camera from such limited information.
in current log from attached,I didn't see any camera error.
is this easy to reproduce.?
is it possible we can set environment here to reproduce this issue?

Comment 11

[becker hsieh{:behsieh}]

is this easy to reproduce.?
is it possible we can set environment here to reproduce this issue?

Comment 12

[Greg Grisco]

(In reply to becker hsieh{:behsieh} from comment #11)
> is this easy to reproduce.?
> is it possible we can set environment here to reproduce this issue?

This crash was found with monkey test and has only been seen once or twice so far, so it seems difficult to reproduce.

Comment 13

[becker hsieh{:behsieh}]

is it possible we could set the same environment with you to reproduce this issue?

Comment 14

[Greg Grisco]

To reproduce the environment you would need to setup some monkey test and run overnight/weekend.  So far we've only seen this crash on AU 129 (it did not reproduce on AU 132).  Maybe if it reproduces on next build we'll have more information to go on.

Comment 15

[cafbot (PoC: ggrisco)]

Observed on: 

Device: msm8909
Gonk Version: AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.135
Moz BuildID: 20150420002502
Manifest: https://www.codeaurora.org/cgit/quic/lf/b2g/manifest/tree/caf_AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.135.xml?h=release
Gecko Version: 37.0
Gaia:  http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=c15a2b6d3a783813959c2b3bffd2a131f4270b9e
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=ed7fd6f380d24ed2c7cd918c0406d91ca6edc23f
Patches: bug 1154439, bug 1154411, bug 1150924, bug 1133147, bug 1142806, bug 1152439

Comment 16

[cafbot (PoC: ggrisco)]

Attachment: EXTRA file attachment - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.135

Comment 17

[cafbot (PoC: ggrisco)]

Attachment: decoded minidump - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.135

Comment 18

[Mike Lee [:mlee]]

Jan,

Is this issue due to the same root cause as bug 1152407?

Thanks,
Mike

Comment 19

[cafbot (PoC: ggrisco)]

Observed on: 

Device: msm8909
Gonk Version: AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.137
Moz BuildID: 20150420002502
Manifest: https://www.codeaurora.org/cgit/quic/lf/b2g/manifest/tree/caf_AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.137.xml?h=release
Gecko Version: 37.0
Gaia:  http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=c15a2b6d3a783813959c2b3bffd2a131f4270b9e
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=ed7fd6f380d24ed2c7cd918c0406d91ca6edc23f
Patches: bug 1154439, bug 1133147, bug 1152439, bug 1142806, bug 1154411

Comment 20

[cafbot (PoC: ggrisco)]

Attachment: EXTRA file attachment - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.137

Comment 21

[cafbot (PoC: ggrisco)]

Attachment: decoded minidump - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.137

Comment 22

[cafbot (PoC: ggrisco)]

Observed on: 

Device: msm8909
Gonk Version: AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.138
Moz BuildID: 20150423002502
Manifest: https://www.codeaurora.org/cgit/quic/lf/b2g/manifest/tree/caf_AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.138.xml?h=release
Gecko Version: 37.0
Gaia:  http://git.mozilla.org/?p=releases/gaia.git;a=commit;h=b838d0e7c163e66660dcb6e387d8339944a7a30e
Gecko: http://git.mozilla.org/?p=releases/gecko.git;a=commit;h=5f087089472477031462d7cc4cd5de1f90139258
Patches: bug 1154439, bug 1133147, bug 1152439, bug 1156503, bug 1154411

Comment 23

[cafbot (PoC: ggrisco)]

Attachment: EXTRA file attachment - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.138

Comment 24

[cafbot (PoC: ggrisco)]

Attachment: decoded minidump - AU_LINUX_GECKO_LF.BR.1.2.3.00.00.00.000.138

Comment 25

[Greg Grisco]

This has become one of the top hitters affecting mtbf stability numbers.

Comment 26

[Mike Lee [:mlee]]

Jan,
Please confirm if this issue has the same root cause and solution as bug 1152407. If so can we mark this as a duplicate of it?

Andrew,
If this issue has a different root cause than bug 1152407 which Jan is working on, please have someone else on the DOM team look into this.


As you can see from CAF's (Greg) comment 25, this issue is significantly impacting reaching fxOS 2.2 MTBF goals.


Thanks!
Mike

Comment 27

[Jan Varga [:janv]]

I think it's the same root cause.

Comment 28

[Mike Lee [:mlee]]

Thanks Jan. Marking as a duplicate of bug 1152407 per comment 27.