983129 - AWSY: ~10 MiB regression in startup resident memory usage on March 13

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Kartikaya Gupta (email:[e-mail])]

Resident memory usage from the "StartSettled" data for fx-team push changesets:

e7f97fa8a86b : 149073920
a1639dd9dd3d : 150249472
c1651a5658df : 149106688
3a8c5cbeccea : 148983808
56ac183ad1e2 : 151355392
e6636696d6de : 149680128
7c458c70cd5c : 158826496 <-- regression!
3a186598d35e : 157863936
709e9207641d : 158973952
b2b8ba047803 : 158322688
b07f0ec10d22 : 159027200

https://tbpl.mozilla.org/?tree=Fx-Team&rev=7c458c70cd5c has 4 patches by Margaret, not sure which one caused it yet. The regression appears on the "explicit" data as well so this isn't just Android randomly loading in more stuff into resident memory.

Comment 1

[Kartikaya Gupta (email:[e-mail])]

From a quick look at the about:memory dumps there's a whole bunch of devtools stuff that wasn't there before, so this is probably bug 979101.

Comment 2

[:Margaret Leibovic]

jryans, have you ever noticed memory problems like this with B2G? I imagine the remote debugger is also disabled by default there, so maybe this hasn't been an issue.

I should probably just back out bug 979101, but that makes me sad.

Comment 3

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Yep, it's been a B2G issue too.  In fact, worse there to some degree, as there's not enough memory to run the Dev Tools at all on low end B2G devices.  :(

I am not sure there are any quick fixes here, so yes, you may have to back it out.

Comment 4

[:Margaret Leibovic]

Fixed by the backout in bug 979101.

Comment 5

[Kartikaya Gupta (email:[e-mail])]

Thanks for the quick fix!