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

Issue 674527 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

1.8%-2.9% regression in system_health.memory_mobile at 438434:438474

Project Member Reported by pmeenan@chromium.org, Dec 15 2016

Issue description

See the link to graphs below.
 
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Dec 15 2016

Cc: hajimehoshi@chromium.org
Owner: hajimehoshi@chromium.org

=== PERF REGRESSION ===


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

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


===== BISECT JOB RESULTS =====
Status: completed


===== SUSPECTED CL(s) =====
Subject : Add a macro for SharedMemory::Share
Author  : hajimehoshi
Commit description:
  
SharedMemory::Share was introduced at crrev.com/2555483002 only on Mac
and not other platforms. This CL adds a macro to SharedMemory::Share
declaration to clarily that the function Share is available only on Mac.

BUG= 671228 
TEST=n/a

Review-Url: https://codereview.chromium.org/2567293003
Cr-Commit-Position: refs/heads/master@{#438434}
Commit  : 73172f9e94e5af7e23867506c9e5a9d6bc99c703
Date    : Wed Dec 14 04:59:46 2016


===== TESTED REVISIONS =====
Revision         Mean      Std Dev  N  Good?
chromium@438433  9150293   270151   6  good
chromium@438434  11827541  167202   6  bad    <--
chromium@438435  11540139  298891   6  bad
chromium@438436  11145557  351608   6  bad
chromium@438439  10660864  395797   6  bad
chromium@438444  10152619  425656   6  bad
chromium@438454  10124629  461129   6  bad
chromium@438474  9863509   264746   6  bad

Bisect job ran on: android_nexus5X_perf_bisect
Bug ID: 674527

Test Command: 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.washingtonpost system_health.memory_mobile
Test Metric: memory:chrome:all_processes:reported_by_os:system_memory:java_heap:proportional_resident_size_avg/load_news/load_news_washingtonpost
Relative Change: 7.79%

Buildbot stdio: http://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5X_perf_bisect/builds/993
Job details: https://chromeperf.appspot.com/buildbucket_job_status/8993203276492813088


Not what you expected? We'll investigate and get back to you!
  https://chromeperf.appspot.com/bad_bisect?try_job_id=5780354386886656

| 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 Tests>AutoBisect.  Thank you!
Owner: pmeenan@chromium.org
My CL just added a macro. I don't think it's the culprit.
Status: WontFix (was: Untriaged)
Closing.  The graphs have recovered.

Sign in to add a comment