498188 - mozconfig-wince should contain --enable-jemalloc option

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[dynamis (Tomoya ASAI)]

Without jemalloc wince build will crash really often.
We should contain next line in the mozconfig-wince for tryserver

ac_add_options --enable-jemalloc

http://hg.mozilla.org/build/buildbot-configs/file/tip/tryserver/mozconfig-wince

Comment 1

[:Gavin Sharp [email: [e-mail]]]

Wouldn't it be better to just change the default value of MOZ_MEMORY in configure for wince?

Comment 2

[timeless]

wtf. why would the build crash w/o jemalloc? we don't really want default configs that don't work.

Comment 3

[dynamis (Tomoya ASAI)]

Wince memory handling is poor to run fennec and jemalloc solve it.
You'll see crash or noise-only canvas browser when you run wince fennec w/o jemalloc (checked with HTC Touch Pro, WILLCOM 03 and Emulator images). You'll clealy understand once you try.
See Blad's note about wince a1 and jemalloc: 
http://blog.mozilla.com/blassey/2009/05/15/fennec-alpha-1-for-windows-mobile/

I'm not sure yet about how it will be with MOZ_MEMORY. If it's actually better, set it please. Anyway, current default tryserver wince builds are completely useless and need fix.

Comment 4

[dynamis (Tomoya ASAI)]

see bug 478044 for more about jemalloc on wince.

Comment 5

[(not currently active) Ted Mielczarek]

(In reply to comment #1)
> Wouldn't it be better to just change the default value of MOZ_MEMORY in
> configure for wince?

Yes, we should do this. Windows Mobile builds without jemalloc are not functional, AFAIK.

CCing vlad, because I'm not sure if WinCE 6 needs this, or just Windows Mobile.

Comment 6

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

CE6 doesn't seem to need it, but it wants it anyway.

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Adds ac_add_options --enable-jemalloc to mozconfig-wince for tryserver

Will try this in try staging.

Comment 8

[(not currently active) Ted Mielczarek]

We really should move this to Build Config and fix it Right.

Comment 9

[Aki Sasaki (not active)]

I filed bug 498531 for Ted; we'll check in this mozconfig in the non-downtime tomorrow for the quick fix.

Comment 10

[Aki Sasaki (not active)]

Comment on attachment 383370 [details] [diff] [review]
Adds ac_add_options --enable-jemalloc to mozconfig-wince for tryserver

revision 1199:05c5b2548d34