WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
50588
reserveAndCommit doesn't commit on MADVISE_FREE_REUSE systems
https://bugs.webkit.org/show_bug.cgi?id=50588
Summary
reserveAndCommit doesn't commit on MADVISE_FREE_REUSE systems
Geoffrey Garen
Reported
2010-12-06 14:17:17 PST
reserveAndCommit doesn't commit on MADVISE_FREE_REUSE systems
Attachments
Patch
(5.36 KB, patch)
2010-12-06 14:18 PST
,
Geoffrey Garen
mjs
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Geoffrey Garen
Comment 1
2010-12-06 14:18:51 PST
Created
attachment 75734
[details]
Patch
Geoffrey Garen
Comment 2
2010-12-06 14:20:05 PST
Ignore that WebKitTools change -- it's a webkit-patch artifact.
Maciej Stachowiak
Comment 3
2010-12-06 14:35:49 PST
Comment on
attachment 75734
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=75734&action=review
Looks good. See two comments inline. Make sure to dro the DumpRenderTree changes and it's r=me.
> JavaScriptCore/wtf/OSAllocatorPosix.cpp:43 > + // MADV_FREE_REUSE reservations start out in the MADV_FREE_REUSE state, but > + // the reserve() API promises something in the MADV_FREE_REUSABLE state.
This comment confuses me. I guess there is a relevant difference between FREE_REUSE and FREE_REUSABLE which should be highlighted here, but it's not explained. May want to tweak it.
> WebKitTools/DumpRenderTree/mac/DumpRenderTree.mm:-353 > - // maps expected result <= JavaScript expression > - NSDictionary *expressions = [NSDictionary dictionaryWithObjectsAndKeys: > - @"0", @"0", > - @"0", @"'0'", > - @"", @"", > - @"", @"''", > - @"", @"new String()", > - @"", @"new String('0')", > - @"", @"throw 1", > - @"", @"{ }", > - @"", @"[ ]", > - @"", @"//", > - @"", @"a.b.c", > - @"", @"(function() { throw 'error'; })()", > - @"", @"null", > - @"", @"undefined", > - @"true", @"true", > - @"false", @"false", > - nil > - ]; > - > - NSAutoreleasePool *pool = [[NSAutoreleasePool alloc] init];
I don't think this, or the other DumpRenderTree changes, were intended.
Geoffrey Garen
Comment 4
2010-12-06 15:14:22 PST
Committed revision 73405.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug