Closed Bug 1094275 Opened 5 years ago Closed 5 years ago

Update the in-tree copy of jemalloc3

Categories

(Core :: Memory Allocator, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla37

People

(Reporter: ggp, Assigned: ggp)

References

Details

Attachments

(1 file, 4 obsolete files)

We need some of the recent changes to jemalloc3 in order to solve the remaining blockers for bug 762449.
Blocks: 801536
Blocks: 899126
Blocks: 1080851
Blocks: 762448
Could you attach the jemalloc 3 update patch here, or at least give the commit sha1 for what the other patches are based on?
Flags: needinfo?(ggoncalves)
Sure, here it is. I'm on 879e76a9e57 for now.
Oh, and here's a patch that imports a slightly outdated version of my bookkeeping patch for
bug 899126. I'm keeping it separate as we'll hopefully get that change upstreamed.
Flags: needinfo?(ggoncalves)
Depends on: 1108045
Can you refresh with current master, which contains the fix for bug 1108045? Then let's start the party by landing this.
Sure, here it is.
Attachment #8534309 - Attachment is obsolete: true
Attachment #8537145 - Flags: review?(mh+mozilla)
And here's the updated bookkeeping patch if you want to include it now too. Feel free to cancel
the review if you don't.
Attachment #8534310 - Attachment is obsolete: true
Attachment #8537146 - Flags: review?(mh+mozilla)
The previous patch builds on all architectures, but fails static rooting analysis on B2G Desktop Linux x64 [1]. This new version incorporates a patch by :sfink that fixes that [2].

1- https://treeherder.mozilla.org/#/jobs?repo=try&revision=336acb2b8869
2- https://treeherder.mozilla.org/#/jobs?repo=try&revision=55d9754cb127
Attachment #8537145 - Attachment is obsolete: true
Attachment #8537145 - Flags: review?(mh+mozilla)
Attachment #8537339 - Flags: review?(mh+mozilla)
Comment on attachment 8537146 [details] [diff] [review]
Patch jemalloc3 so it computes bookkeeping.

Shouldn't this be in bug 899126?
Comment on attachment 8537339 [details] [diff] [review]
Update the in-tree copy of jemalloc3.

Review of attachment 8537339 [details] [diff] [review]:
-----------------------------------------------------------------

https://hg.mozilla.org/integration/mozilla-inbound/rev/b5b1a8e8aee4
Attachment #8537339 - Flags: review?(mh+mozilla) → review+
Depends on: 1112434
https://hg.mozilla.org/mozilla-central/rev/b5b1a8e8aee4
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Depends on: 1112566
Depends on: 1113408
Comment on attachment 8537146 [details] [diff] [review]
Patch jemalloc3 so it computes bookkeeping.

I'll leave a comment about this patch in bug 899126. Sorry it took so long.
Attachment #8537146 - Attachment is obsolete: true
Attachment #8537146 - Flags: review?(mh+mozilla)
Blocks: 1108045
No longer depends on: 1108045
Depends on: 1115793
You need to log in before you can comment on or make changes to this bug.