Issue metadata
Sign in to add a comment
|
1.9%-2.5% regression in memory.top_10_mobile at 600983:601056 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 7
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/11b8966de40000
,
Nov 7
📍 Found significant differences after each of 2 commits. https://pinpoint-dot-chromeperf.appspot.com/job/11b8966de40000 gpu: Fix raster implementation error for no flush result available query by khushalsagar@chromium.org https://chromium.googlesource.com/chromium/src/+/da82bcce69bf5384435976a1945c9cb3270c5abb memory:webview:all_processes:reported_by_os:system_memory:proportional_resident_size: 6.927e+07 → 7.082e+07 (+1.541e+06) Change toggle button output by dtseng@chromium.org https://chromium.googlesource.com/chromium/src/+/bd7d50d18ef4f83698e6fea3fe9d8cc3dd013751 memory:webview:all_processes:reported_by_os:system_memory:proportional_resident_size: 7.082e+07 → 7.133e+07 (+5.15e+05) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: None
,
Nov 28
Unlikely to be my change.
,
Nov 28
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/169ef9efe40000
,
Nov 28
Looks like it got fixed right after. My change did affect timings around context idle state which affects cleaning up of gpu memory. But it got restored with this change: https://chromium.googlesource.com/chromium/src/+/e898b99eac0cf5db379cf8c7e216d772095a7448, which is in the range where the graph recovered. Started a bisect to confirm that the improvement was from this change.
,
Nov 29
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/169ef9efe40000 cc: Rate limit raster metric queries. by khushalsagar@chromium.org https://chromium.googlesource.com/chromium/src/+/e898b99eac0cf5db379cf8c7e216d772095a7448 memory:webview:all_processes:reported_by_os:system_memory:proportional_resident_size: 7.175e+07 → 7.039e+07 (-1.359e+06) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: None
,
Nov 29
That was it. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Nov 7