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

Issue 747937 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

2.1% regression in system_health.memory_mobile at 487901:488028

Project Member Reported by hjd@google.com, Jul 24 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jul 24 2017

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

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


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

android-one
Project Member

Comment 3 by sheriffbot@chromium.org, Jul 24 2017

Labels: Hotlist-Google
Project Member

Comment 4 by 42576172...@developer.gserviceaccount.com, Jul 25 2017

Cc: matthalp@google.com
Owner: matthalp@google.com

=== Auto-CCing suspected CL author matthalp@google.com ===

Hi matthalp@google.com, 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 : Matthew Halpern
  Commit : cd7e31fb911a98c7dc021d543e9dea8c553cdb1f
  Date   : Wed Jul 19 20:50:43 2017
  Subject: Enable GRC

Bisect Details
  Configuration: android_one_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:chrome:all_processes:reported_by_chrome:malloc:effective_size_avg/load_news/load_news_nytimes
  Change       : 0.46% | 15083548.9286 -> 15153059.1111

Revision             Result                  N
chromium@487900      15083549 +- 249875      14      good
chromium@487964      15046495 +- 396052      14      good
chromium@487966      15070877 +- 337465      14      good
chromium@487967      15049545 +- 288474      14      good
chromium@487968      15113568 +- 459136      14      bad       <--
chromium@487972      15156459 +- 231039      14      bad
chromium@487980      15134234 +- 387436      21      bad
chromium@487996      15166520 +- 132376      14      bad
chromium@488028      15153059 +- 194383      9       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-chromium --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/8973189278212852448


For feedback, file a bug with component Speed>Bisection

Comment 5 by hjd@chromium.org, Jul 25 2017

Status: Assigned (was: Untriaged)
I'm going to put the status to assigned, please assign back to me and put the status to untriaged if you don't think this has ended up on the right CL, thanks!
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Jul 25 2017

 Issue 747938  has been merged into this issue.
Project Member

Comment 7 by 42576172...@developer.gserviceaccount.com, Jul 27 2017

Cc: m...@chromium.org
 Issue 749356  has been merged into this issue.
Owner: oysteine@chromium.org
oysteine: any update here?
Status: WontFix (was: Assigned)
Closing since all these regressions are less than 1MiB and the bug is quite old at this point.

Sign in to add a comment