Fully skip async profile installation during CompilationMode.None [215740637]

Fixed

Bug

Status Update

No update yet.

Description

cc...@google.com

created issue #1

Jan 21, 2022 07:45PM

BaselineProfiles will be installed by ProfileInstaller asynchronously in CompilationMode.None, which can significantly interfere with longer benchmarks that don't expect them to be enabled.

Instead, we should explicitly tell the ProfileInstaller to skip via a file (or broadcast, prior to killing the app), to minimize this interference.

Comments

cc...@google.com <cc...@google.com> Feb 8, 2022 07:08PM

Reassigned to ra...@google.com.

ra...@google.com <ra...@google.com> Feb 10, 2022 08:50PM

Accepted by ra...@google.com.

ap...@google.com <ap...@google.com> #2Feb 16, 2022 04:28PM

Project: platform/frameworks/support
Branch: androidx-main

commit 700259f0afe267dfe78b93db932a3cfd827a119d
Author: Sherry Hu <shuanghu@google.com>
Date: Mon May 10 14:23:09 2021

Add transition motion between fold and unfold.

Bug: 186211031
Test: manual
Change-Id: Id60f07311eca2d94ef91dc28ae45823a475160b4

M slidingpanelayout/slidingpanelayout/build.gradle
M slidingpanelayout/slidingpanelayout/src/androidTest/java/androidx/slidingpanelayout/widget/FoldTest.kt
M slidingpanelayout/slidingpanelayout/src/main/java/androidx/slidingpanelayout/widget/SlidingPaneLayout.java

https://android-review.googlesource.com/1702066

ap...@google.com <ap...@google.com> #3Feb 16, 2022 04:28PM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-main

commit 0434487dacd97ba6fabbc8c704173a838d846381
Author: Rahul Ravikumar <rahulrav@google.com>
Date: Wed Feb 09 17:16:12 2022

Skip async profile installation for CompilationMode.None.

* Report warnings when the app is using an older version of Profile Installer.

Fixes:

b/215740637

Test: Ran TrivialStartupBenchmark with `CompilationMode.Partial` and `CompilationMode.None`.
Change-Id: I1426ee77f14f0a6e7b7323764eb79ea9596d8927

M benchmark/benchmark-macro/src/main/java/androidx/benchmark/macro/CompilationMode.kt

https://android-review.googlesource.com/1977029

cc...@google.com <cc...@google.com> #4May 4, 2022 07:17PM

Status: Assigned (reopened)

Is this skip feature broken by us now reinstalling the app every iter? Reopening to check.

ap...@google.com <ap...@google.com> #5May 10, 2022 01:01AM

Marked as fixed.

Project: platform/frameworks/support
Branch: androidx-main

commit f6441f02252c32ab603713c906d35a142ce9fc8d
Author: Rahul Ravikumar <rahulrav@google.com>
Date: Mon May 09 13:48:49 2022

Avoid having to re-install package again for CompilationMode.None() every iteration.

* Also move writing the skip file to post re-install.

Fixes:

b/231976084

b/215740637

Test: SmallListStartupBenchmark

```
SmallListStartupBenchmark_startup[startup=COLD,compilation=None]
startupMs min 232.7, median 242.0, max 262.3
timeToInitialDisplayMs min 230.5, median 239.9, max 260.0

SmallListStartupBenchmark_startup[startup=COLD,compilation=BaselineProfile]
startupMs min 223.9, median 243.6, max 287.2
timeToInitialDisplayMs min 221.5, median 241.2, max 284.9
```

Change-Id: I3051e4bc69d9e12f9aa52d5acfeec3be710de4e6

M benchmark/benchmark-macro/src/main/java/androidx/benchmark/macro/CompilationMode.kt
M benchmark/benchmark-macro/src/main/java/androidx/benchmark/macro/Macrobenchmark.kt

https://android-review.googlesource.com/2093027

Issue 215740637