src/jdk.jfr/share/classes/jdk/jfr/internal/EventInstrumentation.java
author egahlin
Thu, 22 Aug 2019 20:36:52 +0200
branchJEP-349-branch
changeset 57848 bd961c64ecce
parent 57781 7758008806d1
child 58112 e7754025004b
permissions -rw-r--r--
Take timestamp on commit
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.lang.reflect.Field;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
import java.lang.reflect.Modifier;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
import java.lang.reflect.Parameter;
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.HashSet;
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.Set;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
import java.util.function.Consumer;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
import jdk.internal.org.objectweb.asm.ClassReader;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
import jdk.internal.org.objectweb.asm.ClassWriter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
import jdk.internal.org.objectweb.asm.Label;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
import jdk.internal.org.objectweb.asm.MethodVisitor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
import jdk.internal.org.objectweb.asm.Opcodes;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
import jdk.internal.org.objectweb.asm.Type;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
import jdk.internal.org.objectweb.asm.commons.Method;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
import jdk.internal.org.objectweb.asm.tree.AnnotationNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
import jdk.internal.org.objectweb.asm.tree.ClassNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
import jdk.internal.org.objectweb.asm.tree.FieldNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
import jdk.internal.org.objectweb.asm.tree.MethodNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
import jdk.jfr.Enabled;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
import jdk.jfr.Event;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    50
import jdk.jfr.Name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
import jdk.jfr.Registered;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
import jdk.jfr.SettingControl;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
import jdk.jfr.SettingDefinition;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
import jdk.jfr.internal.handlers.EventHandler;
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
 * Class responsible for adding instrumentation to a subclass of {@link Event}.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
public final class EventInstrumentation {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
    static final class SettingInfo {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
        private String methodName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
        private String internalSettingName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
        private String settingDescriptor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
        final String fieldName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
        final int index;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
        // Used when instantiating Setting
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
        SettingControl settingControl;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
        public SettingInfo(String fieldName, int index) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
            this.fieldName = fieldName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
            this.index = index;
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
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
    static final class FieldInfo {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
        private final static Type STRING = Type.getType(String.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
        final String fieldName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
        final String fieldDescriptor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
        final String internalClassName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
        public FieldInfo(String fieldName, String fieldDescriptor, String internalClassName) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
            this.fieldName = fieldName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
            this.fieldDescriptor = fieldDescriptor;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
            this.internalClassName = internalClassName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
        public boolean isString() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
            return STRING.getDescriptor().equals(fieldDescriptor);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    90
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
    public static final String FIELD_EVENT_THREAD = "eventThread";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
    public static final String FIELD_STACK_TRACE = "stackTrace";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
    public static final String FIELD_DURATION = "duration";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
    static final String FIELD_EVENT_HANDLER = "eventHandler";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
    static final String FIELD_START_TIME = "startTime";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
    private static final Type ANNOTATION_TYPE_NAME = Type.getType(Name.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
    private static final Type ANNOTATION_TYPE_REGISTERED = Type.getType(Registered.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
    private static final Type ANNOTATION_TYPE_ENABLED = Type.getType(Enabled.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
    private static final Type TYPE_EVENT_HANDLER = Type.getType(EventHandler.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
    private static final Type TYPE_SETTING_CONTROL = Type.getType(SettingControl.class);
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   105
    private static final Type TYPE_OBJECT  = Type.getType(Object.class);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
    private static final Method METHOD_COMMIT = new Method("commit", Type.VOID_TYPE, new Type[0]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
    private static final Method METHOD_BEGIN = new Method("begin", Type.VOID_TYPE, new Type[0]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
    private static final Method METHOD_END = new Method("end", Type.VOID_TYPE, new Type[0]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
    private static final Method METHOD_IS_ENABLED = new Method("isEnabled", Type.BOOLEAN_TYPE, new Type[0]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
    private static final Method METHOD_TIME_STAMP = new Method("timestamp", Type.LONG_TYPE, new Type[0]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
    private static final Method METHOD_EVENT_SHOULD_COMMIT = new Method("shouldCommit", Type.BOOLEAN_TYPE, new Type[0]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
    private static final Method METHOD_EVENT_HANDLER_SHOULD_COMMIT = new Method("shouldCommit", Type.BOOLEAN_TYPE, new Type[] { Type.LONG_TYPE });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
    private static final Method METHOD_DURATION = new Method("duration", Type.LONG_TYPE, new Type[] { Type.LONG_TYPE });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
    private final ClassNode classNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
    private final List<SettingInfo> settingInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
    private final List<FieldInfo> fieldInfos;;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
    private final Method writeMethod;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
    private final String eventHandlerXInternalName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
    private final String eventName;
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   121
    private final boolean untypedEventHandler;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
    private boolean guardHandlerReference;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
    private Class<?> superClass;
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   124
    private final static boolean streamingCommit = true; //!SecuritySupport.getBooleanProperty("jfr.instrument.streaming");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
    EventInstrumentation(Class<?> superClass, byte[] bytes, long id) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
        this.superClass = superClass;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
        this.classNode = createClassNode(bytes);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
        this.settingInfos = buildSettingInfos(superClass, classNode);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
        this.fieldInfos = buildFieldInfos(superClass, classNode);
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   131
        this.untypedEventHandler = hasUntypedHandler();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
        this.writeMethod = makeWriteMethod(fieldInfos);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
        this.eventHandlerXInternalName = ASMToolkit.getInternalName(EventHandlerCreator.makeEventHandlerName(id));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
        String n =  annotationValue(classNode, ANNOTATION_TYPE_NAME.getDescriptor(), String.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
        this.eventName = n == null ? classNode.name.replace("/", ".") : n;
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   136
    }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   138
    private boolean hasUntypedHandler() {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   139
        for (FieldNode field : classNode.fields) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   140
            if (FIELD_EVENT_HANDLER.equals(field.name)) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   141
                return field.desc.equals(TYPE_OBJECT.getDescriptor());
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   142
            }
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   143
        }
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   144
        throw new InternalError("Class missing handler field");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
    public String getClassName() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
      return classNode.name.replace("/",".");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   149
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
    private ClassNode createClassNode(byte[] bytes) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
        ClassNode classNode = new ClassNode();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
        ClassReader classReader = new ClassReader(bytes);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
        classReader.accept(classNode, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
        return classNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
    boolean isRegistered() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
        Boolean result = annotationValue(classNode, ANNOTATION_TYPE_REGISTERED.getDescriptor(), Boolean.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
        if (result != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
            return result.booleanValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
        if (superClass != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
            Registered r = superClass.getAnnotation(Registered.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
            if (r != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
                return r.value();
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
        return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
    boolean isEnabled() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
        Boolean result = annotationValue(classNode, ANNOTATION_TYPE_ENABLED.getDescriptor(), Boolean.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
        if (result != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
            return result.booleanValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
        if (superClass != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
            Enabled e = superClass.getAnnotation(Enabled.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
            if (e != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
                return e.value();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
        return true;
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
    @SuppressWarnings("unchecked")
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
    private static <T> T annotationValue(ClassNode classNode, String typeDescriptor, Class<?> type) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
        if (classNode.visibleAnnotations != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
            for (AnnotationNode a : classNode.visibleAnnotations) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
                if (typeDescriptor.equals(a.desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
                    List<Object> values = a.values;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
                    if (values != null && values.size() == 2) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
                        Object key = values.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
                        Object value = values.get(1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
                        if (key instanceof String && value != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
                            if (type == value.getClass()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
                                String keyName = (String) key;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
                                if ("value".equals(keyName)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
                                   return (T) value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
                                }
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
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
        return null;
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
    private static List<SettingInfo> buildSettingInfos(Class<?> superClass, ClassNode classNode) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
        Set<String> methodSet = new HashSet<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        List<SettingInfo> settingInfos = new ArrayList<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
        String settingDescriptor = Type.getType(SettingDefinition.class).getDescriptor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
        for (MethodNode m : classNode.methods) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
            if (m.visibleAnnotations != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
                for (AnnotationNode an : m.visibleAnnotations) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
                    // We can't really validate the method at this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
                    // stage. We would need to check that the parameter
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
                    // is an instance of SettingControl.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
                    if (settingDescriptor.equals(an.desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
                        Type returnType = Type.getReturnType(m.desc);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
                        if (returnType.equals(Type.getType(Boolean.TYPE))) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
                            Type[] args = Type.getArgumentTypes(m.desc);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
                            if (args.length == 1) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
                                Type paramType = args[0];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
                                String fieldName = EventControl.FIELD_SETTING_PREFIX + settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
                                int index = settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
                                SettingInfo si = new SettingInfo(fieldName, index);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
                                si.methodName = m.name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
                                si.settingDescriptor = paramType.getDescriptor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
                                si.internalSettingName = paramType.getInternalName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
                                methodSet.add(m.name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
                                settingInfos.add(si);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
                            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
                        }
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
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
        }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   240
        for (Class<?> c = superClass; c != jdk.internal.event.Event.class; c = c.getSuperclass()) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
            for (java.lang.reflect.Method method : c.getDeclaredMethods()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
                if (!methodSet.contains(method.getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
                    // skip private method in base classes
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
                    if (!Modifier.isPrivate(method.getModifiers())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
                        if (method.getReturnType().equals(Boolean.TYPE)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
                            if (method.getParameterCount() == 1) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
                                Parameter param = method.getParameters()[0];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
                                Type paramType = Type.getType(param.getType());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
                                String fieldName = EventControl.FIELD_SETTING_PREFIX + settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
                                int index = settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
                                SettingInfo si = new SettingInfo(fieldName, index);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
                                si.methodName = method.getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
                                si.settingDescriptor = paramType.getDescriptor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
                                si.internalSettingName = paramType.getInternalName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
                                methodSet.add(method.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
                                settingInfos.add(si);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
                            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
                        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   261
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   262
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
        return settingInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
    private static List<FieldInfo> buildFieldInfos(Class<?> superClass, ClassNode classNode) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
        Set<String> fieldSet = new HashSet<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
        List<FieldInfo> fieldInfos = new ArrayList<>(classNode.fields.size());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
        // These two field are added by native as transient so they will be
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
        // ignored by the loop below.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
        // The benefit of adding them manually is that we can
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
        // control in which order they occur and we can add @Name, @Description
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
        // in Java, instead of in native. It also means code for adding implicit
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        // fields for native can be reused by Java.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
        fieldInfos.add(new FieldInfo("startTime", Type.LONG_TYPE.getDescriptor(), classNode.name));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
        fieldInfos.add(new FieldInfo("duration", Type.LONG_TYPE.getDescriptor(), classNode.name));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   277
        for (FieldNode field : classNode.fields) {
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   278
            if (!fieldSet.contains(field.name) && isValidField(field.access, Type.getType(field.desc).getClassName())) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   279
                FieldInfo fi = new FieldInfo(field.name, field.desc, classNode.name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
                fieldInfos.add(fi);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
                fieldSet.add(field.name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   283
        }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   284
        for (Class<?> c = superClass; c != jdk.internal.event.Event.class; c = c.getSuperclass()) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   285
            for (Field field : c.getDeclaredFields()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   286
                // skip private field in base classes
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   287
                if (!Modifier.isPrivate(field.getModifiers())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   288
                    if (isValidField(field.getModifiers(), field.getType().getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   289
                        String fieldName = field.getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
                        if (!fieldSet.contains(fieldName)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
                            Type fieldType = Type.getType(field.getType());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   292
                            String internalClassName = ASMToolkit.getInternalName(c.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   293
                            fieldInfos.add(new FieldInfo(fieldName, fieldType.getDescriptor(), internalClassName));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
                            fieldSet.add(fieldName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
                        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   296
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   297
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   299
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   300
        return fieldInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
    public static boolean isValidField(int access, String className) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
        if (Modifier.isTransient(access) || Modifier.isStatic(access)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
            return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
        return jdk.jfr.internal.Type.isValidJavaFieldType(className);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
    public byte[] buildInstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
        makeInstrumented();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
        return toByteArray();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   315
    private byte[] toByteArray() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
        ClassWriter cw = new ClassWriter(ClassWriter.COMPUTE_FRAMES);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
        classNode.accept(cw);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
        cw.visitEnd();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   319
        byte[] result = cw.toByteArray();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   320
        Utils.writeGeneratedASM(classNode.name, result);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   321
        return result;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   322
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   323
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   324
    public byte[] builUninstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   325
        makeUninstrumented();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   326
        return toByteArray();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   327
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   328
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   329
    private void makeInstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   330
        // MyEvent#isEnabled()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   331
        updateMethod(METHOD_IS_ENABLED, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   332
            Label nullLabel = new Label();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   333
            if (guardHandlerReference) {
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   334
                getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   335
                methodVisitor.visitJumpInsn(Opcodes.IFNULL, nullLabel);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   336
            }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   337
            getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   338
            ASMToolkit.invokeVirtual(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_IS_ENABLED);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   339
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   340
            if (guardHandlerReference) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   341
                methodVisitor.visitLabel(nullLabel);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   342
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   343
                methodVisitor.visitInsn(Opcodes.ICONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   344
                methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   345
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   346
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   347
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   348
        // MyEvent#begin()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   349
        updateMethod(METHOD_BEGIN, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   350
            methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   351
            ASMToolkit.invokeStatic(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   352
            methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   353
            methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   354
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   355
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   356
        // MyEvent#end()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   357
        updateMethod(METHOD_END, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   358
            methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   359
            methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   360
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   361
            ASMToolkit.invokeStatic(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_DURATION);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   362
            methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_DURATION, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   363
            methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   364
            methodVisitor.visitMaxs(0, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   365
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   366
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   367
        // MyEvent#commit() - Java event writer
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   368
        if (streamingCommit) {
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   369
            updateMethod(METHOD_COMMIT, methodVisitor -> {
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   370
                methodVisitor.visitCode();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   371
                // Labels
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   372
                Label enabled = new Label();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   373
                Label end = new Label();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   374
                Label durationEvent = new Label();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   375
                Label writeToHandler = new Label();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   376
                Label timestampOnCommit = new Label();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   377
                // if (!isEnable()) {
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   378
                // return;
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   379
                // }
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   380
                methodVisitor.visitCode();
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   381
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   382
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), METHOD_IS_ENABLED.getName(), METHOD_IS_ENABLED.getDescriptor(), false);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   383
                methodVisitor.visitJumpInsn(Opcodes.IFNE, enabled);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   384
                methodVisitor.visitInsn(Opcodes.RETURN);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   385
                methodVisitor.visitLabel(enabled);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   386
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   387
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   388
                // if (startTime == 0) {
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   389
                // startTime = EventWriter.timestamp();
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   390
                // } else {
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   391
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   392
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   393
                methodVisitor.visitInsn(Opcodes.LCONST_0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   394
                methodVisitor.visitInsn(Opcodes.LCMP);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   395
                methodVisitor.visitJumpInsn(Opcodes.IFNE, durationEvent);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   396
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   397
                methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   398
                methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_START_TIME, "J");
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   399
                methodVisitor.visitJumpInsn(Opcodes.GOTO, writeToHandler);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   400
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   401
                // new
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   402
                // if (duration == 0) {
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   403
                methodVisitor.visitLabel(durationEvent);
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   404
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   405
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   406
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_DURATION, "J");
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   407
                methodVisitor.visitInsn(Opcodes.LCONST_0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   408
                methodVisitor.visitInsn(Opcodes.LCMP);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   409
                methodVisitor.visitJumpInsn(Opcodes.IFNE, timestampOnCommit);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   410
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   411
                // startTime = EventWriter.timestamp() - duration;
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   412
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   413
                methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   414
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   415
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_DURATION, "J");
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   416
                methodVisitor.visitInsn(Opcodes.LSUB);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   417
                methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_START_TIME, "J");
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   418
                methodVisitor.visitJumpInsn(Opcodes.GOTO, writeToHandler);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   419
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   420
                // duration = EventWriter.timestamp() -timestamp
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   421
                // }
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   422
                methodVisitor.visitLabel(timestampOnCommit);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   423
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   424
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   425
                methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   426
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   427
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   428
                methodVisitor.visitInsn(Opcodes.LSUB);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   429
                methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_DURATION, "J");
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   430
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   431
                // if (shouldCommit()) {
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   432
                 methodVisitor.visitLabel(writeToHandler);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   433
                 methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   434
                 methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   435
                 methodVisitor.visitMethodInsn(Opcodes.INVOKESPECIAL, getInternalClassName(), METHOD_EVENT_SHOULD_COMMIT.getName(), METHOD_EVENT_SHOULD_COMMIT.getDescriptor(), false);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   436
                 methodVisitor.visitJumpInsn(Opcodes.IFEQ, end);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   437
                //
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   438
                //   eventHandlerX.write(...);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   439
                getEventHandler(methodVisitor);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   440
                methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, eventHandlerXInternalName);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   441
                for (FieldInfo fi : fieldInfos) {
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   442
                   methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   443
                   methodVisitor.visitFieldInsn(Opcodes.GETFIELD, fi.internalClassName, fi.fieldName, fi.fieldDescriptor);
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   444
                }
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   445
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, eventHandlerXInternalName, writeMethod.getName(), writeMethod.getDescriptor(), false);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   446
                // }
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   447
                methodVisitor.visitLabel(end);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   448
                methodVisitor.visitInsn(Opcodes.RETURN);
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   449
                methodVisitor.visitEnd();
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   450
            });
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   451
        }
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   452
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   453
        if (!streamingCommit) {
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   454
            updateMethod(METHOD_COMMIT, methodVisitor -> {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   455
                // if (!isEnable()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   456
                // return;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   457
                // }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   458
                methodVisitor.visitCode();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   459
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   460
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), METHOD_IS_ENABLED.getName(), METHOD_IS_ENABLED.getDescriptor(), false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   461
                Label l0 = new Label();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   462
                methodVisitor.visitJumpInsn(Opcodes.IFNE, l0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   463
                methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   464
                methodVisitor.visitLabel(l0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   465
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   466
                // if (startTime == 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   467
                // startTime = EventWriter.timestamp();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   468
                // } else {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   469
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   470
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   471
                methodVisitor.visitInsn(Opcodes.LCONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   472
                methodVisitor.visitInsn(Opcodes.LCMP);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   473
                Label durationalEvent = new Label();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   474
                methodVisitor.visitJumpInsn(Opcodes.IFNE, durationalEvent);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   475
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   476
                methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   477
                methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   478
                Label commit = new Label();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   479
                methodVisitor.visitJumpInsn(Opcodes.GOTO, commit);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   480
                // if (duration == 0) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   481
                // duration = EventWriter.timestamp() - startTime;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   482
                // }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   483
                // }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   484
                methodVisitor.visitLabel(durationalEvent);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   485
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   486
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   487
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_DURATION, "J");
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   488
                methodVisitor.visitInsn(Opcodes.LCONST_0);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   489
                methodVisitor.visitInsn(Opcodes.LCMP);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   490
                methodVisitor.visitJumpInsn(Opcodes.IFNE, commit);
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   491
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   492
                methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   493
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   494
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   495
                methodVisitor.visitInsn(Opcodes.LSUB);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   496
                methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_DURATION, "J");
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   497
                methodVisitor.visitLabel(commit);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   498
                // if (shouldCommit()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   499
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   500
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   501
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), METHOD_EVENT_SHOULD_COMMIT.getName(), METHOD_EVENT_SHOULD_COMMIT.getDescriptor(), false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   502
                Label end = new Label();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   503
                // eventHandler.write(...);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   504
                // }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   505
                methodVisitor.visitJumpInsn(Opcodes.IFEQ, end);
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   506
                getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   507
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   508
                methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, eventHandlerXInternalName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   509
                for (FieldInfo fi : fieldInfos) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   510
                    methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   511
                    methodVisitor.visitFieldInsn(Opcodes.GETFIELD, fi.internalClassName, fi.fieldName, fi.fieldDescriptor);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   512
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   513
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   514
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, eventHandlerXInternalName, writeMethod.getName(), writeMethod.getDescriptor(), false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   515
                methodVisitor.visitLabel(end);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   516
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   517
                methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   518
                methodVisitor.visitEnd();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   519
            });
57641
5fb8ececb9e6 Adapt Java event commit() method for streaming
egahlin
parents: 52698
diff changeset
   520
        }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   521
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   522
        // MyEvent#shouldCommit()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   523
        updateMethod(METHOD_EVENT_SHOULD_COMMIT, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   524
            Label fail = new Label();
52698
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   525
            if (guardHandlerReference) {
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   526
                getEventHandler(methodVisitor);
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   527
                methodVisitor.visitJumpInsn(Opcodes.IFNULL, fail);
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   528
            }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   529
            // if (!eventHandler.shouldCommit(duration) goto fail;
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   530
            getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   531
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   532
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_DURATION, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   533
            ASMToolkit.invokeVirtual(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_EVENT_HANDLER_SHOULD_COMMIT);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   534
            methodVisitor.visitJumpInsn(Opcodes.IFEQ, fail);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   535
            for (SettingInfo si : settingInfos) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   536
                // if (!settingsMethod(eventHandler.settingX)) goto fail;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   537
                methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   538
                if (untypedEventHandler) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   539
                    methodVisitor.visitFieldInsn(Opcodes.GETSTATIC, getInternalClassName(), FIELD_EVENT_HANDLER, TYPE_OBJECT.getDescriptor());
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   540
                } else {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   541
                    methodVisitor.visitFieldInsn(Opcodes.GETSTATIC, getInternalClassName(), FIELD_EVENT_HANDLER, Type.getDescriptor(EventHandler.class));
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   542
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   543
                methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, eventHandlerXInternalName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   544
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, eventHandlerXInternalName, si.fieldName, TYPE_SETTING_CONTROL.getDescriptor());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   545
                methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, si.internalSettingName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   546
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), si.methodName, "(" + si.settingDescriptor + ")Z", false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   547
                methodVisitor.visitJumpInsn(Opcodes.IFEQ, fail);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   548
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   549
            // return true
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   550
            methodVisitor.visitInsn(Opcodes.ICONST_1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   551
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   552
            // return false
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   553
            methodVisitor.visitLabel(fail);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   554
            methodVisitor.visitInsn(Opcodes.ICONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   555
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   556
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   557
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   558
57848
bd961c64ecce Take timestamp on commit
egahlin
parents: 57781
diff changeset
   559
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   560
    private void getEventHandler(MethodVisitor methodVisitor) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   561
        if (untypedEventHandler) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   562
            methodVisitor.visitFieldInsn(Opcodes.GETSTATIC, getInternalClassName(), FIELD_EVENT_HANDLER, TYPE_OBJECT.getDescriptor());
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   563
            methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, TYPE_EVENT_HANDLER.getInternalName());
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   564
        } else {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   565
            methodVisitor.visitFieldInsn(Opcodes.GETSTATIC, getInternalClassName(), FIELD_EVENT_HANDLER, Type.getDescriptor(EventHandler.class));
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   566
        }
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   567
    }
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   568
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   569
    private void makeUninstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   570
        updateExistingWithReturnFalse(METHOD_EVENT_SHOULD_COMMIT);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   571
        updateExistingWithReturnFalse(METHOD_IS_ENABLED);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   572
        updateExistingWithEmptyVoidMethod(METHOD_COMMIT);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   573
        updateExistingWithEmptyVoidMethod(METHOD_BEGIN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   574
        updateExistingWithEmptyVoidMethod(METHOD_END);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   575
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   576
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   577
    private final void updateExistingWithEmptyVoidMethod(Method voidMethod) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   578
        updateMethod(voidMethod, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   579
            methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   580
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   581
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   582
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   583
    private final void updateExistingWithReturnFalse(Method voidMethod) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   584
        updateMethod(voidMethod, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   585
            methodVisitor.visitInsn(Opcodes.ICONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   586
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   587
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   588
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   589
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   590
    private MethodNode getMethodNode(Method method) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   591
        for (MethodNode m : classNode.methods) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   592
            if (m.name.equals(method.getName()) && m.desc.equals(method.getDescriptor())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   593
                return m;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   594
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   595
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   596
        return null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   597
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   598
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   599
    private final void updateMethod(Method method, Consumer<MethodVisitor> code) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   600
        MethodNode old = getMethodNode(method);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   601
        int index = classNode.methods.indexOf(old);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   602
        classNode.methods.remove(old);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   603
        MethodVisitor mv = classNode.visitMethod(old.access, old.name, old.desc, null, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   604
        mv.visitCode();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   605
        code.accept(mv);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   606
        mv.visitMaxs(0, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   607
        MethodNode newMethod = getMethodNode(method);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   608
        classNode.methods.remove(newMethod);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   609
        classNode.methods.add(index, newMethod);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   610
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   611
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   612
    public static Method makeWriteMethod(List<FieldInfo> fields) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   613
        StringBuilder sb = new StringBuilder();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   614
        sb.append("(");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   615
        for (FieldInfo v : fields) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   616
            sb.append(v.fieldDescriptor);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   617
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   618
        sb.append(")V");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   619
        return new Method("write", sb.toString());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   620
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   621
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   622
    private String getInternalClassName() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   623
        return classNode.name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   624
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   625
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   626
    public List<SettingInfo> getSettingInfos() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   627
        return settingInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   628
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   629
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   630
    public List<FieldInfo> getFieldInfos() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   631
        return fieldInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   632
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   633
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   634
    public String getEventName() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   635
        return eventName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   636
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   637
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   638
    public void setGuardHandler(boolean guardHandlerReference) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   639
        this.guardHandlerReference = guardHandlerReference;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   640
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   641
}