132629 – Do the bmalloc.

RESOLVED FIXED 132629

Do the bmalloc.

https://bugs.webkit.org/show_bug.cgi?id=132629

Summary Do the bmalloc.

Andreas Kling

Reported Tuesday, May 6, 2014 11:19:26 PM UTC

Attachments
Snack for EWS (431 bytes, patch) 2014-05-06 15:21 PDT, Andreas Kling	no flags	Details Formatted Diff Diff
Snack Fighter II: World Warrior (1.92 KB, patch) 2014-05-06 15:55 PDT, Andreas Kling	no flags	Details Formatted Diff Diff
Snackcraft III: The Frozen Throne (2.63 KB, patch) 2014-05-06 17:26 PDT, Andreas Kling	no flags	Details Formatted Diff Diff
Patch (3.69 KB, patch) 2014-05-10 13:12 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1.07 KB, patch) 2014-08-14 14:44 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1002 bytes, patch) 2014-08-18 11:37 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1003 bytes, patch) 2014-08-18 11:44 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1.03 KB, patch) 2014-08-19 14:00 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (3.49 KB, patch) 2014-08-20 10:46 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1.70 KB, patch) 2014-08-20 13:19 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (4.74 KB, patch) 2014-08-20 17:55 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (991 bytes, patch) 2014-09-03 14:26 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (990 bytes, patch) 2014-09-04 13:28 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (991 bytes, patch) 2014-09-10 13:53 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1000 bytes, patch) 2014-09-10 15:45 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (1000 bytes, patch) 2014-09-11 14:08 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Patch (977 bytes, patch) 2014-09-24 11:50 PDT, Geoffrey Garen	no flags	Details Formatted Diff Diff
Show Obsolete (16) View All Add attachment proposed patch, testcase, etc.

Andreas Kling

Comment 1 Tuesday, May 6, 2014 11:21:13 PM UTC

Created attachment 230938 [details] Snack for EWS

Benjamin Poulain

Comment 2 Tuesday, May 6, 2014 11:37:20 PM UTC

Wooot! :)

Andreas Kling

Comment 3 Tuesday, May 6, 2014 11:55:42 PM UTC

Created attachment 230945 [details] Snack Fighter II: World Warrior

Andreas Kling

Comment 4 Wednesday, May 7, 2014 1:26:18 AM UTC

Created attachment 230958 [details] Snackcraft III: The Frozen Throne

Geoffrey Garen

Comment 5 Saturday, May 10, 2014 9:12:15 PM UTC

Created attachment 231231 [details] Patch

Geoffrey Garen

Comment 6 Saturday, May 10, 2014 9:13:57 PM UTC

Giving this another try, since I think the bots must be crazy.

Csaba Osztrogonác

Comment 7 Monday, May 19, 2014 4:45:21 PM UTC

(In reply to comment #6) > Giving this another try, since I think the bots must be crazy. I this the bots aren't crazy, but this patch made SVG tests crash: https://webkit-queues.appspot.com/patch/231231

Csaba Osztrogonác

Comment 8 Monday, May 19, 2014 4:46:43 PM UTC

Comment on attachment 231231 [details] Patch r- not to force EWS bots run in an infinite loop.

Geoffrey Garen

Comment 9 Thursday, August 14, 2014 10:44:18 PM UTC

Created attachment 236622 [details] Patch

Andreas Kling

Comment 10 Thursday, August 14, 2014 10:58:26 PM UTC

Comment on attachment 236622 [details] Patch If it fits, etc.

Geoffrey Garen

Comment 11 Friday, August 15, 2014 3:26:05 AM UTC

Tests are still crashing on EWS.

Pratik Solanki

Comment 12 Friday, August 15, 2014 3:31:02 AM UTC

Comment on attachment 236622 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=236622&action=review > Source/WTF/wtf/FastMalloc.cpp:104 > +#if (PLATFORM(MAC) || PLATFORM(IOS)) && (CPU(X86_64) || CPU(ARM64)) PLATFORM(COCOA) !

Geoffrey Garen

Comment 13 Monday, August 18, 2014 7:37:42 PM UTC

Created attachment 236770 [details] Patch

Geoffrey Garen

Comment 14 Monday, August 18, 2014 7:38:07 PM UTC

Not really for review. Trying to reproduce EWS failure.

Geoffrey Garen

Comment 15 Monday, August 18, 2014 7:44:08 PM UTC

Created attachment 236771 [details] Patch

Geoffrey Garen

Comment 16 Tuesday, August 19, 2014 10:00:45 PM UTC

Created attachment 236823 [details] Patch

Geoffrey Garen

Comment 17 Wednesday, August 20, 2014 6:46:05 PM UTC

Created attachment 236879 [details] Patch

Geoffrey Garen

Comment 18 Wednesday, August 20, 2014 9:19:14 PM UTC

Created attachment 236893 [details] Patch

Geoffrey Garen

Comment 19 Thursday, August 21, 2014 1:55:28 AM UTC

Created attachment 236905 [details] Patch

Geoffrey Garen

Comment 20 Wednesday, September 3, 2014 10:26:13 PM UTC

Created attachment 237582 [details] Patch

Geoffrey Garen

Comment 21 Wednesday, September 3, 2014 10:27:27 PM UTC

Perhaps, with the fix for bug 136469, this will pass EWS now.

Geoffrey Garen

Comment 22 Thursday, September 4, 2014 9:28:04 PM UTC

Created attachment 237644 [details] Patch

Michael Saboff

Comment 23 Thursday, September 4, 2014 9:32:45 PM UTC

Comment on attachment 237644 [details] Patch r=me

Geoffrey Garen

Comment 24 Friday, September 5, 2014 7:07:35 PM UTC

Committed r173313: <http://trac.webkit.org/changeset/173313>

Geoffrey Garen

Comment 25 Wednesday, September 10, 2014 9:53:50 PM UTC

Reopening to attach new patch.

Geoffrey Garen

Comment 26 Wednesday, September 10, 2014 9:53:53 PM UTC

Created attachment 237905 [details] Patch

Geoffrey Garen

Comment 27 Wednesday, September 10, 2014 11:45:57 PM UTC

Created attachment 237912 [details] Patch

Geoffrey Garen

Comment 28 Thursday, September 11, 2014 10:08:54 PM UTC

Created attachment 237987 [details] Patch

Geoffrey Garen

Comment 29 Thursday, September 11, 2014 10:45:56 PM UTC

Committed r173540: <http://trac.webkit.org/changeset/173540>

Geoffrey Garen

Comment 30 Wednesday, September 24, 2014 7:49:13 PM UTC

Rolled out in <http://trac.webkit.org/changeset/r173565>.

Geoffrey Garen

Comment 31 Wednesday, September 24, 2014 7:50:10 PM UTC

Created attachment 238603 [details] Patch

Geoffrey Garen

Comment 32 Wednesday, September 24, 2014 11:07:49 PM UTC

Committed r173935: <http://trac.webkit.org/changeset/173935>

Geoffrey Garen

Comment 33 Friday, October 24, 2014 6:00:46 PM UTC

Results from perf bots: OSMemoryTesting/Snap2FinishedLoadingPost:Allocations:Total: 0% (no regression) PLT: 5% DOM/GetElement: 42.84% jslib-attr-jquery: 18.1% dromaeo-object-string: 13.3% dom-modify: 12.3% cssquery-jquery: 5% cssquery-prototype: 4.8% cssquery-dojo: 3.8% dromaeo-object-array: 3.4% jslib-style-jquery: 3.2% DYEB: 2.4% Parser/HTML5-8266-FullRender: 4% Parser/HTML5-8266-ParseOnly: 5.6% css-parser-yui: 10.2% Parser/html-parser: 6.5% Parser/textarea-parsing: 11.22% Layout/floats_100_100: 7.22% Layout/floats_100_100_nested: 9.32% Layout/floats_20_100: 11.42% Layout/floats_20_100_nested: 13.86% Layout/layers_overlap_2d: 6.69% Layout/auto-grid-lots-of-data: 4.3% Layout/chapter-reflow-once-random: 3.8% Layout/fixed-grid-lots-of-data: 5.92% Interactive/window-resize: 2% Animation/balls:FrameRate: 4% DOM/CloneNodes: 13.93% DOM/DOMDivWalk: 10.7% DOM/GetElement: 42.84% DOM/ModifyAttribute: 8.7%

Andreas Kling

Comment 34 Friday, October 24, 2014 7:00:53 PM UTC

obamanotbad.jpg

Geoffrey Garen

Comment 35 Friday, October 24, 2014 7:25:32 PM UTC

MallocBench result vs TCMalloc: geometric mean speedup: 2.11x faster biggest speedup (list_allocate): 4.09x faster

Note You need to log in before you can comment on or make changes to this bug.

Status RESOLVED

Resolution FIXED

Priority P2

Severity Normal

Classification Unclassified

Version 528+ (Nightly build)

Hardware Unspecified

OS Unspecified

Product WebKit

Component Platform

Assignee

Andreas Kling

Reported

2014-05-06 15:19 PDT

Modified

2015-01-10 11:54 PST History

CC List

7 users Show

URL

Keywords

Depends on

Blocks