src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.hotspot.lir.test/src/org/graalvm/compiler/hotspot/lir/test/MitigateExceedingMaxOopMapStackOffsetTest.java
author chegar
Thu, 17 Oct 2019 20:54:25 +0100
branchdatagramsocketimpl-branch
changeset 58679 9c3209ff7550
parent 58678 9cf78a70fa4f
parent 58533 46b0b7fe255c
permissions -rw-r--r--
datagramsocketimpl-branch: merge with default
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     1
/*
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     2
 * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     4
 *
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     7
 * published by the Free Software Foundation.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     8
 *
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    13
 * accompanied this code).
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    14
 *
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    18
 *
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    21
 * questions.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    22
 */
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    23
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    24
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    25
package org.graalvm.compiler.hotspot.lir.test;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    26
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    27
import java.util.ArrayList;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    28
import java.util.List;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    29
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    30
import org.graalvm.compiler.core.common.LIRKind;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    31
import org.graalvm.compiler.hotspot.HotSpotBackend;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    32
import org.graalvm.compiler.lir.VirtualStackSlot;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    33
import org.graalvm.compiler.lir.framemap.FrameMapBuilder;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    34
import org.graalvm.compiler.lir.gen.LIRGeneratorTool;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    35
import org.graalvm.compiler.lir.jtt.LIRTest;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    36
import org.graalvm.compiler.lir.jtt.LIRTestSpecification;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    37
import org.graalvm.compiler.lir.stackslotalloc.LSStackSlotAllocator;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    38
import org.graalvm.compiler.nodes.SafepointNode;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    39
import org.graalvm.compiler.nodes.graphbuilderconf.GraphBuilderConfiguration;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    40
import org.graalvm.compiler.nodes.graphbuilderconf.GraphBuilderContext;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    41
import org.graalvm.compiler.nodes.graphbuilderconf.InvocationPlugin;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    42
import org.junit.Assume;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    43
import org.junit.Test;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    44
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    45
import jdk.vm.ci.meta.AllocatableValue;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    46
import jdk.vm.ci.meta.JavaConstant;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    47
import jdk.vm.ci.meta.ResolvedJavaMethod;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    48
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    49
/**
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    50
 * Tests the mitigation against overflowing the max size limit for a HotSpot OopMap. The mitigation
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    51
 * works by {@link LSStackSlotAllocator} placing reference typed stack slots at lower offsets.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    52
 */
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    53
public class MitigateExceedingMaxOopMapStackOffsetTest extends LIRTest {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    54
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    55
    /**
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    56
     * Allocate stacks slots and initializes those at an odd index with a reference constant and
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    57
     * those at an even index with a primitive constant.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    58
     */
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    59
    private static class WriteStackValues extends LIRTestSpecification {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    60
        private final JavaConstant objectConstant;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    61
        private final JavaConstant primitiveConstant;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    62
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    63
        WriteStackValues(JavaConstant objectConstant, JavaConstant primitiveConstant) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    64
            this.objectConstant = objectConstant;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    65
            this.primitiveConstant = primitiveConstant;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    66
        }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    67
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    68
        @Override
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    69
        public void generate(LIRGeneratorTool gen) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    70
            FrameMapBuilder frameMapBuilder = gen.getResult().getFrameMapBuilder();
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    71
            LIRKind objectLirKind = LIRKind.reference(gen.target().arch.getPlatformKind(objectConstant.getJavaKind()));
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    72
            LIRKind primitiveLirKind = LIRKind.value(gen.target().arch.getPlatformKind(primitiveConstant.getJavaKind()));
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    73
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    74
            int numSlots = numPrimitiveSlots + numReferenceSlots;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    75
            List<AllocatableValue> slotList = new ArrayList<>(numSlots);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    76
            // Place reference slots at top and bottom of virtual frame
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    77
            // with primitive slots in the middle. This tests that slot
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    78
            // partitioning works.
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 57537
diff changeset
    79
            AllocatableValue srcObject = gen.emitLoadConstant(objectLirKind, objectConstant);
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    80
            for (int i = 0; i < numReferenceSlots / 2; i++) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    81
                VirtualStackSlot slot = frameMapBuilder.allocateSpillSlot(objectLirKind);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    82
                slotList.add(slot);
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 57537
diff changeset
    83
                gen.emitMove(slot, srcObject);
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    84
            }
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 57537
diff changeset
    85
            AllocatableValue srcPrimitive = gen.emitLoadConstant(objectLirKind, primitiveConstant);
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    86
            for (int i = 0; i < numPrimitiveSlots; i++) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    87
                VirtualStackSlot slot = frameMapBuilder.allocateSpillSlot(primitiveLirKind);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    88
                slotList.add(slot);
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 57537
diff changeset
    89
                gen.emitMove(slot, srcPrimitive);
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    90
            }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    91
            for (int i = numReferenceSlots / 2; i < numReferenceSlots; i++) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    92
                VirtualStackSlot slot = frameMapBuilder.allocateSpillSlot(objectLirKind);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    93
                slotList.add(slot);
58533
46b0b7fe255c 8231973: Update Graal
dlong
parents: 57537
diff changeset
    94
                gen.emitMove(slot, srcObject);
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    95
            }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    96
            slots = slotList.toArray(new AllocatableValue[slotList.size()]);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    97
        }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    98
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
    99
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   100
    /**
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   101
     * Read stacks slots and move their content into a blackhole.
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   102
     */
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   103
    private static class ReadStackValues extends LIRTestSpecification {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   104
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   105
        ReadStackValues() {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   106
        }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   107
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   108
        @Override
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   109
        public void generate(LIRGeneratorTool gen) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   110
            for (int i = 0; i < slots.length; i++) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   111
                gen.emitBlackhole(gen.emitMove(slots[i]));
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   112
            }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   113
        }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   114
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   115
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   116
    @Override
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   117
    protected GraphBuilderConfiguration editGraphBuilderConfiguration(GraphBuilderConfiguration conf) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   118
        InvocationPlugin safepointPlugin = new InvocationPlugin() {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   119
            @Override
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   120
            public boolean apply(GraphBuilderContext b, ResolvedJavaMethod targetMethod, Receiver receiver) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   121
                b.add(new SafepointNode());
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   122
                return true;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   123
            }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   124
        };
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   125
        conf.getPlugins().getInvocationPlugins().register(safepointPlugin, getClass(), "safepoint");
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   126
        return super.editGraphBuilderConfiguration(conf);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   127
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   128
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   129
    /*
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   130
     * Safepoint Snippet
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   131
     */
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   132
    private static void safepoint() {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   133
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   134
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   135
    private static int numPrimitiveSlots;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   136
    private static int numReferenceSlots;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   137
    private static AllocatableValue[] slots;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   138
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   139
    private static final LIRTestSpecification readStackValues = new ReadStackValues();
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   140
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   141
    @SuppressWarnings("unused")
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   142
    @LIRIntrinsic
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   143
    public static void instrinsic(LIRTestSpecification spec) {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   144
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   145
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   146
    private static final LIRTestSpecification writeStackValues = new WriteStackValues(JavaConstant.NULL_POINTER, JavaConstant.LONG_0);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   147
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   148
    public void testStackObjects() {
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   149
        instrinsic(writeStackValues);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   150
        safepoint();
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   151
        instrinsic(readStackValues);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   152
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   153
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   154
    @Test
57537
ecc6e394475f 8226771: Update Graal
dlong
parents: 55509
diff changeset
   155
    public void runStackObjects() {
55509
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   156
        int max = ((HotSpotBackend) getBackend()).getRuntime().getVMConfig().maxOopMapStackOffset;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   157
        Assume.assumeFalse("no limit on oop map size", max == Integer.MAX_VALUE);
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   158
        numPrimitiveSlots = (max / 8) * 2;
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   159
        numReferenceSlots = (max / 8) - 100; // Should be enough margin for all platforms
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   160
        runTest("testStackObjects");
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   161
    }
d58442b8abc1 8225497: Update Graal
jwilhelm
parents:
diff changeset
   162
}