From 52155dbbb0107c5077a6be7edfd91d4311411fc3 Mon Sep 17 00:00:00 2001 From: Albert Mingkun Yang Date: Fri, 25 Jul 2025 07:22:34 +0000 Subject: [PATCH] 8364082: jdk/jfr/event/gc/heapsummary/TestHeapSummaryEventPSParOld.java Eden should be placed first in young Reviewed-by: dholmes --- .../gc/heapsummary/HeapSummaryEventAllGcs.java | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/test/jdk/jdk/jfr/event/gc/heapsummary/HeapSummaryEventAllGcs.java b/test/jdk/jdk/jfr/event/gc/heapsummary/HeapSummaryEventAllGcs.java index 674270b0449..16dd6f44fed 100644 --- a/test/jdk/jdk/jfr/event/gc/heapsummary/HeapSummaryEventAllGcs.java +++ b/test/jdk/jdk/jfr/event/gc/heapsummary/HeapSummaryEventAllGcs.java @@ -161,18 +161,18 @@ public class HeapSummaryEventAllGcs { long toStart = Events.assertField(event, "toSpace.start").getValue(); long toEnd = Events.assertField(event, "toSpace.end").getValue(); Asserts.assertEquals(oldEnd, youngStart, "Young should start where old ends"); - Asserts.assertEquals(youngStart, edenStart, "Eden should be placed first in young"); if (fromStart < toStart) { - // [eden][from][to] - Asserts.assertGreaterThanOrEqual(fromStart, edenEnd, "From should start after eden"); + // [from][to][eden] + Asserts.assertEquals(youngStart, fromStart, "From should be placed first in young"); Asserts.assertLessThanOrEqual(fromEnd, toStart, "To should start after From"); - Asserts.assertLessThanOrEqual(toEnd, youngEnd, "To should start after From"); + Asserts.assertLessThanOrEqual(toEnd, edenStart, "Eden should start after To"); } else { - // [eden][to][from] - Asserts.assertGreaterThanOrEqual(toStart, edenEnd, "From should start after eden"); - Asserts.assertLessThanOrEqual(toEnd, fromStart, "To should start after From"); - Asserts.assertLessThanOrEqual(fromEnd, youngEnd, "To should start after From"); + // [to][from][eden] + Asserts.assertEquals(youngStart, toStart, "To should be placed first in young"); + Asserts.assertLessThanOrEqual(toEnd, fromStart, "From should start after to"); + Asserts.assertLessThanOrEqual(fromEnd, edenStart, "Eden should start after From"); } + Asserts.assertEquals(edenEnd, youngEnd, "Eden should be last of young"); } private static void checkVirtualSpace(RecordedEvent event, String structName) {