Bug 111200 – Add html-parser-threaded perf test now that the threaded parser is enabled by default

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Eric Seidel (no email)]

Add html-parser-threaded perf test now that the threaded parser is enabled by default

[Eric Seidel (no email)]

Created attachment 190996 [details]
Patch

[Ryosuke Niwa]

Comment on attachment 190996 [details]
Patch

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

> PerformanceTests/ChangeLog:9
> +

Are values stable? Could you list sample outputs of this test?

[Eric Seidel (no email)]

Good question.  The test only seems accurate within .5 sec. :(

I assume I should mark it as skipped for now?

Running 1 tests
Running Parser/html-parser-threaded.html (1 of 1)
RESULT Parser: html-parser-threaded= 6481.615 ms
median= 6462.267 ms, stdev= 492.201657996 ms, min= 5770.78700002 ms, max= 7697.552 ms
RESULT Parser: html-parser-threaded: JSHeap= 3162584.4 bytes
median= 3150244.0 bytes, stdev= 1293464.27469 bytes, min= 1129652.0 bytes, max= 5679636.0 bytes
RESULT Parser: html-parser-threaded: Malloc= 211043848.8 bytes
median= 209918588.0 bytes, stdev= 35644152.6867 bytes, min= 154237584.0 bytes, max= 278692600.0 bytes
Finished: 138.979004 s

Erics-MacBook-Pro [5567:PerformanceTests]% run-perf-tests Parser/html-parser-threaded.html --chromium                                                        [~/Projects/WebKit/PerformanceTests]
    wdiff is not installed; please install from MacPorts or elsewhere

Running 1 tests
Running Parser/html-parser-threaded.html (1 of 1)
RESULT Parser: html-parser-threaded= 6434.6968 ms
median= 6390.426 ms, stdev= 466.679802207 ms, min= 5737.43000001 ms, max= 7561.18000002 ms
RESULT Parser: html-parser-threaded: JSHeap= 3097660.4 bytes
median= 3085456.0 bytes, stdev= 1282908.77539 bytes, min= 1129652.0 bytes, max= 5548680.0 bytes
RESULT Parser: html-parser-threaded: Malloc= 209789260.0 bytes
median= 207639304.0 bytes, stdev= 36461134.7981 bytes, min= 154657400.0 bytes, max= 278346448.0 bytes
Finished: 137.554094 s

[Eric Seidel (no email)]

I wonder if something is leaking.  The memory numbers seem rather consistent.

[Ryosuke Niwa]

(In reply to comment #4)
> I wonder if something is leaking.  The memory numbers seem rather consistent.

If the memory is leaking, results page should show that. If you click on a row, you should see the tiny graph showing up at the bottom that shows the time series. Make sure to choose "Memory" tab at the left upper corner first.

[Eric Seidel (no email)]

(In reply to comment #5)
> (In reply to comment #4)
> > I wonder if something is leaking.  The memory numbers seem rather consistent.
> 
> If the memory is leaking, results page should show that. If you click on a row, you should see the tiny graph showing up at the bottom that shows the time series. Make sure to choose "Memory" tab at the left upper corner first.

The results page hasn't been workign for me.

Uncaught TypeError: Cannot read property 'WebKit' of undefined

[Eric Seidel (no email)]

I think this is part of the trouble: bug 107236.

[Eric Seidel (no email)]

Created attachment 191433 [details]
Patch

[Eric Seidel (no email)]

This adds it back to the skipped list so we can still use it, even though it's off for the bots.

[WebKit Review Bot]

Comment on attachment 191433 [details]
Patch

Clearing flags on attachment: 191433

Committed r144785: <http://trac.webkit.org/changeset/144785>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.