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

Issue 689595 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

20.1%-30.1% regression in v8.mobile_infinite_scroll_tbmv2 at 447524:448123

Project Member Reported by lanwei@chromium.org, Feb 7 2017

Issue description

See the link to graphs below.
 
Cc: alexclarke@chromium.org
Owner: alexclarke@chromium.org

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

Hi alexclarke@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 : alexclarke
  Commit : 205b4a6222828e70127989c9451bbbd5e56be02d
  Date   : Fri Feb 03 19:30:16 2017
  Subject: Revert of Dont post delayed DoWork for disabled queues. (patchset #9 id:160001 of https://codereview.chromium.org/2572893002/ )

Bisect Details
  Configuration: android_nexus5X_perf_bisect
  Benchmark    : v8.mobile_infinite_scroll_tbmv2
  Metric       : memory:chrome:all_processes:reported_by_chrome:v8:allocated_objects_size_max/memory:chrome:all_processes:reported_by_chrome:v8:allocated_objects_size_max
  Change       : 28.77% | 43634654.1143 -> 56189980.2667

Revision             Result                   N
chromium@448045      43634654 +- 3798022      7      good
chromium@448046      56145311 +- 6217263      6      bad       <--
chromium@448047      56580206 +- 4937283      6      bad
chromium@448048      57354839 +- 4432759      6      bad
chromium@448050      55484423 +- 6104401      6      bad
chromium@448055      56526807 +- 5114580      6      bad
chromium@448065      56147705 +- 9154033      6      bad
chromium@448084      55013971 +- 6482694      6      bad
chromium@448123      56189980 +- 3297072      6      bad

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 v8.mobile_infinite_scroll_tbmv2

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

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


| 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!
Status: WontFix (was: Untriaged)
Well that's pretty strange, it's probably a bug that caused the memory to go down with that patch.

I do intend to try and re-land but I don't expect memory to go down!


Sign in to add a comment