1014010 - jemalloc_stats function is not defined on ARM linux build

Status: RESOLVED INVALID

(Core :: XPCOM, defect)

Description

[Tatiana Meshkova (:tatiana)]

attempt to load about:memory on Jolla browser (arm-linux non android) failing with this console message:
JavaScript error: chrome://global/content/aboutMemory.js, line 49: NS_ERROR_XPC_GS_RETURNED_FAILURE: Component returned failure code: 0x80570016 (NS_ERROR_XPC_GS_RETURNED_FAILURE) [nsIJSCID.getService]

Looking deeply I've found that nsMemoryReporterManager::Init return NS_ERROR_FAILURE, due to if (!jemalloc_stats) { condition fail.

I think this is cause bug 984237, and possibly related to some bad defines around mozilla/memory/*malloc*

But I don't know that code very well and probably need some help

Comment 1

[Nicholas Nethercote [inactive]]

This is the kind of thing that glandium knows about :)

Comment 2

[Mike Hommey [:glandium]]

The first question I'd like to ask is how exactly are you building the Jolla browser? Because "arm-linux non android" is not exactly very explicit. The non-android arm linux builds on debian have the jemalloc_stats symbol exactly where it's supposed to be: in the iceweasel executable (which is the same as firefox).

Comment 3

[Tatiana Meshkova (:tatiana)]

Attachment: mozconfig.merqtxulrunner config file

Comment 4

[Tatiana Meshkova (:tatiana)]

Attachment: autoconf.mk

Comment 5

[Tatiana Meshkova (:tatiana)]

Attachment: mozilla-config.h C++ defines

Comment 6

[Mike Hommey [:glandium]]

That all looks "normal". You should have the jemalloc_stats symbol exported from the xulrunner executable.


> mk_add_options PROFILE_GEN_SCRIPT=@TOPSRCDIR@/build/profile_pageloader.pl

that script is long gone.

Comment 7

[Mike Hommey [:glandium]]

But I bet you're not using the xulrunner executable, but building your own stuff and linking libxul, or something in that vein. In all likeliness, it's missing the most important part: libmemory.a.

Comment 8

[Tatiana Meshkova (:tatiana)]

nice, adding -lmemory to linking fixed the problem, unlucky that -lmemory is not in any of .pc files currently

Comment 9

[Mike Hommey [:glandium]]

(In reply to Tatiana Meshkova (:tatiana) from comment #8)
> nice, adding -lmemory to linking fixed the problem, unlucky that -lmemory is
> not in any of .pc files currently

The code should work without (that's why there's an if (jemalloc_stats)).

Comment 10

[Mike Hommey [:glandium]]

That said, I'd say that nsMemoryReporterManager::Init "gracefully" returning an error when jemalloc_stats is not there is the expected outcome.