diff options
author | athenian200 <athenian200@outlook.com> | 2020-05-31 11:48:49 -0500 |
---|---|---|
committer | wolfbeast <mcwerewolf@wolfbeast.com> | 2020-06-01 11:00:26 +0200 |
commit | 451509e2c0188a4164d4b3d1d9f5839ed1e95246 (patch) | |
tree | 6f83d663bc761c91eef68caa72c2593e869b16f6 /mailnews/compose/src | |
parent | 7895b8acd9ad145bc39b39bddbc4a6ccb1ce27d8 (diff) | |
download | uxp-28.9-platform.tar.gz |
Issue #1571 - Remove JEMALLOC_USES_MAP_ALIGN and fix 48-bit addressing on SunOS AMD64.28.9-platform
The JEMALLOC_USES_MAP_ALIGN code has turned out to be worse than useless, and in fact it breaks 64-bit SunOS. It's very old and turned out not to be needed anymore because the way the memory allocator works has changed since it was implemented. It prevented a fix I tried for 48-bit addressing from working properly. However, without either this code or the 48-bit addressing fix, the 64-bit version won't even start, which is why I thought the code was still needed.
https://bugzilla.mozilla.org/show_bug.cgi?id=457189
https://hg.mozilla.org/mozilla-central/rev/a26c500b98ab
The 48-bit addressing fix is based on code found here:
https://github.com/OpenIndiana/oi-userland/blob/oi/hipster/components/web/firefox/patches/patch-js_src_gc_Memory.cpp.patch
I already applied these changes to js/src/gc/Memory.cpp, but as I was looking through jemalloc.c I saw that there were very similar ifdefs for Linux on SPARC as the ones I'd had to enaable in Memory.cpp, but for whatever reason the patches I found didn't touch them. So I tried doing for jemalloc.c what was already done for Memory.cpp, and it worked (but only after I removed the map align code).
Diffstat (limited to 'mailnews/compose/src')
0 files changed, 0 insertions, 0 deletions