New issue
Advanced search Search tips

Issue 737549 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

A zero-to-nonzero to 1.2% regression in system_health.memory_mobile at 482284:482444

Project Member Reported by tdres...@chromium.org, Jun 28 2017

Issue description

See the link to graphs below.
 
Project Member

Comment 4 by 42576172...@developer.gserviceaccount.com, Jun 28 2017

Cc: ssid@chromium.org
Owner: ssid@chromium.org

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

Hi ssid@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 : Siddhartha
  Commit : 12fd478c980c2af7ea532d7470a350acc19c3a9d
  Date   : Mon Jun 26 20:23:24 2017
  Subject: Clear GL context when Chrome goes to background on low end devices

Bisect Details
  Configuration: android_one_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:chrome:all_processes:reported_by_chrome:gpu:effective_size_avg/background_media/background_media_imgur

Revision             Result              N
chromium@482283      0.0 +- 0.0          6      good
chromium@482354      0.0 +- 0.0          6      good
chromium@482372      0.0 +- 0.0          6      good
chromium@482375      0.0 +- 0.0          6      good
chromium@482376      0.0 +- 0.0          6      good
chromium@482377      2228224 +- 0.0      6      bad       <--
chromium@482381      2228224 +- 0.0      6      bad
chromium@482389      2228224 +- 0.0      6      bad
chromium@482424      2228224 +- 0.0      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-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=background.media.imgur system_health.memory_mobile

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8975542244303510496

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=6128681266708480


| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Speed>Bisection.  Thank you!
Project Member

Comment 5 by 42576172...@developer.gserviceaccount.com, Jun 28 2017

Issue 737329 has been merged into this issue.
Project Member

Comment 6 by 42576172...@developer.gserviceaccount.com, Jun 28 2017


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

Suspected Commit
  Author : Siddhartha
  Commit : 12fd478c980c2af7ea532d7470a350acc19c3a9d
  Date   : Mon Jun 26 20:23:24 2017
  Subject: Clear GL context when Chrome goes to background on low end devices

Bisect Details
  Configuration: android_one_perf_bisect
  Benchmark    : system_health.memory_mobile
  Metric       : memory:chrome:all_processes:reported_by_chrome:gpu:effective_size_avg/background_media/background_media_imgur

Revision             Result              N
chromium@482283      0.0 +- 0.0          6      good
chromium@482354      0.0 +- 0.0          6      good
chromium@482372      0.0 +- 0.0          6      good
chromium@482375      0.0 +- 0.0          6      good
chromium@482376      0.0 +- 0.0          6      good
chromium@482377      2228224 +- 0.0      6      bad       <--
chromium@482381      2228224 +- 0.0      6      bad
chromium@482389      2228224 +- 0.0      6      bad
chromium@482424      2228224 +- 0.0      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-chromium --output-format=chartjson --upload-results --pageset-repeat=1 --also-run-disabled-tests --story-filter=background.media.imgur system_health.memory_mobile

Debug Info
  https://chromeperf.appspot.com/buildbucket_job_status/8975541946854285440

Is this bisect wrong?
  https://chromeperf.appspot.com/bad_bisect?try_job_id=6052811072077824


| O O | Visit http://www.chromium.org/developers/speed-infra/perf-bug-faq
|  X  | for more information addressing perf regression bugs. For feedback,
| / \ | file a bug with component Speed>Bisection.  Thank you!
Status: Assigned (was: Untriaged)
Explictly assigning. A CL you landed tripped one of the speed metrics we measure in the lab. If this is the first time this has happened to one of your CLs, or if it's been a while, please read: https://chromium.googlesource.com/chromium/src/+/master/docs/speed/addressing_performance_regressions.md

We're looking for one of the following:
1. Justification via explanation
2. Plan to revert or fix
3. Angry rage throwing of equipment at my head

Just be aware that I'm trained in trumpet playing and First Aid and am not afraid to use it.

Note: This was a bulk edit message and not very personal.
ssid: is this expected?
Status: WontFix (was: Assigned)
No response on the bug, not much we can do at this point.

Sign in to add a comment