Summary: | Convert small JIT pool tests into executable fuzzing | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Keith Miller <keith_miller> | ||||||||||||
Component: | New Bugs | Assignee: | Keith Miller <keith_miller> | ||||||||||||
Status: | REOPENED --- | ||||||||||||||
Severity: | Normal | CC: | commit-queue, ews-watchlist, mark.lam, msaboff, saam, tzagallo, webkit-bug-importer | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | WebKit Nightly Build | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=228036 | ||||||||||||||
Bug Depends on: | 226606, 228037 | ||||||||||||||
Bug Blocks: | |||||||||||||||
Attachments: |
|
Description
Keith Miller
2021-05-26 10:10:52 PDT
Created attachment 429768 [details]
Patch
Comment on attachment 429768 [details]
Patch
r=me
Created attachment 429770 [details]
Patch
Created attachment 429773 [details]
Patch for landing
Tools/Scripts/svn-apply failed to apply attachment 429773 [details] to trunk.
Please resolve the conflicts and upload a new patch.
Created attachment 430364 [details]
Patch for landing
Committed r278356 (238388@main): <https://commits.webkit.org/238388@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 430364 [details]. Re-opened since this is blocked by bug 226606 Created attachment 433516 [details]
Patch for landing
Committed r279916 (239665@main): <https://commits.webkit.org/239665@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 433516 [details]. Comment on attachment 433516 [details] Patch for landing View in context: https://bugs.webkit.org/attachment.cgi?id=433516&action=review We also are using the random executable allocation fuzzer for all our tests now: https://trac.webkit.org/changeset/279126/webkit > Source/JavaScriptCore/jit/ExecutableAllocationFuzz.cpp:44 > + static WeakRandom random(Options::seedOfVMRandomForFuzzer() ? Options::seedOfVMRandomForFuzzer() : cryptographicallyRandomNumber()); Why is this defined right here? Maybe we can unify w/ WeakRandom below? Kinda confusing to put this definition here and not use it until way later. I think this should also be a LazyNeverDestroyed like below? Maybe also should use a lock like below? Re-opened since this is blocked by bug 228037 |