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

Issue 855433 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 10
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

55.7% regression in scheduler.tough_scheduling_cases at 562467:568711

Project Member Reported by alexclarke@chromium.org, Jun 22 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Jun 22 2018

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

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


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

android-one
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Jun 23 2018

Cc: amyqiu@google.com
Owner: amyqiu@google.com
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/16905973240000

Rename pages in tough_scheduling_cases by amyqiu@google.com
https://chromium.googlesource.com/chromium/src/+/be71236d2c257e74fb599c1ec96fd7e8b116e2c9
No values → 43.75

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions

Comment 4 by amyqiu@google.com, Jun 25 2018

Cc: nednguyen@chromium.org sadrul@chromium.org
Components: Speed>Benchmarks
Hmm, I can't figure out how this CL could have caused the regression, since it only renamed the page. Looking at the Pinpoint results, it appears that the test wasn't generating values for the commits up to this CL, could that be related?

@nednguyen, do you have any ideas?
Cc: dtu@chromium.org
Note that the regression says "No values → 43.75". So I think you may have actually fixed some issue with the benchmark. 

Dave: what is the way to move forward here?
amyqiu's change changed the story name. Since Pinpoint uses the story name to look up the results, it couldn't find the results before her change. I re-ran the job with the old story name on the range before her change.
Cc: fsam...@chromium.org
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/15e1e850a40000

Surface Sychronization: Clean-up DelegatedFrameHostAndroid by fsamuel@chromium.org
https://chromium.googlesource.com/chromium/src/+/dba483cee6a5f15e2e2d73df16968ab10b38a2bf
27.45 → 44.12 (+16.67)

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Owner: fsam...@chromium.org
Status: Fixed (was: Assigned)
I believe sadrul@ has fixed this in M69.

Sign in to add a comment