test/jdk/jdk/jfr/event/runtime/TestBiasedLockRevocationEvents.java
author rehn
Fri, 23 Nov 2018 10:43:18 +0100
changeset 52672 bbfa1b3aaf7e
parent 51214 67736b4846a0
child 55625 f7e8dbb77156
permissions -rw-r--r--
8212108: SafepointSynchronizer never ending counter (big enough) Reviewed-by: dholmes, eosterlund
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) 2017, 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.runtime;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
import jdk.jfr.Recording;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
import jdk.jfr.consumer.*;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
import jdk.test.lib.Asserts;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
import jdk.test.lib.dcmd.PidJcmdExecutor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
import jdk.test.lib.jfr.EventNames;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
import jdk.test.lib.jfr.Events;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
import jdk.test.lib.process.OutputAnalyzer;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
    36
import java.util.*;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
import java.util.concurrent.FutureTask;
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
    38
import java.util.stream.Collectors;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
51214
67736b4846a0 8207830: [aix] disable jfr in build and tests
goetz
parents: 50391
diff changeset
    40
/**
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
 * @test
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
 * @key jfr
51214
67736b4846a0 8207830: [aix] disable jfr in build and tests
goetz
parents: 50391
diff changeset
    43
 * @requires vm.hasJFR
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
 * @library /test/lib
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
 * @run main/othervm jdk.jfr.event.runtime.TestBiasedLockRevocationEvents
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
public class TestBiasedLockRevocationEvents {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
    public static void main(String[] args) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
        testSingleRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
        testBulkRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
        testSelfRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
        testExitedThreadRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
        testBulkRevocationNoRebias();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
        testRevocationSafepointIdCorrelation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
    // Default value of BiasedLockingBulkRebiasThreshold is 20, and BiasedLockingBulkRevokeTreshold is 40.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
    // Using a value that will hit the first threshold once, and the second one the next time.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
    private static final int BULK_REVOKE_THRESHOLD = 25;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
    static void touch(Object lock) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
        synchronized(lock) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
    static Thread triggerRevocation(int numRevokes, Class<?> lockClass) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
        Object[] locks = new Object[numRevokes];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
        for (int i = 0; i < locks.length; ++i) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
            locks[i] = lockClass.getDeclaredConstructor().newInstance();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
            touch(locks[i]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
        Thread biasBreaker = new Thread("BiasBreaker") {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
            @Override
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
            public void run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
                for (Object lock : locks) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
                    touch(lock);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
            }
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
        biasBreaker.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
        biasBreaker.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
        return biasBreaker;
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
    // Basic stack trace validation, checking the name of the leaf method
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
    static void validateStackTrace(RecordedStackTrace stackTrace, String leafMethodName) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
        List<RecordedFrame> frames = stackTrace.getFrames();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
        Asserts.assertFalse(frames.isEmpty());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
        String name = frames.get(0).getMethod().getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
        Asserts.assertEquals(name, leafMethodName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
    // Validates that the given stack trace refers to lock.touch(); in triggerRevocation
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
    static void validateStackTrace(RecordedStackTrace stackTrace) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
        validateStackTrace(stackTrace, "touch");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   103
    // Retrieve all biased lock revocation events related to the provided lock class, sorted by start time
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   104
    static List<RecordedEvent> getRevocationEvents(Recording recording, String fieldName, Class<?> lockClass) throws Throwable {
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   105
        return Events.fromRecording(recording).stream()
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   106
                .filter(e -> ((RecordedClass)e.getValue(fieldName)).getName().equals(lockClass.getName()))
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   107
                .sorted(Comparator.comparing(RecordedEvent::getStartTime))
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   108
                .collect(Collectors.toList());
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   109
    }
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   110
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
    static void testSingleRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
        Thread biasBreaker = triggerRevocation(1, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   122
        List<RecordedEvent> events = getRevocationEvents(recording, "lockClass", MyLock.class);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   123
        Asserts.assertEQ(events.size(), 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
        Events.assertEventThread(event, "previousOwner", Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
        RecordedClass lockClass = event.getValue("lockClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
        validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
    static void testBulkRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
        Thread biasBreaker = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   146
        List<RecordedEvent> events = getRevocationEvents(recording, "revokedClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
        Events.assertField(event, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
        RecordedClass lockClass = event.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
        validateStackTrace(event.getStackTrace());
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
    static void testSelfRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
        recording.enable(EventNames.BiasedLockSelfRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
        MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
        Thread.holdsLock(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   172
        List<RecordedEvent> events = getRevocationEvents(recording, "lockClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
        Events.assertEventThread(event, Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
        validateStackTrace(event.getStackTrace(), "holdsLock");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
    static void testExitedThreadRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
        FutureTask<MyLock> lockerTask = new FutureTask<>(() -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
           MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
           touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
           return l;
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
        Thread locker = new Thread(lockerTask, "BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
        locker.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
        locker.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
        // Even after joining, the VM has a bit more work to do before the thread is actually removed
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
        // from the threads list. Ensure that this has happened before proceeding.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
        while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
            PidJcmdExecutor jcmd = new PidJcmdExecutor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
            OutputAnalyzer oa = jcmd.execute("Thread.print", true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
            String lockerThreadFound = oa.firstMatch("BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
            if (lockerThreadFound == null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
                break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
        };
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
        MyLock l = lockerTask.get();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   214
        List<RecordedEvent> events = getRevocationEvents(recording, "lockClass", MyLock.class);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   215
        Asserts.assertEQ(events.size(), 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   217
        RecordedEvent event = events.get(0);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   218
        Events.assertEventThread(event, Thread.currentThread());
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   219
        // Previous owner will usually be null, but can also be a thread that
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   220
        // was created after the BiasLocker thread exited due to address reuse.
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   221
        RecordedThread prevOwner = event.getValue("previousOwner");
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   222
        if (prevOwner != null) {
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   223
            Asserts.assertNE(prevOwner.getJavaName(), "BiasLocker");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
        }
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   225
        validateStackTrace(event.getStackTrace());
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
    static void testBulkRevocationNoRebias() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
        Thread biasBreaker0 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
        Thread biasBreaker1 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   240
        List<RecordedEvent> events = getRevocationEvents(recording, "revokedClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
        Asserts.assertEQ(events.size(), 2);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
        // The rebias event should occur before the noRebias one
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   244
        RecordedEvent eventRebias = events.get(0);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   245
        RecordedEvent eventNoRebias = events.get(1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
        Events.assertEventThread(eventRebias, biasBreaker0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
        Events.assertField(eventRebias, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
        Events.assertEventThread(eventNoRebias, biasBreaker1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
        Events.assertField(eventNoRebias, "disableBiasing").equal(true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
        RecordedClass lockClassRebias = eventRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
        Asserts.assertEquals(lockClassRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
        RecordedClass lockClassNoRebias = eventNoRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
        Asserts.assertEquals(lockClassNoRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
        validateStackTrace(eventRebias.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
        validateStackTrace(eventNoRebias.getStackTrace());
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
    static void testRevocationSafepointIdCorrelation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
        recording.enable(EventNames.ExecuteVMOperation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
        triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   277
        // Determine which safepoints included single and bulk revocation VM operations
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   278
        Set<Long> vmOperationsSingle = new HashSet<>();
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   279
        Set<Long> vmOperationsBulk = new HashSet<>();
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   280
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
        for (RecordedEvent event : events) {
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   282
            if (event.getEventType().getName().equals(EventNames.ExecuteVMOperation)) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   283
                String operation = event.getValue("operation");
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   284
                Long safepointId = event.getValue("safepointId");
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   285
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   286
                if (operation.equals("RevokeBias")) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   287
                    vmOperationsSingle.add(safepointId);
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   288
                } else if (operation.equals("BulkRevokeBias")) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   289
                    vmOperationsBulk.add(safepointId);
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   290
                }
50113
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
        int revokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
        int bulkRevokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   296
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   297
        // Match all revoke events to a corresponding VMOperation event
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   299
            if (event.getEventType().getName().equals(EventNames.BiasedLockRevocation)) {
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   300
                Long safepointId = event.getValue("safepointId");
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   301
                String lockClass = ((RecordedClass)event.getValue("lockClass")).getName();
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   302
                if (lockClass.equals(MyLock.class.getName())) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   303
                    Asserts.assertTrue(vmOperationsSingle.contains(safepointId));
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
                    revokeCount++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
            } else if (event.getEventType().getName().equals(EventNames.BiasedLockClassRevocation)) {
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   307
                Long safepointId = event.getValue("safepointId");
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   308
                String lockClass = ((RecordedClass)event.getValue("revokedClass")).getName();
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   309
                if (lockClass.toString().equals(MyLock.class.getName())) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   310
                    Asserts.assertTrue(vmOperationsBulk.contains(safepointId));
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   311
                    bulkRevokeCount++;
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   312
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   315
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
        Asserts.assertGT(bulkRevokeCount, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
        Asserts.assertGT(revokeCount, bulkRevokeCount);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   319
}