Closed Bug 967146 Opened 6 years ago Closed 6 years ago

talos xperf regression: Main Startup File IO Bytes, regressed Jan 21st and Feb 1st


(Core :: General, defect)

Windows 7
Not set





(Reporter: jmaher, Assigned: ehsan)



(Keywords: perf, regression, Whiteboard: [talos_regression])

Taking a look a the graphs on graph server, I see some interesting changes in the xperf data:[[242,63,25]]&sel=none&displayrange=30&datatype=running

In general this data is pretty stable, so a change is noticed.
I strongly suspect the large increase on the 21st is this patch:
the second jump appears to be when we switched tofrom icu*50.dll to icu*52.dll:
aklotz, can you confirm this is expected so we can resolve this bug as wontfix?
This is expected in the sense that we know where it came from and why. OTOH, I think that we might want to talk to some people about this functionality and find out whether we can defer its initialization or something.
Flags: needinfo?(ehsan)
Component: Talos → General
Product: Testing → Core
Vladan, I'm not sure what kind of information you're asking from me.  Please be more specific!  :-)
Flags: needinfo?(ehsan)
Depends on: 970123
With bug 970123 fixed, can you please verify that the regression has gone away?  Thanks!
Flags: needinfo?(jmaher)
thanks for fixing bug 970123, this has resorted back to almost the original level.  There was another slight bump while this bug was open, so 2 regressions and we are back down to about 1/2 of the second bump.
Closed: 6 years ago
Flags: needinfo?(jmaher)
Resolution: --- → FIXED
Assignee: nobody → ehsan
Target Milestone: --- → mozilla30
Duplicate of bug: 970123

Improvement: Mozilla-Inbound - Tp5 No Network Row Major MozAfterPaint (Main Startup File IO Bytes) - WINNT 6.1 (ix) - 20.6% decrease
    Previous: avg 51691558.333 stddev 7263.790 of 12 runs up to revision dfa1bf6d4c60
    New     : avg 41023258.333 stddev 222637.877 of 12 runs since revision 0050e64d10c8
    Change  : -10668300.000 (20.6% / z=1468.696)
    Graph   :

Changeset range:

    : L. David Baron <> - Bug 434102 followup:  Mark reftest as failing on Mac.

    : L. David Baron <> - Update test failure annotation for bug 791480, now that we're running tests on Windows NT 6.2 as well.

    : Ehsan Akhgari <> - Bug 973389 - Move the LOCAL_INCLUDES in mozglue to; r=glandium

    : Ehsan Akhgari <> - Bug 973391 - Move the LOCAL_INCLUDES in webapprt to; r=glandium

    : Ehsan Akhgari <> - Bug 973403 - Move the LOCAL_INCLUDES in memory to; r=glandium

    : Ehsan Akhgari <> - Bug 973402 - Move the LOCAL_INCLUDES in js/src to; r=glandium

    : Ehsan Akhgari <> - Bug 973646 - Port some CXXFLAGS logic in js/src/ to the DEFINES variable; r=glandium

    : Ehsan Akhgari <> - Bug 973395 - Move the LOCAL_INCLUDES in media to; r=glandium

    : Ehsan Akhgari <> - Bug 970123 - Avoid read-aheading icudt52.dll for better startup performance; r=glandium
You need to log in before you can comment on or make changes to this bug.