RESOLVED FIXED 33582
[BREWMP] Add MarkStack fastMalloc implementation for platforms without VirtualAlloc or mmap. 
https://bugs.webkit.org/show_bug.cgi?id=33582
Summary [BREWMP] Add MarkStack fastMalloc implementation for platforms without Virtua...
Kwang Yul Seo
Reported 2010-01-13 02:06:05 PST
Use fastMalloc and fastFree to implement MarkStack::allocateStack and MarkStack::releaseStack respectively. Set to the page size to 4096.
Attachments
MarkStack for platforms without page allocation (3.54 KB, patch)
2010-01-13 02:10 PST, Kwang Yul Seo 		no flags
DetailsFormatted DiffDiff
MarkStack for platforms without page allocation (3.55 KB, patch)
2010-01-13 02:15 PST, Kwang Yul Seo 		no flags
DetailsFormatted DiffDiff
MarkStack for platforms without page allocation (3.57 KB, patch)
2010-01-25 01:51 PST, Kwang Yul Seo 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Kwang Yul Seo
Comment 1 2010-01-13 02:10:25 PST
Created attachment 46437 [details] MarkStack for platforms without page allocation
WebKit Review Bot
Comment 2 2010-01-13 02:12:04 PST
Attachment 46437 [details] did not pass style-queue: Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1 JavaScriptCore/runtime/MarkStackNone.cpp:28: You should add a blank line after implementation file's own header. [build/include_order] [4] Total errors found: 1
Kwang Yul Seo
Comment 3 2010-01-13 02:15:31 PST
Created attachment 46438 [details] MarkStack for platforms without page allocation Fix style errors.
Kwang Yul Seo
Comment 4 2010-01-25 01:51:44 PST
Created attachment 47328 [details] MarkStack for platforms without page allocation Use PLATFORM(BREWMP)
Eric Seidel (no email)
Comment 5 2010-01-26 14:50:34 PST
I don't really know what these functions are for, so I don't know if this is sane.
Oliver Hunt
Comment 6 2010-01-26 15:14:34 PST
Comment on attachment 47328 [details] MarkStack for platforms without page allocation r=me
WebKit Commit Bot
Comment 7 2010-01-27 04:44:03 PST
Comment on attachment 47328 [details] MarkStack for platforms without page allocation Clearing flags on attachment: 47328 Committed r53922: <http://trac.webkit.org/changeset/53922>
WebKit Commit Bot
Comment 8 2010-01-27 04:44:09 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.