diff options
author | 2023-03-03 11:48:38 +0000 | |
---|---|---|
committer | 2023-03-03 14:58:06 +0000 | |
commit | c91b0a323487be456a43de1db59c2e1f6a24244f (patch) | |
tree | 0f9c270e0572fadc8efb3092eb8a09c77c2d1b07 | |
parent | e1ac4dccb7cc4d70ef393613085c22db99a1f134 (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.cc | 8 |
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()); |