New issue
Advanced search Search tips

Issue 764619 link

Starred by 2 users

Issue metadata

Status: WontFix
Merged: issue 763407
Owner:
Closed: Oct 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocking:
issue v8:6796



Sign in to add a comment

Improvements and regressions for lazy deserialization

Project Member Reported by jgruber@chromium.org, Sep 13 2017

Issue description

Project Member

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

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

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


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

android-nexus5X
android-nexus7v2
android-webview-nexus5X
android-webview-nexus6
Blocking: v8:6796
Project Member

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

Mergedinto: 763407
Status: Duplicate (was: Assigned)

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

Suspected Commit
  Author : xunjieli
  Commit : 724bb776bcdea642a5bc9cc20fe14bb0282b4900
  Date   : Wed Sep 06 14:29:20 2017
  Subject: [Telemetry] Use --ignore-certificate-errors-spki-list to bypass cert errors

Bisect Details
  Configuration: android_webview_arm64_aosp_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:webview:all_processes:reported_by_chrome:v8:effective_size_avg/load_news/load_news_nytimes
  Change       : 2.12% | 19298525.3333 -> 18889674.6667

Revision                                 Result                  N
chromium@500322                          19298525 +- 198936      6      good
chromium@500334                          19258724 +- 153363      6      good
chromium@500340                          19335237 +- 218997      6      good
chromium@500343                          19337211 +- 304374      6      good
chromium@500344                          19327364 +- 190800      6      good
chromium@500344,catapult@c5a474f50f      19313296 +- 103965      6      good
chromium@500344,catapult@724bb776bc      9730265 +- 60302.2      6      bad       <--
chromium@500344,catapult@80c47581b1      9691600 +- 100374       6      bad
chromium@500344,catapult@4f611510b9      9709179 +- 202347       6      bad
chromium@500344,catapult@442f46edf7      9683029 +- 103602       6      bad
chromium@500345                          9695325 +- 89499.7      6      bad
chromium@500346                          9692416 +- 74527.4      6      bad
chromium@500370                          9730988 +- 241635       6      bad
chromium@500418                          9685801 +- 104178       6      bad
chromium@500514                          9653784 +- 100349       6      bad
chromium@500706                          9551312 +- 67886.5      6      bad
chromium@501089                          18889675 +- 195946      6      bad

Please refer to the following doc on diagnosing memory regressions:
  https://chromium.googlesource.com/chromium/src/+/master/docs/memory-infra/memory_benchmarks.md

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=load.news.nytimes system_health.memory_mobile

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

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


For feedback, file a bug with component Speed>Bisection
Status: Assigned (was: Duplicate)
Cc: jgruber@chromium.org
 Issue 765552  has been merged into this issue.
Cc: wolenetz@chromium.org crouleau@chromium.org dtu@chromium.org
 Issue 765765  has been merged into this issue.
Status: WontFix (was: Assigned)
Closing now that the experiment has concluded.

Sign in to add a comment