Add fields in OdrefreshReported for compilation times in milliseconds
Currently we are reporting the compilation times for the boot
classpath for primary/secondary architecture and system server
in seconds, but since the values we have observed so far are
4~8 seconds for the former and 8~16 for the latter, we might
benefit from having a finer granularity. Hence, this commit
is producing the compilation time in milliseconds (alongside
the previous values in seconds, for a transition period).
Bug: 243009337
Test: atest ArtGtestTargetChroot
Change-Id: Ided03b350fe9f48f06a2a22d622b3e731d660cf4
diff --git a/stats/atoms.proto b/stats/atoms.proto
index 081af28..f96e8d0 100644
--- a/stats/atoms.proto
+++ b/stats/atoms.proto
@@ -18420,14 +18420,14 @@
optional Status status = 4;
// Compilation time of the boot classpath for the primary architecture
- // ("primary boot classpath").
+ // ("primary boot classpath"), in seconds.
optional int32 primary_bcp_compilation_seconds = 5;
// Compilation time of the boot classpath for the secondary architecture
- // ("secondary boot classpath"), if any.
+ // ("secondary boot classpath"), if any, in seconds.
optional int32 secondary_bcp_compilation_seconds = 6;
- // Compilation time of system_server classpath.
+ // Compilation time of system_server classpath, in seconds.
optional int32 system_server_compilation_seconds = 7;
// Cache space at start of update.
@@ -18435,6 +18435,17 @@
// Cache space at end of update.
optional int32 cache_space_free_end_mib = 9;
+
+ // Compilation time of the boot classpath for the primary architecture
+ // ("primary boot classpath"), in milliseconds.
+ optional int32 primary_bcp_compilation_millis = 10;
+
+ // Compilation time of the boot classpath for the secondary architecture
+ // ("secondary boot classpath"), if any, in milliseconds.
+ optional int32 secondary_bcp_compilation_millis = 11;
+
+ // Compilation time of system_server classpath, in milliseconds.
+ optional int32 system_server_compilation_millis = 12;
};
/*
