Issue metadata
Sign in to add a comment
|
1.5%-26.1% regression in system_health.memory_mobile at 608488:608620 |
||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 20
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/106fef38140000
,
Nov 20
😿 Pinpoint job stopped with an error. https://pinpoint-dot-chromeperf.appspot.com/job/106fef38140000 Deadline exceeded while waiting for HTTP response from URL: https://chromium.googlesource.com/chromium/src/+log/c6b23f1ea2d6f6c44d561462c41f31ed44ca8a22..a404b85a11e6cb9ce60eca8548bbaaacd29b3dbe?format=JSON
,
Nov 21
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/11ebe5ebe40000
,
Nov 21
📍 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
,
Nov 21
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.
,
Nov 21
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/129409d3e40000
,
Nov 21
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/123f25e0140000
,
Nov 21
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14c1e813e40000
,
Nov 22
📍 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
,
Nov 22
📍 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
,
Nov 23
📍 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
,
Nov 23
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.
,
Nov 26
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.
,
Nov 26
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/16db775c140000
,
Nov 26
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14ed177fe40000
,
Nov 26
📍 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
,
Nov 30
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/13e915d3e40000
,
Dec 1
📍 Couldn't reproduce a difference. https://pinpoint-dot-chromeperf.appspot.com/job/13e915d3e40000 |
|||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Nov 20