Bug 181816 - Set the minimum executable allocator size properly
Summary: Set the minimum executable allocator size properly
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: JF Bastien
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-01-18 15:06 PST by JF Bastien
Modified: 2018-01-18 15:53 PST (History)
9 users (show)

See Also:

Attachments
patch (2.19 KB, patch)
2018-01-18 15:09 PST, JF Bastien 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description JF Bastien 2018-01-18 15:06:45 PST 
Executable allocator expects at least two page size's worth of allocation in certain conditions, and that causes some tests to now fail because they ask for less. Set that minimum correctly. We were already rounding up to a page size, so having a minimum of 2 page sizes is fine.
Comment 1 Radar WebKit Bug Importer 2018-01-18 15:07:19 PST 
<rdar://problem/36635533>
Comment 2 JF Bastien 2018-01-18 15:09:04 PST 
Created attachment 331664 [details]
patch
Comment 3 Saam Barati 2018-01-18 15:11:54 PST 
Comment on attachment 331664 [details]
patch

r=me
Comment 4 WebKit Commit Bot 2018-01-18 15:53:27 PST 
Comment on attachment 331664 [details]
patch

Clearing flags on attachment: 331664

Committed r227168: <https://trac.webkit.org/changeset/227168>
Comment 5 WebKit Commit Bot 2018-01-18 15:53:29 PST 
All reviewed patches have been landed.  Closing bug.