Bug 172650

Summary: Layout test workers/wasm-hashset.html is flaky
Product: WebKit Reporter: Matt Lewis <jlewis3>
Component: New BugsAssignee: Saam Barati <saam>
Status: RESOLVED CONFIGURATION CHANGED    
Severity: Normal CC: benjamin, fpizlo, ggaren, gskachkov, jfbastien, keith_miller, mark.lam, msaboff, ryanhaddad, saam, ticaiolima, webkit-bug-importer, ysuzuki
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=178755

Description Matt Lewis 2017-05-26 10:41:43 PDT
workers/wasm-hashset.html is a flaky timeout. 
On the flakiness dashboard the test is timing out on both Mac wk1 debugs and the Sierra WK2 Release.
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=workers%2Fwasm-hashset.html

currently the test is skipped on iOS and windows.

The test usually runs quickly with a 1-5 second run time but each time the test times out, it runs for 155 seconds.
The test was added with https://trac.webkit.org/changeset/215353/webkit
It was marked as slow in the same revision r215353.

https://build.webkit.org/results/Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/r217490%20(1478)/results.html
https://build.webkit.org/builders/Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/builds/1478
Comment 1 Saam Barati 2017-05-31 14:37:19 PDT
(In reply to Matt Lewis from comment #0)
> workers/wasm-hashset.html is a flaky timeout. 
> On the flakiness dashboard the test is timing out on both Mac wk1 debugs and
> the Sierra WK2 Release.
> https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.
> html#showAllRuns=true&tests=workers%2Fwasm-hashset.html
> 
> currently the test is skipped on iOS and windows.
> 
> The test usually runs quickly with a 1-5 second run time but each time the
> test times out, it runs for 155 seconds.
> The test was added with https://trac.webkit.org/changeset/215353/webkit
> It was marked as slow in the same revision r215353.
> 
> https://build.webkit.org/results/
> Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/r217490%20(1478)/results.html
> https://build.webkit.org/builders/
> Apple%20El%20Capitan%20Debug%20WK1%20(Tests)/builds/1478

Hmm, perhaps it's worth looking into this since the jump from 5 seconds to 155 seconds could implicate a deadlock somewhere. I'll look into it.
Comment 3 JF Bastien 2017-10-24 16:03:17 PDT
Might be unrelated, but my latest checkout has a 100% reproducible debug assert about a refcount somewhere in initNormalWorld's callees. I'll try to bisect to see where that started, it might lead to a release crash in the future.
Comment 4 JF Bastien 2017-10-24 16:25:10 PDT
(In reply to JF Bastien from comment #3)
> Might be unrelated, but my latest checkout has a 100% reproducible debug
> assert about a refcount somewhere in initNormalWorld's callees. I'll try to
> bisect to see where that started, it might lead to a release crash in the
> future.

I think my local build was messed up? Ignore me for now, will report back later...
Comment 5 JF Bastien 2017-10-26 08:25:45 PDT
Did this stop flaking after Saam reverted the bmalloc patch? I don't seem to have access to https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=workers%2Fwasm-hashset.html
Comment 6 Matt Lewis 2017-10-30 12:53:03 PDT
This has since stoped flaking according to the flakiness dashboard.
Comment 7 Radar WebKit Bug Importer 2017-11-15 13:11:59 PST
<rdar://problem/35568980>