test/jdk/jdk/jfr/event/runtime/TestBiasedLockRevocationEvents.java
author egahlin
Tue, 15 May 2018 20:24:34 +0200
changeset 50113 caf115bb98ad
child 50336 1b6ea6bcd21a
permissions -rw-r--r--
8199712: Flight Recorder Reviewed-by: coleenp, ihse, erikj, dsamersoff, mseledtsov, egahlin, mgronlun Contributed-by: erik.gahlin@oracle.com, markus.gronlund@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     2
 * Copyright (c) 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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
import java.util.HashMap;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
import java.util.List;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
import java.util.concurrent.FutureTask;
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
    static void testSingleRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
        Thread biasBreaker = triggerRevocation(1, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
        // We may or may not catch a second revocation from the biasBreaker thread exiting
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        Asserts.assertGreaterThanOrEqual(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
        Events.assertEventThread(event, "previousOwner", Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
        RecordedClass lockClass = event.getValue("lockClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
    static void testBulkRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
        Thread biasBreaker = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
        Events.assertEventThread(event, biasBreaker);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
        Events.assertField(event, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
        RecordedClass lockClass = event.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
        Asserts.assertEquals(lockClass.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
        validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
    static void testSelfRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
        recording.enable(EventNames.BiasedLockSelfRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
        MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
        Thread.holdsLock(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
        Asserts.assertEQ(events.size(), 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
        RecordedEvent event = events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
        Events.assertEventThread(event, Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
        validateStackTrace(event.getStackTrace(), "holdsLock");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
    static void testExitedThreadRevocation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
        FutureTask<MyLock> lockerTask = new FutureTask<>(() -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
           MyLock l = new MyLock();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
           touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
           return l;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
        Thread locker = new Thread(lockerTask, "BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
        locker.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
        locker.join();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
        // 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
   195
        // from the threads list. Ensure that this has happened before proceeding.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
        while (true) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
            PidJcmdExecutor jcmd = new PidJcmdExecutor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
            OutputAnalyzer oa = jcmd.execute("Thread.print", true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
            String lockerThreadFound = oa.firstMatch("BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
            if (lockerThreadFound == null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
                break;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
        };
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
        MyLock l = lockerTask.get();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
        touch(l);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
        Events.hasEvents(events);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        // Joining the locker thread can cause revocations as well, search for the interesting one
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
            RecordedClass lockClass = event.getValue("lockClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
            if (lockClass.getName().equals(MyLock.class.getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
                Events.assertEventThread(event, Thread.currentThread());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
                // Previous owner will usually be null, but can also be a thread that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
                // was created after the BiasLocker thread exited due to address reuse.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
                RecordedThread prevOwner = event.getValue("previousOwner");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
                if (prevOwner != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
                    Asserts.assertNE(prevOwner.getJavaName(), "BiasLocker");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
                validateStackTrace(event.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
                return;
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
        Asserts.fail("Did not find any revocation event for MyLock");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
    static void testBulkRevocationNoRebias() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
        Thread biasBreaker0 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
        Thread biasBreaker1 = triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
        Asserts.assertEQ(events.size(), 2);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
        // The rebias event should occur before the noRebias one
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
        RecordedEvent eventRebias = events.get(0).getStartTime().isBefore(events.get(1).getStartTime()) ? events.get(0) : events.get(1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
        RecordedEvent eventNoRebias = events.get(0).getStartTime().isBefore(events.get(1).getStartTime()) ? events.get(1) : events.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
        Events.assertEventThread(eventRebias, biasBreaker0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
        Events.assertField(eventRebias, "disableBiasing").equal(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
        Events.assertEventThread(eventNoRebias, biasBreaker1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
        Events.assertField(eventNoRebias, "disableBiasing").equal(true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
        RecordedClass lockClassRebias = eventRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
        Asserts.assertEquals(lockClassRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
        RecordedClass lockClassNoRebias = eventNoRebias.getValue("revokedClass");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
        Asserts.assertEquals(lockClassNoRebias.getName(), MyLock.class.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
        validateStackTrace(eventRebias.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   261
        validateStackTrace(eventNoRebias.getStackTrace());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   262
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
    static void testRevocationSafepointIdCorrelation() throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
        class MyLock {};
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
        Recording recording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
        recording.enable(EventNames.BiasedLockRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
        recording.enable(EventNames.BiasedLockClassRevocation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
        recording.enable(EventNames.ExecuteVMOperation);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
        recording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        triggerRevocation(BULK_REVOKE_THRESHOLD, MyLock.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
        recording.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   277
        List<RecordedEvent> events = Events.fromRecording(recording);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   278
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   279
        // Find all biased locking related VMOperation events
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
        HashMap<Integer, RecordedEvent> vmOperations = new HashMap<Integer, RecordedEvent>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
            if ((event.getEventType().getName().equals(EventNames.ExecuteVMOperation)) &&
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   283
                    (event.getValue("operation").toString().contains("Bias"))) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   284
                vmOperations.put(event.getValue("safepointId"), event);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   285
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   286
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   287
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   288
        int revokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   289
        int bulkRevokeCount = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
        // Match all revoke events to a corresponding VMOperation event
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   292
        for (RecordedEvent event : events) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   293
            if (event.getEventType().getName().equals(EventNames.BiasedLockRevocation)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
                RecordedEvent vmOpEvent = vmOperations.remove(event.getValue("safepointId"));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
                if (event.getValue("safepointId").toString().equals("-1")) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   296
                    Asserts.assertEquals(vmOpEvent, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   297
                } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
                    Events.assertField(vmOpEvent, "operation").equal("RevokeBias");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   299
                    revokeCount++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   300
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
            } else if (event.getEventType().getName().equals(EventNames.BiasedLockClassRevocation)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
                RecordedEvent vmOpEvent = vmOperations.remove(event.getValue("safepointId"));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
                Events.assertField(vmOpEvent, "operation").equal("BulkRevokeBias");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
                bulkRevokeCount++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
        // All VMOperations should have had a matching revoke event
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
        Asserts.assertEQ(vmOperations.size(), 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
        Asserts.assertGT(bulkRevokeCount, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
        Asserts.assertGT(revokeCount, bulkRevokeCount);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
}