1165836 - [mtbf] [Memory] queued ipc messages flood after long running

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Stability, defect)

Description

[Paul Yang [:pyang] (away)]

STR: running mtbf test for several hours

From memory report, we saw many queued ipc messages showed as 0 memory size.
Suspecting it's a leak.

Comment 1

[Paul Yang [:pyang] (away)]

Attachment: about-memory

ni? ting for further investigation.

Comment 2

[Paul Yang [:pyang] (away)]

Build ID               20150510162500
Gaia Revision          528ef60e7cda09ad43478065f5d33bda398fbeb7
Gaia Date              2015-05-08 23:40:58
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/8d04cc085cf5
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150510.200137
Firmware Date          Sun May 10 20:01:47 EDT 2015
Bootloader             L1TC100118D0

Comment 3

[Paul Yang [:pyang] (away)]

nominate 2.2? since memory issue.

Comment 4

[Ting-Yu Chou [:ting] (away)]

The destination [1] of AppStatusbar's _touchForwarder is not updated after reviveBrowser(). The iframe browser1, browser6, browser213, browser210, and browser212 are this case.

Still checking the others.

[1] https://github.com/mozilla-b2g/gaia/blob/v2.2/apps/system/js/app_statusbar.js#L12

Comment 5

[Ting-Yu Chou [:ting] (away)]

The Settings app is leaked through the observers registered to idle service [1], only the local hash table is cleared when ActoryDestroy() [2].

I will create bugs for this and comment 4 tomorrow.

[1] https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/file/81a304f176ea/dom/ipc/ContentParent.cpp#l4260
[2] https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/file/81a304f176ea/dom/ipc/ContentParent.cpp#l1853

Comment 6

[Ting-Yu Chou [:ting] (away)]

browser217 is leaked by |_lastDisplayedApp| of trusted_ui.js [1], but I think it is all right as it keeps at most one ContentParent.

[1] https://github.com/mozilla-b2g/gaia/blob/v2.2/apps/system/js/trusted_ui.js#L453

Comment 7

[Paul Yang [:pyang] (away)]

All following bugs fixed and it should be marked as resolved