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

Issue 836469 link

Starred by 5 users

Issue metadata

Status: Duplicate
Owner:
Out until 24 Jan
Closed: Oct 12
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression

Blocking:
issue 838940



Sign in to add a comment

5.5%-316.5% regression in loading.desktop at 552589:552596

Project Member Reported by briander...@chromium.org, Apr 24 2018

Issue description

See the link to graphs below.
 
Project Member

Comment 1 by 42576172...@developer.gserviceaccount.com, Apr 24 2018

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

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


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

chromium-rel-mac11-air
chromium-rel-mac12
chromium-rel-win7-gpu-ati
chromium-rel-win7-gpu-intel
Project Member

Comment 3 by 42576172...@developer.gserviceaccount.com, Apr 25 2018

Cc: kbr@chromium.org rdevlin....@chromium.org isherman@chromium.org alex...@chromium.org lukasza@chromium.org pastarmovj@chromium.org
Owner: lukasza@chromium.org
Status: Assigned (was: Untriaged)
📍 Found a significant difference after 1 commit.
https://pinpoint-dot-chromeperf.appspot.com/job/16bce839c40000

Make --site-per-process the default on ToT via fieldtrial_testing_config by lukasza@chromium.org
https://chromium.googlesource.com/chromium/src/+/fb1ccf02ee8ca79e1404abfd3a3a7d540b7d2dbd

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Components: Internals>Sandbox>SiteIsolation
Owner: nasko@chromium.org
nasko@ has kindly offered to help investigate if this regression report offers any useful insights (e.g. via a scenario that highlights/exaggerates a perf regression so that they are visible via chrome tracing).  If this investigation doesn't yield any results, then we can just dupe against the known loading regression tracked in issue 808114.
Cc: hinoka@chromium.org tandrii@chromium.org m...@chromium.org
 Issue 838439  has been merged into this issue.
Cc: -isherman@chromium.org
Blocking: 838940

Comment 9 by kbr@chromium.org, May 2 2018

Cc: -kbr@chromium.org
Cc: -hinoka@chromium.org -tandrii@chromium.org
Cc: tdres...@chromium.org
 Issue 839370  has been merged into this issue.
Cc: hinoka@chromium.org isherman@chromium.org kbr@chromium.org tandrii@chromium.org
 Issue 836533  has been merged into this issue.

Comment 13 by kbr@chromium.org, May 10 2018

Cc: -kbr@chromium.org
This bug might be fixed by r587329 (i.e. it might be a dupe of  issue 560446 ), but I am not sure how to double-check that (e.g. if there is a way to ask performance bots for help with this verification)...
Issue 838441 has been merged into this issue.
Looking at the graph over time, it has continued staying at the higher end, though has gone up/down and currently varies quite a bit. We have fixed some regressions (priority inversion  issue 560446 ) and at this time I don't think there is anything else we are planning to investigate. I think resolving as duplicate of issue 808114 makes sense at this time.

To answer comment 14, I don't see that revision making any difference on the performance graph for the original graph cited in this bug.
Mergedinto: 808114
Status: Duplicate (was: Assigned)
Resolving as duplicate to 808114. If someone thinks this is incorrect, feel free to reopen.
Cc: primiano@google.com
 Issue 841244  has been merged into this issue.

Sign in to add a comment