New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 787771 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
OOO until 2019-02-10
Closed: Jan 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

31.3% regression in system_health.common_mobile at 517810:517837

Project Member Reported by hjd@google.com, Nov 22 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Nov 22 2017

All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=787771

(For debugging:) Original alerts at time of bug-filing:
  https://chromeperf.appspot.com/group_report?sid=7802e70592dcbf3692da297280dcf99d7ad44b7fd5587c70c1ac939cd8f5a345


Bot(s) for this bug's original alert(s):

android-nexus7v2
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Nov 22 2017

Cc: petermarshall@chromium.org
Owner: petermarshall@chromium.org
Status: Assigned (was: Untriaged)

=== Auto-CCing suspected CL author petermarshall@chromium.org ===

Hi petermarshall@chromium.org, the bisect results pointed to your CL, please take a look at the
results.


=== BISECT JOB RESULTS ===
Perf regression found with culprit

Suspected Commit
  Author : Peter Marshall
  Commit : 365bb5bb4a1dd62908baf0fad9d075fbcab695ed
  Date   : Mon Nov 20 09:50:01 2017
  Subject: [regexp] Add a CSA fast path for replace with global regexp.

Bisect Details
  Configuration: android_nexus7_perf_bisect
  Benchmark    : system_health.common_mobile
  Metric       : total:500ms_window:renderer_eqt_max/load_tools/load_tools_drive
  Change       : 29.21% | 931.554333333 -> 1203.70483333

Revision                           Result                  N
chromium@517809                    931.554 +- 180.227      6      good
chromium@517823                    880.544 +- 85.7612      6      good
chromium@517827                    890.905 +- 60.9877      6      good
chromium@517829                    894.399 +- 118.197      6      good
chromium@517829,v8@a9a1671345      913.045 +- 120.434      6      good
chromium@517829,v8@365bb5bb4a      1132.92 +- 231.031      9      bad       <--
chromium@517829,v8@f24a7e758a      1149.99 +- 158.345      6      bad
chromium@517830                    1162.93 +- 99.7496      6      bad
chromium@517837                    1203.7 +- 273.854       6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=load.tools.drive system_health.common_mobile

More information on addressing performance regressions:
  http://g.co/ChromePerformanceRegressions

Debug information about this bisect:
  https://chromeperf.appspot.com/buildbucket_job_status/8962230119740280784


For feedback, file a bug with component Speed>Bisection
Project Member

Comment 4 by 42576172...@developer.gserviceaccount.com, Nov 22 2017

 Issue 787775  has been merged into this issue.
Status: WontFix (was: Assigned)
See https://bugs.chromium.org/p/chromium/issues/detail?id=787801

Sign in to add a comment