997353 - Do not use compile-time page size for PowerPC

Status: RESOLVED FIXED

(Core :: Memory Allocator, defect)

Description

[Gustavo Luiz Duarte]

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 1

[Gustavo Luiz Duarte]

Attachment: Disable MALLOC_STATIC_SIZES on PowerPC

Comment 2

[Jan Beich]

Comment on attachment 8407737 [details] [diff] [review]
Disable MALLOC_STATIC_SIZES on PowerPC

Green on Try.

https://tbpl.mozilla.org/?tree=Try&rev=438f9bf04477

Comment 3

[Cameron Kaiser [:spectre]]

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.

Comment 4

[Mike Hommey [:glandium]]

Attachment: Bug 997353 - Make powerpc not use static page sizes in mozjemalloc.

powerpc covers powerpc and powerpc64.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/05a653181a64
Make powerpc not use static page sizes in mozjemalloc. r=njn

Comment 6

[Mike Hommey [:glandium]]

Comment on attachment 9069876 [details]
Bug 997353 - Make powerpc not use static page sizes in mozjemalloc.

Beta/Release Uplift Approval Request

• User impact if declined: Startup crash on powerpc (tier 3). This has been the case for a long time, and in many cases, downstreams have been patching this for just as long. It would be nice to get this in ESR68 so that the amount of required patching decreases.
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: Yes
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): The patch adds a preprocessor condition to include powerpc. It's practically NPOTB.
• String changes made/needed:

Comment 7

[Julien Cristau [:jcristau]]

Comment on attachment 9069876 [details]
Bug 997353 - Make powerpc not use static page sizes in mozjemalloc.

tier3 fix, approved for 68.0b8

Comment 8

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/05a653181a64

Comment 9

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/releases/mozilla-beta/rev/236675dcbcab