src/jdk.jfr/share/classes/jdk/jfr/internal/EventInstrumentation.java
author mgronlun
Wed, 30 Oct 2019 19:43:52 +0100
changeset 58863 c16ac7a2eba4
parent 52698 ca6b58b8ffc5
permissions -rw-r--r--
8226511: Implement JFR Event Streaming Reviewed-by: egahlin, mseledtsov, mgronlun Contributed-by: erik.gahlin@oracle.com, mikhailo.seledtsov@oracle.com, markus.gronlund@oracle.com
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
     2
 * Copyright (c) 2016, 2019, Oracle and/or its affiliates. All rights reserved.
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     4
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    10
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    15
 * accompanied this code).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    16
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    20
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    23
 * questions.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    24
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    25
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    26
package jdk.jfr.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;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
    EventInstrumentation(Class<?> superClass, byte[] bytes, long id) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        this.superClass = superClass;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
        this.classNode = createClassNode(bytes);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
        this.settingInfos = buildSettingInfos(superClass, classNode);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
        this.fieldInfos = buildFieldInfos(superClass, classNode);
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   130
        this.untypedEventHandler = hasUntypedHandler();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
        this.writeMethod = makeWriteMethod(fieldInfos);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
        this.eventHandlerXInternalName = ASMToolkit.getInternalName(EventHandlerCreator.makeEventHandlerName(id));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
        String n =  annotationValue(classNode, ANNOTATION_TYPE_NAME.getDescriptor(), String.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
        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
   135
    }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   137
    private boolean hasUntypedHandler() {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   138
        for (FieldNode field : classNode.fields) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   139
            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
   140
                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
   141
            }
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
        throw new InternalError("Class missing handler field");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
    public String getClassName() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
      return classNode.name.replace("/",".");
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
    private ClassNode createClassNode(byte[] bytes) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
        ClassNode classNode = new ClassNode();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
        ClassReader classReader = new ClassReader(bytes);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
        classReader.accept(classNode, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
        return classNode;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
    boolean isRegistered() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
        Boolean result = annotationValue(classNode, ANNOTATION_TYPE_REGISTERED.getDescriptor(), Boolean.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
        if (result != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
            return result.booleanValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
        if (superClass != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
            Registered r = superClass.getAnnotation(Registered.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
            if (r != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
                return r.value();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
        return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
    boolean isEnabled() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
        Boolean result = annotationValue(classNode, ANNOTATION_TYPE_ENABLED.getDescriptor(), Boolean.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
        if (result != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
            return result.booleanValue();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
        if (superClass != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
            Enabled e = superClass.getAnnotation(Enabled.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
            if (e != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
                return e.value();
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
        return true;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
    @SuppressWarnings("unchecked")
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
    private static <T> T annotationValue(ClassNode classNode, String typeDescriptor, Class<?> type) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
        if (classNode.visibleAnnotations != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
            for (AnnotationNode a : classNode.visibleAnnotations) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
                if (typeDescriptor.equals(a.desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
                    List<Object> values = a.values;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
                    if (values != null && values.size() == 2) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
                        Object key = values.get(0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
                        Object value = values.get(1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
                        if (key instanceof String && value != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
                            if (type == value.getClass()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
                                String keyName = (String) key;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
                                if ("value".equals(keyName)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
                                   return (T) value;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
                                }
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
        return null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
    private static List<SettingInfo> buildSettingInfos(Class<?> superClass, ClassNode classNode) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
        Set<String> methodSet = new HashSet<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
        List<SettingInfo> settingInfos = new ArrayList<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
        String settingDescriptor = Type.getType(SettingDefinition.class).getDescriptor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
        for (MethodNode m : classNode.methods) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
            if (m.visibleAnnotations != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
                for (AnnotationNode an : m.visibleAnnotations) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
                    // We can't really validate the method at this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
                    // stage. We would need to check that the parameter
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
                    // is an instance of SettingControl.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
                    if (settingDescriptor.equals(an.desc)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
                        Type returnType = Type.getReturnType(m.desc);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
                        if (returnType.equals(Type.getType(Boolean.TYPE))) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
                            Type[] args = Type.getArgumentTypes(m.desc);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
                            if (args.length == 1) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
                                Type paramType = args[0];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
                                String fieldName = EventControl.FIELD_SETTING_PREFIX + settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
                                int index = settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
                                SettingInfo si = new SettingInfo(fieldName, index);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
                                si.methodName = m.name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
                                si.settingDescriptor = paramType.getDescriptor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
                                si.internalSettingName = paramType.getInternalName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
                                methodSet.add(m.name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
                                settingInfos.add(si);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
                            }
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
        }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   239
        for (Class<?> c = superClass; c != jdk.internal.event.Event.class; c = c.getSuperclass()) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
            for (java.lang.reflect.Method method : c.getDeclaredMethods()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
                if (!methodSet.contains(method.getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
                    // skip private method in base classes
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
                    if (!Modifier.isPrivate(method.getModifiers())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
                        if (method.getReturnType().equals(Boolean.TYPE)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
                            if (method.getParameterCount() == 1) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
                                Parameter param = method.getParameters()[0];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
                                Type paramType = Type.getType(param.getType());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
                                String fieldName = EventControl.FIELD_SETTING_PREFIX + settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
                                int index = settingInfos.size();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
                                SettingInfo si = new SettingInfo(fieldName, index);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
                                si.methodName = method.getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
                                si.settingDescriptor = paramType.getDescriptor();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
                                si.internalSettingName = paramType.getInternalName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
                                methodSet.add(method.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
                                settingInfos.add(si);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
                            }
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
        return settingInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
    private static List<FieldInfo> buildFieldInfos(Class<?> superClass, ClassNode classNode) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
        Set<String> fieldSet = new HashSet<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
        List<FieldInfo> fieldInfos = new ArrayList<>(classNode.fields.size());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
        // These two field are added by native as transient so they will be
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
        // ignored by the loop below.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
        // The benefit of adding them manually is that we can
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
        // control in which order they occur and we can add @Name, @Description
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
        // in Java, instead of in native. It also means code for adding implicit
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
        // fields for native can be reused by Java.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        fieldInfos.add(new FieldInfo("startTime", Type.LONG_TYPE.getDescriptor(), classNode.name));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   275
        fieldInfos.add(new FieldInfo("duration", Type.LONG_TYPE.getDescriptor(), classNode.name));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
        for (FieldNode field : classNode.fields) {
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   277
            if (!fieldSet.contains(field.name) && isValidField(field.access, Type.getType(field.desc).getClassName())) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   278
                FieldInfo fi = new FieldInfo(field.name, field.desc, classNode.name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   279
                fieldInfos.add(fi);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
                fieldSet.add(field.name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
        }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   283
        for (Class<?> c = superClass; c != jdk.internal.event.Event.class; c = c.getSuperclass()) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   284
            for (Field field : c.getDeclaredFields()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   285
                // skip private field in base classes
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   286
                if (!Modifier.isPrivate(field.getModifiers())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   287
                    if (isValidField(field.getModifiers(), field.getType().getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   288
                        String fieldName = field.getName();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   289
                        if (!fieldSet.contains(fieldName)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
                            Type fieldType = Type.getType(field.getType());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
                            String internalClassName = ASMToolkit.getInternalName(c.getName());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   292
                            fieldInfos.add(new FieldInfo(fieldName, fieldType.getDescriptor(), internalClassName));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   293
                            fieldSet.add(fieldName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
                        }
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
        return fieldInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   300
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
    public static boolean isValidField(int access, String className) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
        if (Modifier.isTransient(access) || Modifier.isStatic(access)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
            return false;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
        return jdk.jfr.internal.Type.isValidJavaFieldType(className);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
    public byte[] buildInstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
        makeInstrumented();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
        return toByteArray();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
    private byte[] toByteArray() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   315
        ClassWriter cw = new ClassWriter(ClassWriter.COMPUTE_FRAMES);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
        classNode.accept(cw);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
        cw.visitEnd();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
        byte[] result = cw.toByteArray();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   319
        Utils.writeGeneratedASM(classNode.name, result);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   320
        return result;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   321
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   322
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   323
    public byte[] builUninstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   324
        makeUninstrumented();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   325
        return toByteArray();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   326
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   327
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   328
    private void makeInstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   329
        // MyEvent#isEnabled()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   330
        updateMethod(METHOD_IS_ENABLED, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   331
            Label nullLabel = new Label();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   332
            if (guardHandlerReference) {
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   333
                getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   334
                methodVisitor.visitJumpInsn(Opcodes.IFNULL, nullLabel);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   335
            }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   336
            getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   337
            ASMToolkit.invokeVirtual(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_IS_ENABLED);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   338
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   339
            if (guardHandlerReference) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   340
                methodVisitor.visitLabel(nullLabel);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   341
                methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   342
                methodVisitor.visitInsn(Opcodes.ICONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   343
                methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   344
            }
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
        // MyEvent#begin()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   348
        updateMethod(METHOD_BEGIN, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   349
            methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   350
            ASMToolkit.invokeStatic(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   351
            methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   352
            methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   353
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   354
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   355
        // MyEvent#end()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   356
        updateMethod(METHOD_END, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   357
            methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   358
            methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   359
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   360
            ASMToolkit.invokeStatic(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_DURATION);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   361
            methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_DURATION, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   362
            methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   363
            methodVisitor.visitMaxs(0, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   364
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   365
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   366
        updateMethod(METHOD_COMMIT, methodVisitor -> {
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   367
            // if (!isEnable()) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   368
            // return;
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   369
            // }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   370
            methodVisitor.visitCode();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   371
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   372
            methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), METHOD_IS_ENABLED.getName(), METHOD_IS_ENABLED.getDescriptor(), false);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   373
            Label l0 = new Label();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   374
            methodVisitor.visitJumpInsn(Opcodes.IFNE, l0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   375
            methodVisitor.visitInsn(Opcodes.RETURN);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   376
            methodVisitor.visitLabel(l0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   377
            methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   378
            // if (startTime == 0) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   379
            // startTime = EventWriter.timestamp();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   380
            // } else {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   381
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   382
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   383
            methodVisitor.visitInsn(Opcodes.LCONST_0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   384
            methodVisitor.visitInsn(Opcodes.LCMP);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   385
            Label durationalEvent = new Label();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   386
            methodVisitor.visitJumpInsn(Opcodes.IFNE, durationalEvent);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   387
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   388
            methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   389
            methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_START_TIME, "J");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   390
            Label commit = new Label();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   391
            methodVisitor.visitJumpInsn(Opcodes.GOTO, commit);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   392
            // if (duration == 0) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   393
            // duration = EventWriter.timestamp() - startTime;
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   394
            // }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   395
            // }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   396
            methodVisitor.visitLabel(durationalEvent);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   397
            methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   398
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   399
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_DURATION, "J");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   400
            methodVisitor.visitInsn(Opcodes.LCONST_0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   401
            methodVisitor.visitInsn(Opcodes.LCMP);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   402
            methodVisitor.visitJumpInsn(Opcodes.IFNE, commit);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   403
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   404
            methodVisitor.visitMethodInsn(Opcodes.INVOKESTATIC, TYPE_EVENT_HANDLER.getInternalName(), METHOD_TIME_STAMP.getName(), METHOD_TIME_STAMP.getDescriptor(), false);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   405
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   406
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_START_TIME, "J");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   407
            methodVisitor.visitInsn(Opcodes.LSUB);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   408
            methodVisitor.visitFieldInsn(Opcodes.PUTFIELD, getInternalClassName(), FIELD_DURATION, "J");
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   409
            methodVisitor.visitLabel(commit);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   410
            // if (shouldCommit()) {
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   411
            methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   412
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   413
            methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), METHOD_EVENT_SHOULD_COMMIT.getName(), METHOD_EVENT_SHOULD_COMMIT.getDescriptor(), false);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   414
            Label end = new Label();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   415
            // eventHandler.write(...);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   416
            // }
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   417
            methodVisitor.visitJumpInsn(Opcodes.IFEQ, end);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   418
            getEventHandler(methodVisitor);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   419
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   420
            methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, eventHandlerXInternalName);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   421
            for (FieldInfo fi : fieldInfos) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   422
                methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   423
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, fi.internalClassName, fi.fieldName, fi.fieldDescriptor);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   424
            }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   425
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   426
            methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, eventHandlerXInternalName, writeMethod.getName(), writeMethod.getDescriptor(), false);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   427
            methodVisitor.visitLabel(end);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   428
            methodVisitor.visitFrame(Opcodes.F_SAME, 0, null, 0, null);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   429
            methodVisitor.visitInsn(Opcodes.RETURN);
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   430
            methodVisitor.visitEnd();
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   431
        });
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   432
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   433
        // MyEvent#shouldCommit()
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   434
        updateMethod(METHOD_EVENT_SHOULD_COMMIT, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   435
            Label fail = new Label();
52698
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   436
            if (guardHandlerReference) {
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   437
                getEventHandler(methodVisitor);
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   438
                methodVisitor.visitJumpInsn(Opcodes.IFNULL, fail);
ca6b58b8ffc5 8214287: SpecJbb2005StressModule got uncaught exception
mgronlun
parents: 52334
diff changeset
   439
            }
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   440
            // if (!eventHandler.shouldCommit(duration) goto fail;
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   441
            getEventHandler(methodVisitor);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   442
            methodVisitor.visitVarInsn(Opcodes.ALOAD, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   443
            methodVisitor.visitFieldInsn(Opcodes.GETFIELD, getInternalClassName(), FIELD_DURATION, "J");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   444
            ASMToolkit.invokeVirtual(methodVisitor, TYPE_EVENT_HANDLER.getInternalName(), METHOD_EVENT_HANDLER_SHOULD_COMMIT);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   445
            methodVisitor.visitJumpInsn(Opcodes.IFEQ, fail);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   446
            for (SettingInfo si : settingInfos) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   447
                // if (!settingsMethod(eventHandler.settingX)) goto fail;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   448
                methodVisitor.visitIntInsn(Opcodes.ALOAD, 0);
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   449
                if (untypedEventHandler) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   450
                    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
   451
                } else {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   452
                    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
   453
                }
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   454
                methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, eventHandlerXInternalName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   455
                methodVisitor.visitFieldInsn(Opcodes.GETFIELD, eventHandlerXInternalName, si.fieldName, TYPE_SETTING_CONTROL.getDescriptor());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   456
                methodVisitor.visitTypeInsn(Opcodes.CHECKCAST, si.internalSettingName);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   457
                methodVisitor.visitMethodInsn(Opcodes.INVOKEVIRTUAL, getInternalClassName(), si.methodName, "(" + si.settingDescriptor + ")Z", false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   458
                methodVisitor.visitJumpInsn(Opcodes.IFEQ, fail);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   459
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   460
            // return true
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   461
            methodVisitor.visitInsn(Opcodes.ICONST_1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   462
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   463
            // return false
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   464
            methodVisitor.visitLabel(fail);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   465
            methodVisitor.visitInsn(Opcodes.ICONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   466
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   467
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   468
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   469
58863
c16ac7a2eba4 8226511: Implement JFR Event Streaming
mgronlun
parents: 52698
diff changeset
   470
52334
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   471
    private void getEventHandler(MethodVisitor methodVisitor) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   472
        if (untypedEventHandler) {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   473
            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
   474
            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
   475
        } else {
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   476
            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
   477
        }
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   478
    }
a181612f0715 8203629: Produce events in the JDK without a dependency on jdk.jfr
egahlin
parents: 50113
diff changeset
   479
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   480
    private void makeUninstrumented() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   481
        updateExistingWithReturnFalse(METHOD_EVENT_SHOULD_COMMIT);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   482
        updateExistingWithReturnFalse(METHOD_IS_ENABLED);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   483
        updateExistingWithEmptyVoidMethod(METHOD_COMMIT);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   484
        updateExistingWithEmptyVoidMethod(METHOD_BEGIN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   485
        updateExistingWithEmptyVoidMethod(METHOD_END);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   486
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   487
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   488
    private final void updateExistingWithEmptyVoidMethod(Method voidMethod) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   489
        updateMethod(voidMethod, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   490
            methodVisitor.visitInsn(Opcodes.RETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   491
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   492
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   493
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   494
    private final void updateExistingWithReturnFalse(Method voidMethod) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   495
        updateMethod(voidMethod, methodVisitor -> {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   496
            methodVisitor.visitInsn(Opcodes.ICONST_0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   497
            methodVisitor.visitInsn(Opcodes.IRETURN);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   498
        });
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   499
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   500
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   501
    private MethodNode getMethodNode(Method method) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   502
        for (MethodNode m : classNode.methods) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   503
            if (m.name.equals(method.getName()) && m.desc.equals(method.getDescriptor())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   504
                return m;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   505
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   506
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   507
        return null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   508
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   509
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   510
    private final void updateMethod(Method method, Consumer<MethodVisitor> code) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   511
        MethodNode old = getMethodNode(method);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   512
        int index = classNode.methods.indexOf(old);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   513
        classNode.methods.remove(old);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   514
        MethodVisitor mv = classNode.visitMethod(old.access, old.name, old.desc, null, null);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   515
        mv.visitCode();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   516
        code.accept(mv);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   517
        mv.visitMaxs(0, 0);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   518
        MethodNode newMethod = getMethodNode(method);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   519
        classNode.methods.remove(newMethod);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   520
        classNode.methods.add(index, newMethod);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   521
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   522
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   523
    public static Method makeWriteMethod(List<FieldInfo> fields) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   524
        StringBuilder sb = new StringBuilder();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   525
        sb.append("(");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   526
        for (FieldInfo v : fields) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   527
            sb.append(v.fieldDescriptor);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   528
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   529
        sb.append(")V");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   530
        return new Method("write", sb.toString());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   531
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   532
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   533
    private String getInternalClassName() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   534
        return classNode.name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   535
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   536
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   537
    public List<SettingInfo> getSettingInfos() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   538
        return settingInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   539
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   540
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   541
    public List<FieldInfo> getFieldInfos() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   542
        return fieldInfos;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   543
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   544
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   545
    public String getEventName() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   546
        return eventName;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   547
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   548
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   549
    public void setGuardHandler(boolean guardHandlerReference) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   550
        this.guardHandlerReference = guardHandlerReference;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   551
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   552
}