New issue
Advanced search Search tips

Issue 828019 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

8.6% regression in system_health.common_mobile at 546463:546523

Project Member Reported by pmeenan@chromium.org, Apr 2 2018

Issue description

See the link to graphs below.
 
All graphs for this bug:
  https://chromeperf.appspot.com/group_report?bug_id=828019

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


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

android-nexus5
Cc: primiano@chromium.org dpranke@chromium.org mlamouri@chromium.org fdoray@chromium.org agrieve@chromium.org dcheng@chromium.org ssid@chromium.org
Owner: ssid@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/14a3646f440000

Revert "Android: Add code to parse the CFI info added to the apk" by fdoray@chromium.org
https://chromium.googlesource.com/chromium/src/+/88328b6800f987ba9f532842862021cb48875618

Revert "Enable extracting unwind table on official builds without channel" by agrieve@chromium.org
https://chromium.googlesource.com/chromium/src/+/a2474b6490ea9cfda76528059fbf23369e7d281f

Do not change media notification icon multiple times by ssid@chromium.org
https://chromium.googlesource.com/chromium/src/+/a91837ee530b963d208e9f08f5a657e649a5323e

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

Comment 4 by ssid@chromium.org, Apr 24 2018

Cc: -agrieve@chromium.org -dpranke@chromium.org -mlamouri@chromium.org -dcheng@chromium.org -fdoray@chromium.org -primiano@chromium.org

Comment 5 by ssid@chromium.org, Apr 24 2018

Cc: beccahughes@chromium.org
There seems to be an exact same amount of drop in graphs, but unclear what is the cause.
There are media controls CLs like:
https://chromium-review.googlesource.com/990192
https://chromium-review.googlesource.com/991075
https://chromium-review.googlesource.com/982900

But, I am not sure if this fixes the regression or these are unrelated improvements and the regression caused by my cl still exists. Also I was expecting my cl to improve cpu time. Can you help with what the metric is measuring?
None of those media controls CLs should have had a negative performance impact.

Doesn't pinpoint apply the CLs and run tests before and after to see which CL has the performance differences. Maybe someone from the performance team can help here with the tools?
Project Member

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

Cc: primiano@chromium.org dpranke@chromium.org mlamouri@chromium.org fdoray@chromium.org agrieve@chromium.org dcheng@chromium.org
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/14adc039c40000

Revert "Android: Add code to parse the CFI info added to the apk" by fdoray@chromium.org
https://chromium.googlesource.com/chromium/src/+/88328b6800f987ba9f532842862021cb48875618

Revert "Enable extracting unwind table on official builds without channel" by agrieve@chromium.org
https://chromium.googlesource.com/chromium/src/+/a2474b6490ea9cfda76528059fbf23369e7d281f

Do not change media notification icon multiple times by ssid@chromium.org
https://chromium.googlesource.com/chromium/src/+/a91837ee530b963d208e9f08f5a657e649a5323e

Understanding performance regressions:
  http://g.co/ChromePerformanceRegressions
Components: Speed>Metrics>SystemHealthRegressions
Cc: -pmeenan@chromium.org
Components: -Speed>Metrics>SystemHealthRegressions
Components: Speed>Metrics>SystemHealthRegressions
Ping! This bug has been flagged as stale by Chrome Speed Metrics bug triage, and it has an owner. Any update on this issue?

Sign in to add a comment