mozjemalloc's Valgrind annotations are bogus and, and mozjemalloc should never be called when Valgrind is running.
Created attachment 8477874 [details] [diff] [review] Remove the bogus Valgrind annotations from mozjemalloc This patch removes the annotations and adds an abort that is triggered if jemalloc is initialized while running under Valgrind.
Attachment #8477874 - Flags: review?(mh+mozilla)
Attachment #8477874 - Flags: review?(mh+mozilla) → review+
Immediately after landing it, I realized the above patch has a problem: it assumes that "running Valgrind" means "running Memcheck". But if you're running Cachegrind, for example, it's better to run with jemalloc enabled, because that's the standard configuration. So I removed the abort: https://hg.mozilla.org/integration/mozilla-inbound/rev/8d937c3db92b
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.