Bug 128109

Summary: Turn GenGC on
Product: WebKit Reporter: Mark Hahnenberg <mhahnenberg>
Component: JavaScriptCoreAssignee: Mark Hahnenberg <mhahnenberg>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, buildbot, cmarcelo, commit-queue, ggaren, kling, koivisto, ossy, rniwa
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from webkit-ews-11 for mac-mountainlion-wk2 none

Description Mark Hahnenberg 2014-02-03 10:44:49 PST
GenGC looks like it's almost ready for primetime. We'll start by enabling it on X86-64.
Comment 1 Mark Hahnenberg 2014-02-03 12:40:52 PST
Created attachment 223008 [details]
Patch
Comment 2 Oliver Hunt 2014-02-03 14:36:58 PST
Comment on attachment 223008 [details]
Patch

\o/
Comment 3 Build Bot 2014-02-03 14:41:19 PST
Comment on attachment 223008 [details]
Patch

Attachment 223008 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5653330361581568

New failing tests:
http/tests/security/contentSecurityPolicy/source-list-parsing-paths-01.html
Comment 4 Build Bot 2014-02-03 14:41:21 PST
Created attachment 223020 [details]
Archive of layout-test-results from webkit-ews-11 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-11  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 5 Geoffrey Garen 2014-02-03 14:41:54 PST
Please file follow-up bugs for known performance issues. (I believe you mentioned a SunSpider regression to me in person.)
Comment 6 Mark Hahnenberg 2014-02-03 14:50:42 PST
(In reply to comment #5)
> Please file follow-up bugs for known performance issues. (I believe you mentioned a SunSpider regression to me in person.)

I filed bug 128129 for the sunspider regression.
Comment 7 WebKit Commit Bot 2014-02-03 15:07:24 PST
Comment on attachment 223008 [details]
Patch

Clearing flags on attachment: 223008

Committed r163330: <http://trac.webkit.org/changeset/163330>
Comment 8 WebKit Commit Bot 2014-02-03 15:07:26 PST
All reviewed patches have been landed.  Closing bug.
Comment 9 Andreas Kling 2014-02-03 21:42:22 PST
Looks like 3% regression on Membuster.
Comment 10 Andreas Kling 2014-02-03 22:28:39 PST
Filed bug 128160 and bug 128161 to track Membuster and DYEB regressions.
Comment 11 Antti Koivisto 2014-02-04 03:44:46 PST
PLT and html5 loading test agree this was ~3% performance regression.
Comment 12 Csaba Osztrogonác 2014-02-04 08:21:28 PST
(In reply to comment #7)
> (From update of attachment 223008 [details])
> Clearing flags on attachment: 223008
> 
> Committed r163330: <http://trac.webkit.org/changeset/163330>

It seems the GenGC has a serious bug, because from this change
there are many flaky failures on the Apple Mac bots:
- http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK1%20%28Tests%29?numbuilds=100
- http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK2%20%28Tests%29?numbuilds=100
- http://build.webkit.org/builders/Apple%20MountainLion%20Release%20WK1%20%28Tests%29?numbuilds=100

Apple Mac EWS bots are broken too because of this regression.

Could you fix it as soon as possible please?
Comment 13 Mark Hahnenberg 2014-02-04 08:38:26 PST
(In reply to comment #12)
> (In reply to comment #7)
> > (From update of attachment 223008 [details] [details])
> > Clearing flags on attachment: 223008
> > 
> > Committed r163330: <http://trac.webkit.org/changeset/163330>
> 
> It seems the GenGC has a serious bug, because from this change
> there are many flaky failures on the Apple Mac bots:
> - http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK1%20%28Tests%29?numbuilds=100
> - http://build.webkit.org/builders/Apple%20Mavericks%20Release%20WK2%20%28Tests%29?numbuilds=100
> - http://build.webkit.org/builders/Apple%20MountainLion%20Release%20WK1%20%28Tests%29?numbuilds=100
> 
> Apple Mac EWS bots are broken too because of this regression.
> 
> Could you fix it as soon as possible please?

My guess is that these flaky tests depend on some object being finalized before the test times out. Technically this is a bad assumption because garbage collection isn't designed to run finalizers before a hard deadline. OTOH, it worked before so we should be able to make it work again. I'm working on a simple patch to fix this.