Closed
Bug 557935
Opened 14 years ago
Closed 14 years ago
Show jemalloc stats in about:memory
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: cjones, Assigned: cjones)
References
Details
Attachments
(1 file)
|
3.16 KB,
patch
|
vlad
:
review+
|
Details | Diff | Splinter Review |
It's a TODO in the current code because of the weird way we link in jemalloc on linux.
|
Assignee | |
Comment 1•14 years ago
|
||
Note: we could also use libc's mallinfo() for non-jemalloc builds, but I'm not sure that I care about those.
Assignee: nobody → jones.chris.g
Attachment #437729 -
Flags: review?(vladimir)
Attachment #437729 -
Flags: review?(vladimir) → review+
Comment on attachment 437729 [details] [diff] [review] Report jemalloc stats in about:memory on linux ick, but ok! Can't we cheat and include jemalloc.h anyway? Worst case we could split up jemalloc.h into jemalloc-structs.h and jemalloc.h so that we could include the structs bit here. But this is probably ok for now.
|
|
Assignee | |
Comment 3•14 years ago
|
||
Yeah, I thought about splitting the header. I'll push with that.
|
|
Assignee | |
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/f223c7d06f91 (with split header, ran it by vlad again before pushing)
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•