test/jdk/jdk/jfr/api/consumer/recordingstream/TestClose.java
author egahlin
Thu, 28 Nov 2019 16:38:25 +0100
changeset 59310 72f3dd43dd28
parent 58863 c16ac7a2eba4
child 59327 2c3578aa0bdf
permissions -rw-r--r--
8234888: EventStream::close doesn't abort streaming thread Reviewed-by: mgronlun, mseledtsov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     1
/*
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     2
 * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     4
 *
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    10
 *
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    15
 * accompanied this code).
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    16
 *
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    20
 *
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    23
 * questions.
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    24
 */
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    25
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    26
package jdk.jfr.api.consumer.recordingstream;
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    27
59310
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
    28
import java.time.Instant;
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    29
import java.util.concurrent.CompletableFuture;
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    30
import java.util.concurrent.CountDownLatch;
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    31
import java.util.concurrent.atomic.AtomicLong;
59310
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
    32
import java.util.concurrent.atomic.AtomicReference;
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    33
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    34
import jdk.jfr.Event;
59310
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
    35
import jdk.jfr.Recording;
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
    36
import jdk.jfr.consumer.EventStream;
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    37
import jdk.jfr.consumer.RecordingStream;
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    38
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    39
/**
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    40
 * @test
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    41
 * @summary Tests RecordingStream::close()
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    42
 * @key jfr
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    43
 * @requires vm.hasJFR
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    44
 * @library /test/lib
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    45
 * @run main/othervm jdk.jfr.api.consumer.recordingstream.TestClose
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    46
 */
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    47
public class TestClose {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    48
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    49
    private static class CloseEvent extends Event {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    50
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    51
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    52
    public static void main(String... args) throws Exception {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    53
        testCloseUnstarted();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    54
        testCloseStarted();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    55
        testCloseTwice();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    56
        testCloseStreaming();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    57
        testCloseMySelf();
59310
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
    58
        testCloseNoEvents();
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    59
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    60
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    61
    private static void testCloseMySelf() throws Exception {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    62
        log("Entering testCloseMySelf()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    63
        CountDownLatch l1 = new CountDownLatch(1);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    64
        CountDownLatch l2 = new CountDownLatch(1);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    65
        RecordingStream r = new RecordingStream();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    66
        r.onEvent(e -> {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    67
            try {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    68
                l1.await();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    69
                r.close();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    70
                l2.countDown();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    71
            } catch (InterruptedException ie) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    72
                throw new Error(ie);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    73
            }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    74
        });
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    75
        r.startAsync();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    76
        CloseEvent c = new CloseEvent();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    77
        c.commit();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    78
        l1.countDown();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    79
        l2.await();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    80
        log("Leaving testCloseMySelf()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    81
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    82
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    83
    private static void testCloseStreaming() throws Exception {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    84
        log("Entering testCloseStreaming()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    85
        CountDownLatch streaming = new CountDownLatch(1);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    86
        RecordingStream r = new RecordingStream();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    87
        AtomicLong count = new AtomicLong();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    88
        r.onEvent(e -> {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    89
            if (count.incrementAndGet() > 100) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    90
                streaming.countDown();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    91
            }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    92
        });
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    93
        r.startAsync();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    94
        var streamingLoop = CompletableFuture.runAsync(() -> {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    95
            while (true) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    96
                CloseEvent c = new CloseEvent();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    97
                c.commit();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    98
            }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
    99
        });
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   100
        streaming.await();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   101
        r.close();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   102
        streamingLoop.cancel(true);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   103
        log("Leaving testCloseStreaming()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   104
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   105
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   106
    private static void testCloseStarted() {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   107
        log("Entering testCloseStarted()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   108
        RecordingStream r = new RecordingStream();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   109
        r.startAsync();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   110
        r.close();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   111
        log("Leaving testCloseStarted()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   112
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   113
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   114
    private static void testCloseUnstarted() {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   115
        log("Entering testCloseUnstarted()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   116
        RecordingStream r = new RecordingStream();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   117
        r.close();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   118
        log("Leaving testCloseUnstarted()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   119
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   120
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   121
    private static void testCloseTwice() {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   122
        log("Entering testCloseTwice()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   123
        RecordingStream r = new RecordingStream();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   124
        r.startAsync();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   125
        r.close();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   126
        r.close();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   127
        log("Leaving testCloseTwice()");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   128
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   129
59310
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   130
    private static void testCloseNoEvents() throws Exception {
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   131
        try (Recording r = new Recording()) {
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   132
            r.start();
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   133
            CountDownLatch finished = new CountDownLatch(2);
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   134
            AtomicReference<Thread> streamingThread = new AtomicReference<>();
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   135
            try (EventStream es = EventStream.openRepository()) {
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   136
                es.setStartTime(Instant.EPOCH);
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   137
                es.onFlush( () -> {
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   138
                    streamingThread.set(Thread.currentThread());
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   139
                    finished.countDown();;
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   140
                });
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   141
                es.startAsync();
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   142
                finished.await();
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   143
            } // <- EventStream::close should terminate thread
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   144
            while (streamingThread.get().isAlive()) {
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   145
                Thread.sleep(10);
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   146
            }
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   147
        }
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   148
    }
72f3dd43dd28 8234888: EventStream::close doesn't abort streaming thread
egahlin
parents: 58863
diff changeset
   149
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   150
    private static void log(String msg) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   151
        System.out.println(msg);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   152
    }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents:
diff changeset
   153
}