Closed Bug 588663 Opened 14 years ago Closed 14 years ago

investigate talos impact of 130078

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: tnikkel, Unassigned)

References

Details

Using http://perf.snarkfest.net/compare-talos/ we can compare try talos to some m-c talos results. Use these changesets. original rev 48487f904abe a9b21fa2b7ab 3a8430b73a91 8fb3151c8db4 new revision f9eb580dd56b new revision 05a791444e43 origina rev a75699fa48e3 3d4ba021c4dd 55953a91b4d6 f001894b50ea 9480e51be5b7 Some wins, some loses, some things look broken.
Another copy of that tool that might work better http://services.forerunnerdesigns.com/compare-talos/index.php
Links to the results so you don't have to re-enter the info http://services.forerunnerdesigns.com/compare-talos/index.php?oldRevs=48487f904abe,a9b21fa2b7ab,3a8430b73a91,8fb3151c8db4,&newRev=f9eb580dd56b&tests=a11y,tdhtml,tdhtml_nochrome,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tgfx,tgfx_nochrome,tscroll,tsvg,tsvg_opacity,ts,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen&submit=true http://services.forerunnerdesigns.com/compare-talos/index.php?oldRevs=a75699fa48e3,3d4ba021c4dd,55953a91b4d6,f001894b50ea,9480e51be5b7&newRev=05a791444e43&tests=a11y,tdhtml,tdhtml_nochrome,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tgfx,tgfx_nochrome,tscroll,tsvg,tsvg_opacity,ts,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen&submit=true
Depends on: 589396
New numbers: new rev f46b618cc0ab orig 01d7e7401b33 99367f10f79e 87677ea8efa3 6d8a749a3c57 c6da8fefd6a8 http://services.forerunnerdesigns.com/compare-talos/index.php?oldRevs=01d7e7401b33,99367f10f79e,87677ea8efa3,6d8a749a3c57,c6da8fefd6a8&newRev=f46b618cc0ab&tests=a11y,tdhtml,tdhtml_nochrome,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tgfx,tgfx_nochrome,tscroll,tsvg,tsvg_opacity,ts,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen&submit=true new rev cf64eea45e07 orig 160a97bac310 98ad1cc8627e dc804317e500 810ff22372ac 3e4fc3864378 http://services.forerunnerdesigns.com/compare-talos/index.php?oldRevs=160a97bac310,98ad1cc8627e,dc804317e500,810ff22372ac,3e4fc3864378&newRev=cf64eea45e07&tests=a11y,tdhtml,tdhtml_nochrome,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tgfx,tgfx_nochrome,tscroll,tsvg,tsvg_opacity,ts,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen&submit=true
I guess Tp4 still needs investigation. Tp4_xres on Linux-32 too. Maybe the latter one would be easier to figure out and give us wins on other platforms if we can fix it.
Tp4_xres on linux 32 is in bytes, so it goes from ~700 bytes to ~2000 bytes. For comparison Tp4_xres on linux 64 is ~13 MB. I compared a build with and without 130078 (not based on the same revision, but fairly close) on linux 64 and the build with 130078 was 1MB smaller. That was only one run, so I don't know if that's enough to see whats going on. Tp4 on mac (32bit) and winxp are the ones that seem to show a regression. All other platforms seem to show an improvement or within the normal bounds.
Discussed linux Tp4_xres numbers on irc with roc and Karl. We concluded there is no way the 32 bit numbers can represent anything meaningful.
new rev 44cbdbf82ad9 orig revs f1b1e827d401 18bb00bd1564 39755990a9ae 13adb6fe5ca0 d207cb43a7bf http://services.forerunnerdesigns.com/compare-talos/index.html?oldRevs=f1b1e827d401,18bb00bd1564,39755990a9ae,13adb6fe5ca0,d207cb43a7bf&newRev=44cbdbf82ad9&tests=a11y,tdhtml,tdhtml_nochrome,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tgfx,tgfx_nochrome,tscroll,tsvg,tsvg_opacity,ts,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen&submit=true new rev 03a19452a127 orig revs f1b1e827d401 18bb00bd1564 39755990a9ae 13adb6fe5ca0 d207cb43a7bf http://services.forerunnerdesigns.com/compare-talos/index.html?oldRevs=f1b1e827d401,18bb00bd1564,39755990a9ae,13adb6fe5ca0,d207cb43a7bf&newRev=03a19452a127&tests=a11y,tdhtml,tdhtml_nochrome,tp4,tp4_memset,tp4_pbytes,tp4_rss,tp4_shutdown,tp4_xres,dromaeo_basics,dromaeo_css,dromaeo_dom,dromaeo_jslib,dromaeo_sunspider,dromaeo_v8,tsspider,tsspider_nochrome,tgfx,tgfx_nochrome,tscroll,tsvg,tsvg_opacity,ts,ts_cold,ts_cold_generated_max,ts_cold_generated_max_shutdown,ts_cold_generated_med,ts_cold_generated_med_shutdown,ts_cold_generated_min,ts_cold_generated_min_shutdown,ts_cold_shutdown,ts_places_generated_max,ts_places_generated_max_shutdown,ts_places_generated_med,ts_places_generated_med_shutdown,ts_places_generated_min,ts_places_generated_min_shutdown,ts_shutdown,twinopen&submit=true
As discussed on IRC, paint suppression is likely to be an issue here for the Tp4 numbers.
The automated talos scripts flagged the Tp4 Xres increase and the mac Tp4 increase.
Bug 591435 landed and turned all Tp4 results into mostly wins, and a few no change from per 130078. Calling this resolved.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.