From Geoff: Since JSC::MachineThreads::tryCopyOtherThreadStack reads another thread's stack. We should use one of the ASan annotations to make ASan ignore this function or treat it in a more lax way.
<rdar://problem/19828843>
<rdar://problem/19853304>
Created attachment 246705 [details] Ignore JSC::MachineThreads::tryCopyOtherThreadStack
Comment on attachment 246705 [details] Ignore JSC::MachineThreads::tryCopyOtherThreadStack r=me
Comment on attachment 246705 [details] Ignore JSC::MachineThreads::tryCopyOtherThreadStack Clearing flags on attachment: 246705 Committed r180227: <http://trac.webkit.org/changeset/180227>
All reviewed patches have been landed. Closing bug.
This fixes run-webkit-tests --threaded
Geoff, did this actually fix the issue? Looking at bug 141990 comment 10, ASan can still crash under this function, because the violation happens in memcpy, not in this function.
Mark Lam replaced this fix with a different one in <https://trac.webkit.org/r180649>. We now hit this code path on worker tests too, and this fix was ineffective.
Created attachment 247361 [details] Patch with updated fix This patch was reviewed and r+'ed offline by Alexey.
The fix is landed in r180649: <http://trac.webkit.org/r180649>.