Bug 163633

Summary: REGRESSION (r207480): 3 Dromaeo tests failing
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: New BugsAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, benjamin, bfulgham, cdumez, cmarcelo, commit-queue, dbates, fpizlo, hyatt, ysuzuki
Priority: P2    
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Crashlog
none
the patch mark.lam: review+

Description Ryan Haddad 2016-10-18 17:10:29 PDT
https://build.webkit.org/builders/Apple%20Yosemite%20Release%20WK2%20%28Perf%29/builds/6396

Running Dromaeo/cssquery-jquery.html (53 of 152)
crash: Dromaeo/cssquery-jquery.html
FAILED
Finished: 110.979434 s

Running Dromaeo/cssquery-prototype.html (54 of 152)
crash: Dromaeo/cssquery-prototype.html
FAILED
Finished: 198.212171 s

Running Dromaeo/dom-attr.html (55 of 152)
crash: Dromaeo/dom-attr.html
FAILED
Finished: 29.326325 s
Comment 1 Alexey Proskuryakov 2016-10-18 21:02:04 PDT
This seems pretty urgent.
Comment 2 Ryan Haddad 2016-10-18 21:41:56 PDT
May be related to https://trac.webkit.org/changeset/207479?
Comment 3 Chris Dumez 2016-10-18 21:42:59 PDT
(In reply to comment #2)
> May be related to https://trac.webkit.org/changeset/207479?

Looks like the most suspicious indeed. It'd be good to see a crash trace to confirm. I tried to reproduce locally but it does not crash for me :/
Comment 4 Ryan Haddad 2016-10-19 14:50:18 PDT
Crashlogs on the bot point to https://trac.webkit.org/changeset/207480

Thread 17 Crashed:: WTF::AutomaticThread
0   com.apple.JavaScriptCore      	0x000000010a411159 0x10988c000 + 12079449
1   com.apple.JavaScriptCore      	0x000000010a3f90a4 bool WTF::ConditionBase::waitForSecondsImpl<WTF::Lock>(WTF::Lock&, double) + 340 (Condition.h:74)
2   com.apple.JavaScriptCore      	0x000000010a420db8 std::__1::__function::__func<WTF::AutomaticThread::start(WTF::Locker<WTF::LockBase> const&)::$_0, std::__1::allocator<WTF::AutomaticThread::start(WTF::Locker<WTF::LockBase> const&)::$_0>, void ()>::operator()() + 232 (AutomaticThread.cpp:133)
3   com.apple.JavaScriptCore      	0x000000010a425f72 WTF::threadEntryPoint(void*) + 178 (Threading.cpp:61)
4   com.apple.JavaScriptCore      	0x000000010a42638f WTF::wtfThreadEntryPoint(void*) + 15 (ThreadingPthreads.cpp:166)
5   libsystem_pthread.dylib       	0x00007fff8b83799d _pthread_body + 131
6   libsystem_pthread.dylib       	0x00007fff8b83791a _pthread_start + 168
7   libsystem_pthread.dylib       	0x00007fff8b835351 thread_start + 13
Comment 5 Ryan Haddad 2016-10-19 14:50:33 PDT
Created attachment 292114 [details]
Crashlog
Comment 6 Ryan Haddad 2016-10-19 15:02:48 PDT
Looks like that change may also be causing flaky crashes during LayoutTests:

https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK1%20(Tests)/r207566%20(699)/results.html
Comment 7 Filip Pizlo 2016-10-19 15:07:13 PDT
(In reply to comment #6)
> Looks like that change may also be causing flaky crashes during LayoutTests:
> 
> https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK1%20(Tests)/
> r207566%20(699)/results.html

Looking.
Comment 8 Filip Pizlo 2016-10-19 15:28:10 PDT
Created attachment 292120 [details]
the patch
Comment 9 Filip Pizlo 2016-10-19 15:31:22 PDT
Comment on attachment 292120 [details]
the patch

View in context: https://bugs.webkit.org/attachment.cgi?id=292120&action=review

> Source/WTF/wtf/ParkingLot.cpp:708
> -    
> +

Reverted.
Comment 10 Mark Lam 2016-10-19 15:54:01 PDT
Comment on attachment 292120 [details]
the patch

r=me if EWS complaints are not relevanl.
Comment 11 Filip Pizlo 2016-10-19 15:55:36 PDT
(In reply to comment #10)
> Comment on attachment 292120 [details]
> the patch
> 
> r=me if EWS complaints are not relevanl.

Thanks, I will run tests locally to be sure.
Comment 12 Filip Pizlo 2016-10-19 16:40:33 PDT
Landed in https://trac.webkit.org/changeset/207577.

Lets hope that's it.