test/jdk/jdk/jfr/event/gc/collection/GCEventAll.java
author egahlin
Tue, 15 May 2018 20:24:34 +0200
changeset 50113 caf115bb98ad
child 59053 ba6c248cae19
permissions -rw-r--r--
8199712: Flight Recorder Reviewed-by: coleenp, ihse, erikj, dsamersoff, mseledtsov, egahlin, mgronlun Contributed-by: erik.gahlin@oracle.com, markus.gronlund@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     2
 * Copyright (c) 2013, 2018, Oracle and/or its affiliates. All rights reserved.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     4
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    10
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    15
 * accompanied this code).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    16
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    20
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    23
 * questions.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    24
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    25
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    26
package jdk.jfr.event.gc.collection;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
import java.lang.management.ManagementFactory;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
import java.time.Duration;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
import java.time.Instant;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
import java.util.ArrayList;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
import java.util.Arrays;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
import java.util.HashSet;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
import java.util.List;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
import java.util.Random;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
import java.util.Set;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
import java.util.stream.Collectors;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
import jdk.jfr.EventType;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
import jdk.jfr.FlightRecorder;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
import jdk.jfr.Recording;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
import jdk.jfr.consumer.RecordedEvent;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
import jdk.test.lib.Asserts;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
import jdk.test.lib.jfr.EventNames;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
import jdk.test.lib.jfr.Events;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
import jdk.test.lib.jfr.GCHelper;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
/**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
 * Tests for event garbage_collection.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
 * The test function is called from TestGCEvent*.java, with different worker threads.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
 * Groups all events belonging to the same garbage collection (the same gcId).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
 * The group of events belonging to the same GC is called a batch.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
 * This class contains the verifications done and the worker threads used to generate GCs.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
 * The helper logic are in class GCHelper.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
 * Summary of verifications:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
 *   All gcIds in garbage_collection event are unique.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
 *   All events in batch has the same gcId.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
 *   Number of garbage_collection events == GarbageCollectionMXBean.getCollectionCount()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
 *   garbage_collection.sum_pause_time approximately equals GarbageCollectionMXBean.getCollectionTime()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
 *   Batch contains expected events depending on garbage_collection.name
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
 *   garbage_collection_start.timestamp == garbage_collection.startTime.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
 *   garbage_collection.timestamp >= timestamp for all other events in batch.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
 *   The start_garbage_collection and garbage_collection events must be synchronized.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
 *     This means that there may be multiple start_garbage_collection before a garbage_collection,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
 *     but garbage_collection.gcId must be equal to latest start_garbage_collection.gcId.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
 *   start_garbage_collection must be the first event in the batch,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
 *     that means no event with same gcId before garbage_collection_start event.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
 *   garbage_collection.name matches what is expected by the collectors specified in initial_configuration.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
 *   Duration for event "vm/gc/phases/pause" >= 1. Duration for phase level events >= 0.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
public class GCEventAll {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
    private String youngCollector = null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
    private String oldCollector = null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
     *  Trigger GC events by generating garbage and calling System.gc() concurrently.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
    public static void doTest() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
        // Trigger GC events by generating garbage and calling System.gc() concurrently.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
        Thread[] workerThreads = new Thread[] {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
                new Thread(GCEventAll.GarbageRunner.create(10)),
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
                new Thread(GCEventAll.SystemGcWaitRunner.create(10, 2, 1000))};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
        GCEventAll test = new GCEventAll();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
        test.doSingleTest(workerThreads);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
     * Runs the test once with given worker threads.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
     * @param workerThreads Threads that generates GCs.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
     * @param gcIds Set of all used gcIds
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
     * @throws Exception
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
    private void doSingleTest(Thread[] workerThreads) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
        enableAllGcEvents(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
        // Start with a full GC to minimize risk of getting extra GC between
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
        // getBeanCollectionCount() and recording.start().
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
        doSystemGc();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
        GCHelper.CollectionSummary startBeanCount = GCHelper.CollectionSummary.createFromMxBeans();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        for (Thread t : workerThreads) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
            t.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
        for (Thread t : workerThreads) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
            t.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
        // End with a full GC to minimize risk of getting extra GC between
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
        // recording.stop and getBeanCollectionCount().
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        doSystemGc();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
        // Add an extra System.gc() to make sure we get at least one full garbage_collection batch at
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
        // the end of the test. This extra System.gc() is only necessary when using "UseConcMarkSweepGC" and "+ExplicitGCInvokesConcurrent".
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
        doSystemGc();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
        GCHelper.CollectionSummary deltaBeanCount = GCHelper.CollectionSummary.createFromMxBeans();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
        deltaBeanCount = deltaBeanCount.calcDelta(startBeanCount);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
        List<RecordedEvent> events = Events.fromRecording(recording).stream()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
            .filter(evt -> EventNames.isGcEvent(evt.getEventType()))
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
            .collect(Collectors.toList());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
        RecordedEvent configEvent = GCHelper.getConfigEvent(events);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
        youngCollector = Events.assertField(configEvent, "youngCollector").notEmpty().getValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
        oldCollector = Events.assertField(configEvent, "oldCollector").notEmpty().getValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
        verify(events, deltaBeanCount);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
    private void enableAllGcEvents(Recording recording) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
        FlightRecorder flightrecorder = FlightRecorder.getFlightRecorder();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
        for (EventType et : flightrecorder.getEventTypes()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
            if (EventNames.isGcEvent(et)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
                recording.enable(et.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
                System.out.println("Enabled GC event: " + et.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
        System.out.println("All GC events enabled");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
    private static synchronized void doSystemGc() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
        System.gc();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
     * Does all verifications of the received events.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
     *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
     * @param events All flight recorder events.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
     * @param beanCounts Number of collections and sum pause time reported by GarbageCollectionMXBeans.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
     * @param gcIds All used gcIds. Must be unique.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
     * @throws Exception
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
    private void verify(List<RecordedEvent> events, GCHelper.CollectionSummary beanCounts) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
        List<GCHelper.GcBatch> gcBatches = null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
        GCHelper.CollectionSummary eventCounts = null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
        // For some GC configurations, the JFR recording may have stopped before we received the last gc event.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
        try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
            events = filterIncompleteGcBatch(events);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
            gcBatches = GCHelper.GcBatch.createFromEvents(events);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
            eventCounts = GCHelper.CollectionSummary.createFromEvents(gcBatches);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
            verifyUniqueIds(gcBatches);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
            verifyCollectorNames(gcBatches);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
            verifyCollectionCause(gcBatches);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
            verifyCollectionCount(eventCounts, beanCounts);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
            verifyPhaseEvents(gcBatches);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
            verifySingleGcBatch(gcBatches);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
        } catch (Throwable t) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
            log(events, gcBatches, eventCounts, beanCounts);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
            if (gcBatches != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
                for (GCHelper.GcBatch batch : gcBatches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
                    System.out.println(String.format("Batch:%n%s", batch.getLog()));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
            throw t;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
     * When using collector ConcurrentMarkSweep with -XX:+ExplicitGCInvokesConcurrent, the JFR recording may
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
     * stop before we have received the last garbage_collection event.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
     *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
     * This function does 3 things:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
     * 1. Check if the last batch is incomplete.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
     * 2. If it is incomplete, then asserts that incomplete batches are allowed for this configuration.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
     * 3. If incomplete batches are allowed, then the incomplete batch is removed.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
     *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
     * @param events All events
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
     * @return All events with any incomplete batch removed.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
     * @throws Throwable
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
    private List<RecordedEvent> filterIncompleteGcBatch(List<RecordedEvent> events) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
        List<RecordedEvent> returnEvents = new ArrayList<RecordedEvent>(events);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
        int lastGcId = getLastGcId(events);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
        List<RecordedEvent> lastBatchEvents = getEventsWithGcId(events, lastGcId);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
        String[] endEvents = {GCHelper.event_garbage_collection, GCHelper.event_old_garbage_collection, GCHelper.event_young_garbage_collection};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        boolean isComplete = containsAnyPath(lastBatchEvents, endEvents);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
        if (!isComplete) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
            // The last GC batch does not contain an end event. The batch is incomplete.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
            // This is only allowed if we are using old_collector="ConcurrentMarkSweep" and "-XX:+ExplicitGCInvokesConcurrent"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
            boolean isExplicitGCInvokesConcurrent = hasInputArgument("-XX:+ExplicitGCInvokesConcurrent");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
            boolean isConcurrentMarkSweep = GCHelper.gcConcurrentMarkSweep.equals(oldCollector);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
            String msg = String.format(
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
                    "Incomplete batch only allowed for '%s' with -XX:+ExplicitGCInvokesConcurrent",
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
                    GCHelper.gcConcurrentMarkSweep);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
            Asserts.assertTrue(isConcurrentMarkSweep && isExplicitGCInvokesConcurrent, msg);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
            // Incomplete batch is allowed with the current settings. Remove incomplete batch.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
            returnEvents.removeAll(lastBatchEvents);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
        return returnEvents;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
    private boolean hasInputArgument(String arg) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
        return ManagementFactory.getRuntimeMXBean().getInputArguments().contains(arg);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
    private List<RecordedEvent> getEventsWithGcId(List<RecordedEvent> events, int gcId) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
        List<RecordedEvent> batchEvents = new ArrayList<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
            if (GCHelper.isGcEvent(event) && GCHelper.getGcId(event) == gcId) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
                batchEvents.add(event);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
        return batchEvents;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
    private boolean containsAnyPath(List<RecordedEvent> events, String[] paths) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
        List<String> pathList = Arrays.asList(paths);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
            if (pathList.contains(event.getEventType().getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
                return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
        return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
    private int getLastGcId(List<RecordedEvent> events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
        int lastGcId = -1;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
            if (GCHelper.isGcEvent(event)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
                int gcId = GCHelper.getGcId(event);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
                if (gcId > lastGcId) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
                    lastGcId = gcId;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   261
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   262
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
        Asserts.assertTrue(lastGcId != -1, "No gcId found");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
        return lastGcId;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
     * Verifies collection count reported by flight recorder events against the values
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
     * reported by GarbageCollectionMXBean.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
     * Number of collections should match exactly.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
     * Sum pause time are allowed some margin of error because of rounding errors in measurements.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
    private void verifyCollectionCount(GCHelper.CollectionSummary eventCounts, GCHelper.CollectionSummary beanCounts) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        verifyCollectionCount(youngCollector, eventCounts.collectionCountYoung, beanCounts.collectionCountYoung);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
        verifyCollectionCount(oldCollector, eventCounts.collectionCountOld, beanCounts.collectionCountOld);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   277
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   278
    private void verifyCollectionCount(String collector, long eventCounts, long beanCounts) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   279
        if (GCHelper.gcConcurrentMarkSweep.equals(collector) || GCHelper.gcG1Old.equals(oldCollector)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
            // ConcurrentMarkSweep mixes old and new collections. Not same values as in MXBean.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
            // MXBean does not report old collections for G1Old, so we have nothing to compare with.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
            return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   283
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   284
        // JFR events and GarbageCollectorMXBean events are not updated at the same time.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   285
        // This means that number of collections may diff.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   286
        // We allow a diff of +- 1 collection count.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   287
        long minCount = Math.max(0, beanCounts - 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   288
        long maxCount = beanCounts + 1;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   289
        Asserts.assertGreaterThanOrEqual(eventCounts, minCount, "Too few event counts for collector " + collector);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
        Asserts.assertLessThanOrEqual(eventCounts, maxCount, "Too many event counts for collector " + collector);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   292
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   293
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
     * Verifies that all events belonging to a single GC are ok.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
     * A GcBatch contains all flight recorder events that belong to a single GC.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   296
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   297
    private void verifySingleGcBatch(List<GCHelper.GcBatch> batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
        for (GCHelper.GcBatch batch : batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   299
            //System.out.println("batch:\r\n" + batch.getLog());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   300
            try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
                RecordedEvent endEvent = batch.getEndEvent();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
                Asserts.assertNotNull(endEvent, "No end event in batch.");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
                Asserts.assertNotNull(batch.getName(), "No method name in end event.");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
                long longestPause = Events.assertField(endEvent, "longestPause").atLeast(0L).getValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
                Events.assertField(endEvent, "sumOfPauses").atLeast(longestPause).getValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
                Instant batchStartTime = endEvent.getStartTime();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
                Instant batchEndTime = endEvent.getEndTime();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
                for (RecordedEvent event : batch.getEvents()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
                    if (event.getEventType().getName().contains("AllocationRequiringGC")) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
                        // Unlike other events, these are sent *before* a GC.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
                        Asserts.assertLessThanOrEqual(event.getStartTime(), batchStartTime, "Timestamp in event after start event, should be sent before GC start");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
                    } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
                        Asserts.assertGreaterThanOrEqual(event.getStartTime(), batchStartTime, "startTime in event before batch start event, should be sent after GC start");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   315
                    Asserts.assertLessThanOrEqual(event.getEndTime(), batchEndTime, "endTime in event after batch end event, should be sent before GC end");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
                // Verify that all required events has been received.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   319
                String[] requiredEvents = GCHelper.requiredEvents.get(batch.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   320
                Asserts.assertNotNull(requiredEvents, "No required events specified for " + batch.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   321
                for (String requiredEvent : requiredEvents) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   322
                    boolean b = batch.containsEvent(requiredEvent);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   323
                    Asserts.assertTrue(b, String.format("%s does not contain event %s", batch, requiredEvent));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   324
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   325
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   326
                // Verify that we have exactly one heap_summary "Before GC" and one "After GC".
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   327
                int countBeforeGc = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   328
                int countAfterGc = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   329
                for (RecordedEvent event : batch.getEvents()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   330
                    if (GCHelper.event_heap_summary.equals(event.getEventType().getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   331
                        String when = Events.assertField(event, "when").notEmpty().getValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   332
                        if ("Before GC".equals(when)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   333
                            countBeforeGc++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   334
                        } else if ("After GC".equals(when)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   335
                            countAfterGc++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   336
                        } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   337
                            Asserts.fail("Unknown value for heap_summary.when: '" + when + "'");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   338
                        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   339
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   340
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   341
                if (!GCHelper.gcConcurrentMarkSweep.equals(batch.getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   342
                    // We do not get heap_summary events for ConcurrentMarkSweep
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   343
                    Asserts.assertEquals(1, countBeforeGc, "Unexpected number of heap_summary.before_gc");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   344
                    Asserts.assertEquals(1, countAfterGc, "Unexpected number of heap_summary.after_gc");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   345
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   346
            } catch (Throwable e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   347
                GCHelper.log("verifySingleGcBatch failed for gcEvent:");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   348
                GCHelper.log(batch.getLog());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   349
                throw e;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   350
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   351
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   352
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   353
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   354
    private Set<Integer> verifyUniqueIds(List<GCHelper.GcBatch> batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   355
        Set<Integer> gcIds = new HashSet<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   356
        for (GCHelper.GcBatch batch : batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   357
            Integer gcId = new Integer(batch.getGcId());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   358
            Asserts.assertFalse(gcIds.contains(gcId), "Duplicate gcId: " + gcId);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   359
            gcIds.add(gcId);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   360
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   361
        return gcIds;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   362
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   363
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   364
    private void verifyPhaseEvents(List<GCHelper.GcBatch> batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   365
        for (GCHelper.GcBatch batch : batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   366
            for(RecordedEvent event : batch.getEvents()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   367
                if (event.getEventType().getName().contains(GCHelper.pauseLevelEvent)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   368
                    Instant batchStartTime = batch.getEndEvent().getStartTime();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   369
                    Asserts.assertGreaterThanOrEqual(
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   370
                        event.getStartTime(), batchStartTime, "Phase startTime >= batch startTime. Event:" + event);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   371
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   372
                    // Duration for event "vm/gc/phases/pause" must be >= 1. Other phase event durations must be >= 0.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   373
                    Duration minDuration = Duration.ofNanos(GCHelper.event_phases_pause.equals(event.getEventType().getName()) ? 1 : 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   374
                    Duration duration = event.getDuration();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   375
                    Asserts.assertGreaterThanOrEqual(duration, minDuration, "Wrong duration. Event:" + event);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   376
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   377
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   378
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   379
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   380
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   381
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   382
     * Verifies that the collector name in initial configuration matches the name in garbage configuration event.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   383
     * If the names are not equal, then we check if this is an expected collector override.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   384
     * For example, if old collector in initial config is "G1Old" we allow both event "G1Old" and "SerialOld".
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   385
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   386
    private void verifyCollectorNames(List<GCHelper.GcBatch> batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   387
        for (GCHelper.GcBatch batch : batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   388
            String name = batch.getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   389
            Asserts.assertNotNull(name, "garbage_collection.name was null");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   390
            boolean isYoung = batch.isYoungCollection();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   391
            String expectedName = isYoung ? youngCollector : oldCollector;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   392
            if (!expectedName.equals(name)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   393
                // Collector names not equal. Check if the collector has been overridden by an expected collector.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   394
                String overrideKey = expectedName + "." + name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   395
                boolean isOverride = GCHelper.collectorOverrides.contains(overrideKey);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   396
                Asserts.assertTrue(isOverride, String.format("Unexpected event name(%s) for collectors(%s, %s)", name, youngCollector, oldCollector));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   397
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   398
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   399
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   400
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   401
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   402
     * Verifies field "cause" in garbage_collection event.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   403
     * Only check that at cause is not null and that at least 1 cause is "System.gc()"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   404
     * We might want to check more cause reasons later.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   405
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   406
    private void verifyCollectionCause(List<GCHelper.GcBatch> batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   407
        int systemGcCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   408
        for (GCHelper.GcBatch batch : batches) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   409
            RecordedEvent endEvent = batch.getEndEvent();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   410
            String cause = Events.assertField(endEvent, "cause").notEmpty().getValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   411
            // A System.GC() can be consolidated into a GCLocker GC
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   412
            if (cause.equals("System.gc()") || cause.equals("GCLocker Initiated GC")) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   413
                systemGcCount++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   414
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   415
            Asserts.assertNotNull(batch.getName(), "garbage_collection.name was null");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   416
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   417
        final String msg = "No event with cause=System.gc(), collectors(%s, %s)";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   418
        Asserts.assertTrue(systemGcCount > 0, String.format(msg, youngCollector, oldCollector));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   419
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   420
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   421
    private void log(List<RecordedEvent> events, List<GCHelper.GcBatch> batches,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   422
        GCHelper.CollectionSummary eventCounts, GCHelper.CollectionSummary beanCounts) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   423
        GCHelper.log("EventCounts:");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   424
        if (eventCounts != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   425
            GCHelper.log(eventCounts.toString());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   426
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   427
        GCHelper.log("BeanCounts:");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   428
        if (beanCounts != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   429
            GCHelper.log(beanCounts.toString());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   430
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   431
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   432
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   433
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   434
     * Thread that does a number of System.gc().
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   435
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   436
    public static class SystemGcRunner implements Runnable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   437
        private final int totalCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   438
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   439
        public SystemGcRunner(int totalCollections) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   440
            this.totalCollections = totalCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   441
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   442
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   443
        public static SystemGcRunner create(int totalCollections) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   444
            return new SystemGcRunner(totalCollections);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   445
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   446
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   447
        public void run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   448
            for (int i = 0; i < totalCollections; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   449
                GCEventAll.doSystemGc();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   450
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   451
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   452
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   453
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   454
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   455
     * Thread that creates garbage until a certain number of GCs has been run.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   456
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   457
    public static class GarbageRunner implements Runnable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   458
        private final int totalCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   459
        public byte[] dummyBuffer = null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   460
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   461
        public GarbageRunner(int totalCollections) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   462
            this.totalCollections = totalCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   463
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   464
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   465
        public static GarbageRunner create(int totalCollections) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   466
            return new GarbageRunner(totalCollections);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   467
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   468
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   469
        public void run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   470
            long currCollections = GCHelper.CollectionSummary.createFromMxBeans().sum();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   471
            long endCollections = totalCollections + currCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   472
            Random r = new Random(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   473
            while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   474
                for (int i = 0; i < 1000; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   475
                    dummyBuffer = new byte[r.nextInt(10000)];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   476
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   477
                if (GCHelper.CollectionSummary.createFromMxBeans().sum() >= endCollections) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   478
                    break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   479
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   480
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   481
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   482
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   483
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   484
    /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   485
     * Thread that runs System.gc() and then wait for a number of GCs or a maximum time.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   486
     */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   487
    public static class SystemGcWaitRunner implements Runnable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   488
        private final int totalCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   489
        private final int minWaitCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   490
        private final long maxWaitMillis;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   491
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   492
        public SystemGcWaitRunner(int totalCollections, int minWaitCollections, long maxWaitMillis) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   493
            this.totalCollections = totalCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   494
            this.minWaitCollections = minWaitCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   495
            this.maxWaitMillis = maxWaitMillis;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   496
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   497
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   498
        public static SystemGcWaitRunner create(int deltaCollections, int minWaitCollections, long maxWaitMillis) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   499
            return new SystemGcWaitRunner(deltaCollections, minWaitCollections, maxWaitMillis);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   500
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   501
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   502
        public void run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   503
            long currCount = GCHelper.CollectionSummary.createFromMxBeans().sum();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   504
            long endCount = totalCollections + currCount;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   505
            long nextSystemGcCount = currCount + minWaitCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   506
            long now = System.currentTimeMillis();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   507
            long nextSystemGcMillis = now + maxWaitMillis;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   508
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   509
            while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   510
                if (currCount >= nextSystemGcCount || System.currentTimeMillis() > nextSystemGcMillis) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   511
                    GCEventAll.doSystemGc();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   512
                    currCount = GCHelper.CollectionSummary.createFromMxBeans().sum();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   513
                    nextSystemGcCount = currCount + minWaitCollections;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   514
                } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   515
                    try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   516
                        Thread.sleep(20);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   517
                    } catch (InterruptedException e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   518
                        e.printStackTrace();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   519
                        break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   520
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   521
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   522
                currCount = GCHelper.CollectionSummary.createFromMxBeans().sum();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   523
                if (currCount >= endCount) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   524
                    break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   525
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   526
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   527
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   528
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   529
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   530
}