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

Issue 907109 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

1.5%-26.1% regression in system_health.memory_mobile at 608488:608620

Project Member Reported by mustaq@chromium.org, Nov 20

Issue description

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

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


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

Android Nexus5 Perf
android-nexus5x-perf

system_health.memory_mobile - Benchmark documentation link:
  https://bit.ly/system-health-benchmarks
Cc: oysteine@chromium.org cjgrant@chromium.org danyao@chromium.org
Owner: danyao@chromium.org
Status: Assigned (was: Untriaged)
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/11ebe5ebe40000

Introduce bundle-specific native libraries by cjgrant@chromium.org
https://chromium.googlesource.com/chromium/src/+/50ab16b8b9eff6611a5b6ec5eade28c54aa7726c
memory:chrome:all_processes:reported_by_os:system_memory:proportional_resident_size: 1.997e+08 → No values

Revert "Introduce bundle-specific native libraries" by oysteine@chromium.org
https://chromium.googlesource.com/chromium/src/+/597c6ab3c70e51e91e2b0114c12d62ad062a0b8d
memory:chrome:all_processes:reported_by_os:system_memory:proportional_resident_size: No values → 2.308e+08

[Nav Experiment] Temporarily disable broken ReadingListTestCase test. by danyao@chromium.org
https://chromium.googlesource.com/chromium/src/+/42cbbce1449e218024d320135f59470a05ca9373
memory:chrome:all_processes:reported_by_os:system_memory:proportional_resident_size: 2.308e+08 → 2.289e+08 (-1.853e+06)

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

Benchmark documentation link:
  https://bit.ly/system-health-benchmarks
Owner: ----
My change disables a test in iOS so can't be the cause of Clank performance. I'll kick off another biset per instructions at http://g.co/ChromePerformanceRegressions.
Cc: chromium...@skia-public.iam.gserviceaccount.com
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/129409d3e40000

Roll src/third_party/depot_tools 91de11099dcd..59127a5d3941 (1 commits) by chromium-autoroll@skia-public.iam.gserviceaccount.com
https://chromium.googlesource.com/chromium/src/+/5e4780da74b289c1c52fa6df51c4bf71d337a130
memory:chrome:all_processes:reported_by_os:system_memory:private_dirty_size: 1.255e+08 → 1.229e+08 (-2.538e+06)

Introduce bundle-specific native libraries by cjgrant@chromium.org
https://chromium.googlesource.com/chromium/src/+/50ab16b8b9eff6611a5b6ec5eade28c54aa7726c
memory:chrome:all_processes:reported_by_os:system_memory:private_dirty_size: 1.229e+08 → No values

Revert "Introduce bundle-specific native libraries" by oysteine@chromium.org
https://chromium.googlesource.com/chromium/src/+/597c6ab3c70e51e91e2b0114c12d62ad062a0b8d
memory:chrome:all_processes:reported_by_os:system_memory:private_dirty_size: No values → 1.558e+08

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

Benchmark documentation link:
  https://bit.ly/system-health-benchmarks
Cc: a...@chromium.org
📍 Found significant differences after each of 3 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/123f25e0140000

Introduce bundle-specific native libraries by cjgrant@chromium.org
https://chromium.googlesource.com/chromium/src/+/50ab16b8b9eff6611a5b6ec5eade28c54aa7726c
memory:chrome:all_processes:reported_by_os:system_memory:proportional_resident_size: 1.986e+08 → No values

Revert "Introduce bundle-specific native libraries" by oysteine@chromium.org
https://chromium.googlesource.com/chromium/src/+/597c6ab3c70e51e91e2b0114c12d62ad062a0b8d
memory:chrome:all_processes:reported_by_os:system_memory:proportional_resident_size: No values → 2.29e+08

Make gfx::Image's NSImage constructor retain the incoming image. by avi@chromium.org
https://chromium.googlesource.com/chromium/src/+/c58186cc608c79f65998f4ac7e173a9c5ddf7dd8
memory:chrome:all_processes:reported_by_os:system_memory:proportional_resident_size: 2.293e+08 → 2.312e+08 (+1.902e+06)

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

Benchmark documentation link:
  https://bit.ly/system-health-benchmarks
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/14c1e813e40000

Introduce bundle-specific native libraries by cjgrant@chromium.org
https://chromium.googlesource.com/chromium/src/+/50ab16b8b9eff6611a5b6ec5eade28c54aa7726c
memory:chrome:all_processes:reported_by_os:system_memory:native_heap:proportional_resident_size: 4.263e+07 → No values

Revert "Introduce bundle-specific native libraries" by oysteine@chromium.org
https://chromium.googlesource.com/chromium/src/+/597c6ab3c70e51e91e2b0114c12d62ad062a0b8d
memory:chrome:all_processes:reported_by_os:system_memory:native_heap:proportional_resident_size: No values → 4.314e+07

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

Benchmark documentation link:
  https://bit.ly/system-health-benchmarks
Owner: a...@chromium.org
Wow, each of the 4 bisects say something different!

A close look at all bisects shows 4 commits:
- chromium-autoroll@skia...: this one seems to reduce memory requirement.
- danyao@: already confirmed to be unrelated.
- cjgrant@: got reverted immediately, so can't affect this.
- avi@: I have no clue.

So assigning to avi@.  Feel free bounce it back to me or close if yours is unrelated.
Owner: ----
Status: Available (was: Assigned)
My change was Mac-only, and moved a "retain" from one side of a call boundary to another, with zero net change.

This is a mobile regression and so my change is not relevant.
Owner: oysteine@chromium.org
Status: Assigned (was: Available)
📍 Found significant differences after each of 2 commits.
https://pinpoint-dot-chromeperf.appspot.com/job/16db775c140000

Introduce bundle-specific native libraries by cjgrant@chromium.org
https://chromium.googlesource.com/chromium/src/+/50ab16b8b9eff6611a5b6ec5eade28c54aa7726c
memory:chrome:all_processes:reported_by_chrome:gpu:effective_size: 3.849e+07 → No values

Revert "Introduce bundle-specific native libraries" by oysteine@chromium.org
https://chromium.googlesource.com/chromium/src/+/597c6ab3c70e51e91e2b0114c12d62ad062a0b8d
memory:chrome:all_processes:reported_by_chrome:gpu:effective_size: No values → 6.909e+07

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

Benchmark documentation link:
  https://bit.ly/system-health-benchmarks
📍 Couldn't reproduce a difference.
https://pinpoint-dot-chromeperf.appspot.com/job/13e915d3e40000

Sign in to add a comment