Closed Bug 649841 Opened 9 years ago Closed 9 years ago

First-run low-memory warning: raise to 512MB

Categories

(Firefox for Android Graveyard :: General, defect, P3)

ARM
Android
defect

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 6

People

(Reporter: tarend, Assigned: dougt)

References

Details

Attachments

(1 file)

Change from currently 256MB to 512 MB -->
Phones with less than 512MB memory should see the warning.

See also Bug 633649
Assignee: nobody → doug.turner
Attachment #528121 - Flags: review?(doug.turner) → review+
Landed: http://hg.mozilla.org/mozilla-central/rev/743b3a54b087
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Was this patch tested on devices with 512MB RAM capacity? 

A user filed report in bug 652787 on the 04/26 nightly is using an LG Optimus 2X [1], and a Nexus S [2]. Both devices have 512MB RAM capacity. This leads to a suprising message on first boot, which would indicate that both those devices are unsupported.

[1] http://pdadb.net/index.php?m=specs&id=2693&c=lg_p990_optimus_2x_lg_star
[2] http://pdadb.net/index.php?m=specs&id=2690&c=samsung_gt-i9020_nexus_s
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Blocks: 652787
I assume the confusion is around "with less than 512MB"

The patch is "less than or equal to 512MB":

>+ return Integer.parseInt(totalMem) <= 524288L;

which I assume should be:

>+ return Integer.parseInt(totalMem) < 524288L;
Correct me if I'm wrong, but MemTotal will only list usable not all all physical addressable space? As it seems 512MB of what would be be addressable by the android kernel, or user space is never the case. This check would then be insufficient and false, as phones with 512MB RAM will get this message.
Let's not reopen this bug if we have opened bug 652787. Only one bug is enough.
Status: REOPENED → RESOLVED
Closed: 9 years ago9 years ago
Resolution: --- → FIXED
Depends on: 632649
Verified Fixed - Mozilla/5.0 (Android; Linux armv7l; rv:6.0a1) Gecko/20110426 Firefox/6.0a1 Fennec/6.0a1 ID:20110426043230
Status: RESOLVED → VERIFIED
(In reply to comment #8)
> Verified Fixed - Mozilla/5.0 (Android; Linux armv7l; rv:6.0a1) Gecko/20110426
> Firefox/6.0a1 Fennec/6.0a1 ID:20110426043230

Which phone, and would you be able to cat the output from MemTotal  in /proc/meminfo, and post it for bug 652787?
Target Milestone: --- → Firefox 6
See Bug 659412 - Motorola Defy users report that they see the low memory warning.
You need to log in before you can comment on or make changes to this bug.