src/jdk.jfr/share/classes/jdk/jfr/internal/RequestEngine.java
author egahlin
Fri, 07 Dec 2018 14:19:20 +0100
changeset 52899 325c95779368
parent 50113 caf115bb98ad
child 57360 5d043a159d5c
child 58863 c16ac7a2eba4
permissions -rw-r--r--
8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it Reviewed-by: mgronlun
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) 2016, 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.internal;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
import java.security.AccessControlContext;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
import java.security.AccessController;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
import java.security.PrivilegedAction;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
import java.util.ArrayList;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
import java.util.Collection;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
import java.util.List;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
import java.util.Objects;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
import java.util.concurrent.CopyOnWriteArrayList;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
import java.util.function.Predicate;
52899
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    37
import jdk.jfr.Event;
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    38
import jdk.jfr.EventType;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
public final class RequestEngine {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
    private final static JVM jvm = JVM.getJVM();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
    final static class RequestHook {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
        private final Runnable hook;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
        private final PlatformEventType type;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
        private final AccessControlContext accessControllerContext;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
        private long delta;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
        // Java events
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
        private RequestHook(AccessControlContext acc, PlatformEventType eventType, Runnable hook) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
            this.hook = hook;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
            this.type = eventType;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
            this.accessControllerContext = acc;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
        // native events
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
        RequestHook(PlatformEventType eventType) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
            this(null, eventType, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
        private void execute() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
            try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
                if (accessControllerContext == null) { // native
52899
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    65
                    if (type.isJDK()) {
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    66
                        hook.run();
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    67
                    } else {
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    68
                        jvm.emitEvent(type.getId(), JVM.counterTime(), 0);
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
    69
                    }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
                    Logger.log(LogTag.JFR_SYSTEM_EVENT, LogLevel.DEBUG, ()-> "Executed periodic hook for " + type.getLogName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
                } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
                    executeSecure();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
            } catch (Throwable e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
                // Prevent malicious user to propagate exception callback in the wrong context
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
                Logger.log(LogTag.JFR_SYSTEM_EVENT, LogLevel.WARN, "Exception occured during execution of period hook for " + type.getLogName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
        private void executeSecure() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
            AccessController.doPrivileged(new PrivilegedAction<Void>() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
                @Override
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
                public Void run() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
                    try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
                        hook.run();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
                        Logger.log(LogTag.JFR_EVENT, LogLevel.DEBUG, ()-> "Executed periodic hook for " + type.getLogName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
                    } catch (Throwable t) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
                        // Prevent malicious user to propagate exception callback in the wrong context
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
                        Logger.log(LogTag.JFR_EVENT, LogLevel.WARN, "Exception occured during execution of period hook for " + type.getLogName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
                    return null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
            }, accessControllerContext);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
        }
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
    private final static List<RequestHook> entries = new CopyOnWriteArrayList<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
    private static long lastTimeMillis;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
    public static void addHook(AccessControlContext acc, PlatformEventType type, Runnable hook) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
        Objects.requireNonNull(acc);
52899
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   102
        addHookInternal(acc, type, hook);
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   103
    }
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   104
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   105
    private static void addHookInternal(AccessControlContext acc, PlatformEventType type, Runnable hook) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
        RequestHook he = new RequestHook(acc, type, hook);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
        for (RequestHook e : entries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
            if (e.hook == hook) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
                throw new IllegalArgumentException("Hook has already been added");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
        he.type.setEventHook(true);
52899
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   113
        // Insertion takes O(2*n), could be O(1) with HashMap, but
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   114
        // thinking is that CopyOnWriteArrayList is faster
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   115
        // to iterate over, which will happen more over time.
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
        entries.add(he);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        logHook("Added", type);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
52899
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   120
    public static void addTrustedJDKHook(Class<? extends Event> eventClass, Runnable runnable) {
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   121
        if (eventClass.getClassLoader() != null) {
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   122
            throw new SecurityException("Hook can only be registered for event classes that are loaded by the bootstrap class loader");
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   123
        }
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   124
        if (runnable.getClass().getClassLoader() != null) {
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   125
            throw new SecurityException("Runnable hook class must be loaded by the bootstrap class loader");
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   126
        }
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   127
        EventType eType = MetadataRepository.getInstance().getEventType(eventClass);
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   128
        PlatformEventType pType = PrivateAccess.getInstance().getPlatformEventType(eType);
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   129
        addHookInternal(null, pType, runnable);
325c95779368 8207829: FlightRecorderMXBeanImpl is leaking the first classloader which calls it
egahlin
parents: 50113
diff changeset
   130
    }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
    private static void logHook(String action, PlatformEventType type) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
        if (type.isJDK() || type.isJVM()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
            Logger.log(LogTag.JFR_SYSTEM_EVENT, LogLevel.INFO, action + " periodic hook for " + type.getLogName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
        } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
            Logger.log(LogTag.JFR_EVENT, LogLevel.INFO, action + " periodic hook for " + type.getLogName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
    // Takes O(2*n), see addHook.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
    public static boolean removeHook(Runnable hook) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
        for (RequestHook rh : entries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
            if (rh.hook == hook) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
                entries.remove(rh);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
                rh.type.setEventHook(false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
                logHook("Removed", rh.type);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
                return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
        return false;
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
    // Only to be used for JVM events. No access control contest
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
    // or check if hook already exists
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
    static void addHooks(List<RequestHook> newEntries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
        List<RequestHook> addEntries = new ArrayList<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
        for (RequestHook rh : newEntries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
            rh.type.setEventHook(true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
            addEntries.add(rh);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
            logHook("Added", rh.type);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
        entries.addAll(newEntries);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
    static void doChunkEnd() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
        doChunk(x -> x.isEndChunk());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
    static void doChunkBegin() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
        doChunk(x -> x.isBeginChunk());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
    private static void doChunk(Predicate<PlatformEventType> predicate) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
        for (RequestHook requestHook : entries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
            PlatformEventType s = requestHook.type;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
            if (s.isEnabled() && predicate.test(s)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
                requestHook.execute();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
    static long doPeriodic() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
        return run_requests(entries);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
    // code copied from native impl.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
    private static long run_requests(Collection<RequestHook> entries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
        long last = lastTimeMillis;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
        // Bug 9000556 - current time millis has rather lame resolution
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
        // The use of os::elapsed_counter() is deliberate here, we don't
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
        // want it exchanged for os::ft_elapsed_counter().
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
        // Keeping direct call os::elapsed_counter() here for reliable
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
        // real time values in order to decide when registered requestable
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
        // events are due.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
        long now = System.currentTimeMillis();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
        long min = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
        long delta = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
        if (last == 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
            last = now;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
        // time from then to now
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
        delta = now - last;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
        if (delta < 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
            // to handle time adjustments
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
            // for example Daylight Savings
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
            lastTimeMillis = now;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
            return 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        for (RequestHook he : entries) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
            long left = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
            PlatformEventType es = he.type;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
            // Not enabled, skip.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
            if (!es.isEnabled() || es.isEveryChunk()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
                continue;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
            long r_period = es.getPeriod();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
            long r_delta = he.delta;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
            // add time elapsed.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
            r_delta += delta;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
            // above threshold?
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
            if (r_delta >= r_period) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
                // Bug 9000556 - don't try to compensate
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
                // for wait > period
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
                r_delta = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
                he.execute();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
                ;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
            // calculate time left
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
            left = (r_period - r_delta);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
            /**
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
             * nothing outside checks that a period is >= 0, so left can end up
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
             * negative here. ex. (r_period =(-1)) - (r_delta = 0) if it is,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
             * handle it.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
             */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
            if (left < 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
                left = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
            // assign delta back
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
            he.delta = r_delta;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
            if (min == 0 || left < min) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
                min = left;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
        lastTimeMillis = now;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
        return min;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
}