test/jdk/jdk/jfr/event/runtime/TestBiasedLockRevocationEvents.java
author rwestberg
Mon, 04 Jun 2018 16:21:08 +0200
changeset 50391 2bea53d7a27b
parent 50336 1b6ea6bcd21a
child 51214 67736b4846a0
permissions -rw-r--r--
8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint Reviewed-by: mgronlun, egahlin
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
 * @test
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
 * @key jfr
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
 * @library /test/lib
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
 * @run main/othervm jdk.jfr.event.runtime.TestBiasedLockRevocationEvents
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
public class TestBiasedLockRevocationEvents {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
    public static void main(String[] args) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
        testSingleRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
        testBulkRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
        testSelfRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
        testExitedThreadRevocation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
        testBulkRevocationNoRebias();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
        testRevocationSafepointIdCorrelation();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
    // Default value of BiasedLockingBulkRebiasThreshold is 20, and BiasedLockingBulkRevokeTreshold is 40.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
    // 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
    60
    private static final int BULK_REVOKE_THRESHOLD = 25;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
    static void touch(Object lock) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
        synchronized(lock) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
        }
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
    static Thread triggerRevocation(int numRevokes, Class<?> lockClass) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
        Object[] locks = new Object[numRevokes];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
        for (int i = 0; i < locks.length; ++i) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
            locks[i] = lockClass.getDeclaredConstructor().newInstance();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
            touch(locks[i]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
        Thread biasBreaker = new Thread("BiasBreaker") {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
            @Override
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
            public void run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
                for (Object lock : locks) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
                    touch(lock);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
                }
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
        biasBreaker.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
        biasBreaker.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
        return biasBreaker;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
    // Basic stack trace validation, checking the name of the leaf method
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
    static void validateStackTrace(RecordedStackTrace stackTrace, String leafMethodName) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
        List<RecordedFrame> frames = stackTrace.getFrames();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
        Asserts.assertFalse(frames.isEmpty());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
        String name = frames.get(0).getMethod().getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
        Asserts.assertEquals(name, leafMethodName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
    // Validates that the given stack trace refers to lock.touch(); in triggerRevocation
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
    static void validateStackTrace(RecordedStackTrace stackTrace) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
        validateStackTrace(stackTrace, "touch");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   102
    // 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
   103
    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
   104
        return Events.fromRecording(recording).stream()
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   105
                .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
   106
                .sorted(Comparator.comparing(RecordedEvent::getStartTime))
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   107
                .collect(Collectors.toList());
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   108
    }
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   109
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
    static void testSingleRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
        Thread biasBreaker = triggerRevocation(1, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   121
        List<RecordedEvent> events = getRevocationEvents(recording, "lockClass", MyLock.class);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   122
        Asserts.assertEQ(events.size(), 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        Events.assertEventThread(event, "previousOwner", Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
        RecordedClass lockClass = event.getValue("lockClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
        validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
    static void testBulkRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
        Thread biasBreaker = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   145
        List<RecordedEvent> events = getRevocationEvents(recording, "revokedClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
        Events.assertField(event, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
        RecordedClass lockClass = event.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
        validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
    static void testSelfRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
        recording.enable(EventNames.BiasedLockSelfRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
        MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
        Thread.holdsLock(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   171
        List<RecordedEvent> events = getRevocationEvents(recording, "lockClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
        Events.assertEventThread(event, Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
        validateStackTrace(event.getStackTrace(), "holdsLock");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
    static void testExitedThreadRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
        FutureTask<MyLock> lockerTask = new FutureTask<>(() -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
           MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
           touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
           return l;
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
        Thread locker = new Thread(lockerTask, "BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
        locker.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
        locker.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
        // 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
   199
        // from the threads list. Ensure that this has happened before proceeding.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
        while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
            PidJcmdExecutor jcmd = new PidJcmdExecutor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
            OutputAnalyzer oa = jcmd.execute("Thread.print", true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
            String lockerThreadFound = oa.firstMatch("BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
            if (lockerThreadFound == null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
                break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
            }
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
        MyLock l = lockerTask.get();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   213
        List<RecordedEvent> events = getRevocationEvents(recording, "lockClass", MyLock.class);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   214
        Asserts.assertEQ(events.size(), 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   216
        RecordedEvent event = events.get(0);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   217
        Events.assertEventThread(event, Thread.currentThread());
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   218
        // 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
   219
        // 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
   220
        RecordedThread prevOwner = event.getValue("previousOwner");
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   221
        if (prevOwner != null) {
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   222
            Asserts.assertNE(prevOwner.getJavaName(), "BiasLocker");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
        }
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   224
        validateStackTrace(event.getStackTrace());
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
    static void testBulkRevocationNoRebias() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
        Thread biasBreaker0 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
        Thread biasBreaker1 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
        recording.stop();
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   239
        List<RecordedEvent> events = getRevocationEvents(recording, "revokedClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
        Asserts.assertEQ(events.size(), 2);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
        // 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
   243
        RecordedEvent eventRebias = events.get(0);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   244
        RecordedEvent eventNoRebias = events.get(1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
        Events.assertEventThread(eventRebias, biasBreaker0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
        Events.assertField(eventRebias, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
        Events.assertEventThread(eventNoRebias, biasBreaker1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
        Events.assertField(eventNoRebias, "disableBiasing").equal(true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
        RecordedClass lockClassRebias = eventRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
        Asserts.assertEquals(lockClassRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
        RecordedClass lockClassNoRebias = eventNoRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
        Asserts.assertEquals(lockClassNoRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
        validateStackTrace(eventRebias.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
        validateStackTrace(eventNoRebias.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   261
    static void testRevocationSafepointIdCorrelation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   262
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
        recording.enable(EventNames.ExecuteVMOperation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
        triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   276
        // Determine which safepoints included single and bulk revocation VM operations
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   277
        Set<Integer> vmOperationsSingle = new HashSet<>();
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   278
        Set<Integer> vmOperationsBulk = new HashSet<>();
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   279
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
        for (RecordedEvent event : events) {
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   281
            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
   282
                String operation = event.getValue("operation");
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   283
                Integer safepointId = event.getValue("safepointId");
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   284
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   285
                if (operation.equals("RevokeBias")) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   286
                    vmOperationsSingle.add(safepointId);
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   287
                } else if (operation.equals("BulkRevokeBias")) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   288
                    vmOperationsBulk.add(safepointId);
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   289
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
            }
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
        int revokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
        int bulkRevokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   296
        // Match all revoke events to a corresponding VMOperation event
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   297
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
            if (event.getEventType().getName().equals(EventNames.BiasedLockRevocation)) {
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   299
                Integer safepointId = event.getValue("safepointId");
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   300
                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
   301
                if (lockClass.equals(MyLock.class.getName())) {
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   302
                    Asserts.assertTrue(vmOperationsSingle.contains(safepointId));
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
                    revokeCount++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
            } else if (event.getEventType().getName().equals(EventNames.BiasedLockClassRevocation)) {
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   306
                Integer safepointId = event.getValue("safepointId");
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   307
                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
   308
                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
   309
                    Asserts.assertTrue(vmOperationsBulk.contains(safepointId));
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   310
                    bulkRevokeCount++;
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   311
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
            }
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
        Asserts.assertGT(bulkRevokeCount, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
        Asserts.assertGT(revokeCount, bulkRevokeCount);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
}