test/jdk/jdk/jfr/event/runtime/TestBiasedLockRevocationEvents.java
author pliden
Wed, 20 Nov 2019 10:37:46 +0100
changeset 59152 59272e9e0635
parent 59005 01d47d511f5f
permissions -rw-r--r--
8234383: Test TestBiasedLockRevocationEvents.java assumes -XX:UseBiasedLocking is enabled Reviewed-by: mgronlun, tschatzl
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
55625
f7e8dbb77156 8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents: 52672
diff changeset
     2
 * Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
50113
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
 *
59152
59272e9e0635 8234383: Test TestBiasedLockRevocationEvents.java assumes -XX:UseBiasedLocking is enabled
pliden
parents: 59005
diff changeset
    46
 * @run main/othervm -XX:+UseBiasedLocking jdk.jfr.event.runtime.TestBiasedLockRevocationEvents
50113
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
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   104
    static List<RecordedEvent> getRevocationEvents(Recording recording, String eventTypeName, String fieldName, Class<?> lockClass) throws Throwable {
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   105
        return Events.fromRecording(recording).stream()
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   106
                .filter(e -> e.getEventType().getName().equals(eventTypeName))
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   107
                .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
   108
                .sorted(Comparator.comparing(RecordedEvent::getStartTime))
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   109
                .collect(Collectors.toList());
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   110
    }
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   111
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
    static void testSingleRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
        Thread biasBreaker = triggerRevocation(1, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
        recording.stop();
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   123
        List<RecordedEvent> events = getRevocationEvents(recording, EventNames.BiasedLockRevocation, "lockClass", MyLock.class);
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   124
        Asserts.assertEQ(events.size(), 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
        Events.assertEventThread(event, "previousOwner", Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
        RecordedClass lockClass = event.getValue("lockClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
        validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
    static void testBulkRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
        Thread biasBreaker = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
        recording.stop();
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   147
        List<RecordedEvent> events = getRevocationEvents(recording, EventNames.BiasedLockClassRevocation, "revokedClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
        Events.assertField(event, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
        RecordedClass lockClass = event.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
        validateStackTrace(event.getStackTrace());
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
    static void testSelfRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
        recording.enable(EventNames.BiasedLockSelfRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
        MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
        Thread.holdsLock(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
        recording.stop();
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   173
        List<RecordedEvent> events = getRevocationEvents(recording, EventNames.BiasedLockSelfRevocation, "lockClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
        Events.assertEventThread(event, Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
        validateStackTrace(event.getStackTrace(), "holdsLock");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
    static void testExitedThreadRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
        FutureTask<MyLock> lockerTask = new FutureTask<>(() -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
           MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
           touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
           return l;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
        Thread locker = new Thread(lockerTask, "BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
        locker.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
        locker.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
        // 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
   201
        // from the threads list. Ensure that this has happened before proceeding.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
        while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
            PidJcmdExecutor jcmd = new PidJcmdExecutor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
            OutputAnalyzer oa = jcmd.execute("Thread.print", true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
            String lockerThreadFound = oa.firstMatch("BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
            if (lockerThreadFound == null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
                break;
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
        MyLock l = lockerTask.get();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
        recording.stop();
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   215
        List<RecordedEvent> events = getRevocationEvents(recording, EventNames.BiasedLockRevocation, "lockClass", MyLock.class);
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   216
        Asserts.assertEQ(events.size(), 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   218
        RecordedEvent event = events.get(0);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   219
        Events.assertEventThread(event, Thread.currentThread());
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   220
        // 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
   221
        // 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
   222
        RecordedThread prevOwner = event.getValue("previousOwner");
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   223
        if (prevOwner != null) {
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   224
            Asserts.assertNE(prevOwner.getJavaName(), "BiasLocker");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
        }
50336
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   226
        validateStackTrace(event.getStackTrace());
50113
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
    static void testBulkRevocationNoRebias() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
        Thread biasBreaker0 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
        Thread biasBreaker1 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
        recording.stop();
59005
01d47d511f5f 8231922: Could not find field with name revokedClass
mgronlun
parents: 55625
diff changeset
   241
        List<RecordedEvent> events = getRevocationEvents(recording, EventNames.BiasedLockClassRevocation, "revokedClass", MyLock.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
        Asserts.assertEQ(events.size(), 2);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
        // 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
   245
        RecordedEvent eventRebias = events.get(0);
1b6ea6bcd21a 8203237: JFR TestBiasedLockRevocationEvents should ignore events unrelated to the test
rwestberg
parents: 50113
diff changeset
   246
        RecordedEvent eventNoRebias = events.get(1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
        Events.assertEventThread(eventRebias, biasBreaker0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
        Events.assertField(eventRebias, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
        Events.assertEventThread(eventNoRebias, biasBreaker1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
        Events.assertField(eventNoRebias, "disableBiasing").equal(true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
        RecordedClass lockClassRebias = eventRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
        Asserts.assertEquals(lockClassRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
        RecordedClass lockClassNoRebias = eventNoRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
        Asserts.assertEquals(lockClassNoRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
        validateStackTrace(eventRebias.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
        validateStackTrace(eventNoRebias.getStackTrace());
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
    static void testRevocationSafepointIdCorrelation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
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
55625
f7e8dbb77156 8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents: 52672
diff changeset
   277
        // Determine which safepoints included bulk revocation VM operations
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   278
        Set<Long> vmOperationsBulk = new HashSet<>();
50391
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");
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   283
                Long safepointId = event.getValue("safepointId");
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   284
55625
f7e8dbb77156 8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents: 52672
diff changeset
   285
                if (operation.equals("BulkRevokeBias")) {
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   286
                    vmOperationsBulk.add(safepointId);
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   287
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   288
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   289
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
        int bulkRevokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   292
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   293
        // Match all revoke events to a corresponding VMOperation event
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
        for (RecordedEvent event : events) {
55625
f7e8dbb77156 8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents: 52672
diff changeset
   295
            if (event.getEventType().getName().equals(EventNames.BiasedLockClassRevocation)) {
52672
bbfa1b3aaf7e 8212108: SafepointSynchronizer never ending counter (big enough)
rehn
parents: 51214
diff changeset
   296
                Long safepointId = event.getValue("safepointId");
50391
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   297
                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
   298
                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
   299
                    Asserts.assertTrue(vmOperationsBulk.contains(safepointId));
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   300
                    bulkRevokeCount++;
2bea53d7a27b 8202095: JFR TestBiasedLockRevocationEvents should cope with multiple events during a single safepoint
rwestberg
parents: 50336
diff changeset
   301
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
        Asserts.assertGT(bulkRevokeCount, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
}