All users were logged out of Bugzilla on October 13th, 2018

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

RESOLVED FIXED

Status

RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: pyang, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
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.
(Reporter)

Comment 1

4 years ago
Created attachment 8606937 [details]
about-memory

ni? ting for further investigation.
Flags: needinfo?(janus926)
(Reporter)

Updated

4 years ago
Blocks: 1121812
(Reporter)

Comment 2

4 years ago
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
(Reporter)

Comment 3

4 years ago
nominate 2.2? since memory issue.
blocking-b2g: --- → 2.2?
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
Flags: needinfo?(janus926)
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
Depends on: 1166591
Depends on: 1166592
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
(Reporter)

Comment 7

3 years ago
All following bugs fixed and it should be marked as resolved
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Updated

3 years ago
blocking-b2g: 2.2? → ---
You need to log in before you can comment on or make changes to this bug.