summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Roland Levillain <rpl@google.com> 2023-03-03 11:48:38 +0000
committer Roland Levillain <rpl@google.com> 2023-03-03 14:58:06 +0000
commitc91b0a323487be456a43de1db59c2e1f6a24244f (patch)
tree0f9c270e0572fadc8efb3092eb8a09c77c2d1b07
parente1ac4dccb7cc4d70ef393613085c22db99a1f134 (diff)
Temporarily suspend checks of GC duration metrics for the Gen-CC GC.
We have observed that sometimes the GC duration (both for full-heap and young-generation Generational CC collections) is null (b/271112044). Temporarily suspend the corresponding checks in test `HeapTest.GCMetrics` (part of ART gtest `art_runtime_tests`) while we investigate. Test: ART_USE_READ_BARRIER=true ART_USE_GENERATIONAL_CC=true \ m test-art-host-gtest-art_runtime_tests Bug: 271112044 Bug: 270391874 Change-Id: I8123dbe11b077c52a2859e0f2aeaf1d734ddc1ef
-rw-r--r--runtime/gc/heap_test.cc8
1 files changed, 8 insertions, 0 deletions
diff --git a/runtime/gc/heap_test.cc b/runtime/gc/heap_test.cc
index 8bcdfe79a6..874189c380 100644
--- a/runtime/gc/heap_test.cc
+++ b/runtime/gc/heap_test.cc
@@ -180,8 +180,16 @@ TEST_F(HeapTest, GCMetrics) {
AnyIsNonNull<uint64_t>, full_gc_scanned_bytes_delta, young_gc_scanned_bytes_delta);
EXPECT_PRED2(AnyIsNonNull<uint64_t>, full_gc_freed_bytes, young_gc_freed_bytes);
EXPECT_PRED2(AnyIsNonNull<uint64_t>, full_gc_freed_bytes_delta, young_gc_freed_bytes_delta);
+ // We have observed that sometimes the GC duration (both for full-heap and
+ // young-generation collections) is null (b/271112044). Temporarily
+ // suspend the following checks while we investigate.
+ //
+ // TODO(b/271112044): Investigate and adjust these expectations and/or the
+ // corresponding metric logic.
+#if 0
EXPECT_PRED2(AnyIsNonNull<uint64_t>, full_gc_duration, young_gc_duration);
EXPECT_PRED2(AnyIsNonNull<uint64_t>, full_gc_duration_delta, young_gc_duration_delta);
+#endif
} else {
// Check that only full-heap GC metrics are non-null after trigerring the collection.
EXPECT_FALSE(full_gc_collection_time->IsNull());