Note: There are a few cases of duplicates in user autocompletion which are being worked on.

[ARMv6] Use more conservative GC settings for ARMv6

RESOLVED FIXED in Firefox 16

Status

()

Firefox for Android
General
RESOLVED FIXED
5 years ago
3 years ago

People

(Reporter: mfinkle, Unassigned)

Tracking

16 Branch
Firefox 17
x86_64
Linux
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox16 fixed)

Details

(Whiteboard: [ARMv6])

Attachments

(1 attachment)

Created attachment 653203 [details] [diff] [review]
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.
Attachment #653203 - Flags: review?(blassey.bugs)
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?
Attachment #653203 - Flags: review?(blassey.bugs) → review+
> 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

5 years ago
https://hg.mozilla.org/mozilla-central/rev/a2535133b802
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 17
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
Attachment #653203 - Flags: approval-mozilla-aurora?
Attachment #653203 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
http://hg.mozilla.org/releases/mozilla-aurora/rev/433dd8914307
status-firefox16: --- → fixed
Blocks: 792131

Updated

5 years ago
Whiteboard: [ARMv6]
Depends on: 995165
You need to log in before you can comment on or make changes to this bug.