I am getting the following assertion when attempting to start Fennec, a xulrunner based application, on Ubunto 8.10 jemalloc.c:5801: Failed assertion: "((alignment - 1) & alignment) == 0 && alignment >= sizeof(void *)" Stack looks like this: Stack: UNKNOWN [/lib/tls/i686/cmov/libpthread.so.0 +0x0000E618] abort+0x00000188 [/lib/tls/i686/cmov/libc.so.6 +0x0002D268] memalign+0x000000AF [../../mobile-debug/mobile/dist/bin/fennec +0x0001FCEA] __libc_memalign+0x00000042 [/lib/tls/i686/cmov/libc.so.6 +0x00073A42] ___tls_get_addr+0x000000AF [/lib/ld-linux.so.2 +0x0001081F] UNKNOWN [/home/mfinkle/Source/mozilla-trunk/mobile-debug/mobile/dist/bin/xulrunner/libxul.so +0x01473BAD] NS_InitXPCOM3_P+0x000000D9 [/home/mfinkle/Source/mozilla-trunk/mobile-debug/mobile/dist/bin/xulrunner/libxul.so +0x0140FBBC] UNKNOWN [/home/mfinkle/Source/mozilla-trunk/mobile-debug/mobile/dist/bin/xulrunner/libxul.so +0x001CD847] XRE_main+0x000021BD [/home/mfinkle/Source/mozilla-trunk/mobile-debug/mobile/dist/bin/xulrunner/libxul.so +0x001D3800] UNKNOWN [../../mobile-debug/mobile/dist/bin/fennec +0x000024AD] __libc_start_main+0x000000E5 [/lib/tls/i686/cmov/libc.so.6 +0x00016685]
Component: Keyboard: Navigation → jemalloc
QA Contact: keyboard.navigation → jemalloc
Created attachment 410067 [details] [diff] [review] Patch Bug analysis for the record: this started asserting because a boolean TLS variable was added, with size == 1 on Ubuntu. jemalloc's memalign asserts that the alignment is at least pointer-width. We can fix just by going up to that size if necessary; the alignment guarantee offered by the memalign API is still upheld.
Assignee: nobody → dmandelin
Status: NEW → ASSIGNED
Attachment #410067 - Flags: review?
Attachment #410067 - Flags: review? → review?(pavlov)
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
I'd love to see this fix moved over to 1.9.2 to make debugging fennec on n900 possible without patch
I can reproduce in on Fedora12/Firefox 3.6.b3. It would be great to have it in 1.9.2.
Attachment #410067 - Flags: approval188.8.131.52?
Comment on attachment 410067 [details] [diff] [review] Patch Approving this because we believe it's still hurting the ability to debug fennec. If that's stopped being a problem for other reasons, then we should clear this approval rather than adding what looks like workaround code, right?
Attachment #410067 - Flags: approval184.108.40.206? → approval220.127.116.11+
Attachment #410067 - Flags: approval18.104.22.168+ → approval22.214.171.124?
lets take this, should have done long ago
Comment on attachment 410067 [details] [diff] [review] Patch a=LegNeato for 126.96.36.199. Please ONLY land this on mozilla-1.9.2 default, as we are still working on 188.8.131.52 on the relbranch
Attachment #410067 - Flags: approval184.108.40.206? → approval220.127.116.11+
status1.9.2: --- → .6-fixed
Attachment #410067 - Flags: approval18.104.22.168+ → approval22.214.171.124+
You need to log in before you can comment on or make changes to this bug.