Bug 25945

Summary:

Add support for MADV_FREE to TCMalloc

Product:

WebKit

Reporter:

Cameron Zwarich (cpst) <zwarich>

Component:

JavaScriptCore

Assignee:

Cameron Zwarich (cpst) <zwarich>

Status:

RESOLVED FIXED

Severity:

Normal

Keywords:

InRadar

Priority:

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

Attachments:

Description	Flags
Proposed patch	mrowe: review+

Cameron Zwarich (cpst)

Reported 2009-05-21 13:24:24 PDT

We should do this.

Attachments
Proposed patch (2.47 KB, patch) 2009-05-21 13:36 PDT, Cameron Zwarich (cpst)	mrowe: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Cameron Zwarich (cpst)

Comment 1 2009-05-21 13:25:24 PDT

<rdar://problem/6910754>

Cameron Zwarich (cpst)

Comment 2 2009-05-21 13:36:40 PDT

Created attachment 30552 [details] Proposed patch

Mark Rowe (bdash)

Comment 3 2009-05-21 13:39:52 PDT

Comment on attachment 30552 [details] Proposed patch r=me. A comment on why we prefer MADV_FREE over MADV_DONTNEED could be make the subtleties in this code a little less confusing.

Cameron Zwarich (cpst)

Comment 4 2009-05-21 15:13:25 PDT

Landed in r43988 with a comment explaining the change as per your suggestion.

Note You need to log in before you can comment on or make changes to this bug.