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

Issue 762940 link

Starred by 5 users

Issue metadata

Status: WontFix
Owner:
Closed: Jan 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

4.9%-6559.1% regression in thread_times.key_silk_cases at 499617:499898

Project Member Reported by tdres...@chromium.org, Sep 7 2017

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=762940

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


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

android-nexus5X
android-nexus6
android-nexus7v2
android-webview-nexus6
Cc: sunn...@chromium.org
Owner: sunn...@chromium.org
Status: Assigned (was: Untriaged)

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

Hi sunnyps@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 : Sunny Sachanandani
  Commit : 2eb3c8ac7ffc6840dc07f5e8312576e107eece23
  Date   : Wed Sep 06 02:01:21 2017
  Subject: cc: Flush after every tile task set completes.

Bisect Details
  Configuration: android_webview_nexus6_aosp_perf_bisect
  Benchmark    : thread_times.key_silk_cases
  Metric       : thread_GPU_cpu_time_per_frame/http___jsbin.com_UVIgUTa_38_quiet
  Change       : 5974.27% | 0.00624333619353 -> 0.37923733827

Revision             Result                         N
chromium@499656      0.00624334 +- 0.00091715       6      good
chromium@499763      0.00642974 +- 0.000672611      6      good
chromium@499816      0.00624219 +- 0.00108806       6      good
chromium@499843      0.00600326 +- 0.00118731       6      good
chromium@499844      0.00594695 +- 0.00106342       6      good
chromium@499845      0.355537 +- 0.0456925          6      bad       <--
chromium@499847      0.365251 +- 0.0165041          6      bad
chromium@499850      0.369785 +- 0.0165255          6      bad
chromium@499856      0.368071 +- 0.0343779          6      bad
chromium@499869      0.379237 +- 0.0367379          6      bad

To Run This Test
  src/tools/perf/run_benchmark -v --browser=android-webview --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=http...jsbin.com.UVIgUTa.38.quiet thread_times.key_silk_cases

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

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


For feedback, file a bug with component Speed>Bisection
 Issue 763085  has been merged into this issue.
I looked at a few graphs, not a real regression. Just back to previous levels after  issue 760886  was fixed.

I'll look at a few more graphs and close this as WontFix.
Cc: petermarshall@chromium.org
 Issue 763277  has been merged into this issue.
 Issue 763275  has been merged into this issue.
 Issue 763274  has been merged into this issue.
 Issue 763276  has been merged into this issue.
Cc: rmcilroy@chromium.org xidac...@chromium.org
 Issue 763087  has been merged into this issue.
 Issue 763087  has been merged into this issue.
Project Member

Comment 12 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

Cc: jarin@google.com jarin@chromium.org
 Issue 763763  has been merged into this issue.
Project Member

Comment 13 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763763  has been merged into this issue.
Project Member

Comment 14 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763862  has been merged into this issue.
Project Member

Comment 15 by 42576172...@developer.gserviceaccount.com, Sep 11 2017

 Issue 763878  has been merged into this issue.
Project Member

Comment 16 by 42576172...@developer.gserviceaccount.com, Sep 12 2017

 Issue 763890  has been merged into this issue.
Project Member

Comment 17 by 42576172...@developer.gserviceaccount.com, Sep 28 2017

Cc: hablich@chromium.org marja@chromium.org
 Issue 769693  has been merged into this issue.
 Issue 769693  has been merged into this issue.
Should this be WontFixed?
Status: WontFix (was: Assigned)
WontFixing per #5 and #19.

Sign in to add a comment