Closed Bug 997353 Opened 6 years ago Closed 1 year ago
Do not use compile-time page size for Power
47 bytes, text/x-phabricator-request
|Details | Review|
The PowerPC architecture allows various memory page sizes and the possible page sizes are not bound to 64 or 32 bits, making it impossible to predict the runtime page size. If I try to run the JS engine (built for 32 bits) on top of a kernel configured with 64k memory page size, I get the message "Compile-time page size does not divide the runtime one". That is because jemalloc assumes PPC32 has 4k page size while PPC64 has 64k page size. No such assumption can be made for the PowerPC architecture. This is a real use-case for the Fedora PowerPC infrastructure, as we use PPC64 servers to build 32bit binary packages, and we have been carrying this patch since mozilla-11 (no idea why no one ever sent it upstream until now).
Comment on attachment 8407737 [details] [diff] [review] Disable MALLOC_STATIC_SIZES on PowerPC Green on Try. https://tbpl.mozilla.org/?tree=Try&rev=438f9bf04477
This shouldn't affect TenFourFox since we don't actually use jemalloc. glandium is probably the right reviewer for this, or knows who would be.
Assignee: nobody → gustavold
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/05a653181a64 Make powerpc not use static page sizes in mozjemalloc. r=njn
You need to log in before you can comment on or make changes to this bug.