783912 - [ARMv6] Use more conservative GC settings for ARMv6

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Mark Finkle (:mfinkle) (use needinfo?)]

Attachment: patch

Since ARNv6 devices are likely to have less available memory, we should use more conservative GC settings. Bug 782694 provides settings used by b2g for 128MB devices. Fennec on ARMv6 is looking to support devices with as little as 256MB RAM.

Gregor: Should we tweak the values for higher memory devices too? Most "high-end" Android devices are still in the 512MB-1GB RAM range.

Comment 1

[Brad Lassey [:blassey] (use needinfo?)]

Comment on attachment 653203 [details] [diff] [review]
patch

Review of attachment 653203 [details] [diff] [review]:
-----------------------------------------------------------------

It would be better if we could tune these dynamically at runtime. Can you file a bug for that as a follow up?

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

> It would be better if we could tune these dynamically at runtime. Can you
> file a bug for that as a follow up?

Filed bug 783943

http://hg.mozilla.org/integration/mozilla-inbound/rev/a2535133b802

Comment 3

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/a2535133b802

Comment 4

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 653203 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
User impact if declined: more OOM on armv6 builds
Testing completed (on m-c, etc.): a few nights
Risk to taking this patch (and alternatives if risky): low, we are only lowering armv6 settings
String or UUID changes made by this patch: none

I did not request beta approval since we are only targeting higher end armv6 phones for Fx15

Comment 5

[Mark Finkle (:mfinkle) (use needinfo?)]

http://hg.mozilla.org/releases/mozilla-aurora/rev/433dd8914307