Skip to content

Commit e4c24f0

Browse files
committed
8270171: Shenandoah: Cleanup TestStringDedup and TestStringDedupStress tests
Reviewed-by: shade
1 parent 3588634 commit e4c24f0

File tree

2 files changed

+1
-20
lines changed

2 files changed

+1
-20
lines changed

test/hotspot/jtreg/gc/shenandoah/TestStringDedup.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@
4949
* @key randomness
5050
* @requires vm.gc.Shenandoah
5151
* @library /test/lib
52-
* @modules java.base/jdk.internal.misc:open
5352
* @modules java.base/java.lang:open
5453
* java.management
5554
*
@@ -72,7 +71,6 @@
7271
* @key randomness
7372
* @requires vm.gc.Shenandoah
7473
* @library /test/lib
75-
* @modules java.base/jdk.internal.misc:open
7674
* @modules java.base/java.lang:open
7775
* java.management
7876
*
@@ -89,22 +87,15 @@
8987
import java.util.*;
9088
import jdk.test.lib.Utils;
9189

92-
import sun.misc.*;
93-
9490
public class TestStringDedup {
9591
private static Field valueField;
96-
private static Unsafe unsafe;
9792

9893
private static final int UniqueStrings = 20;
9994
// How many GC cycles are needed to complete deduplication.
10095
private static final int GCCount = Integer.getInteger("GCCount", 3);
10196

10297
static {
10398
try {
104-
Field field = Unsafe.class.getDeclaredField("theUnsafe");
105-
field.setAccessible(true);
106-
unsafe = (Unsafe) field.get(null);
107-
10899
valueField = String.class.getDeclaredField("value");
109100
valueField.setAccessible(true);
110101
} catch (Exception e) {

test/hotspot/jtreg/gc/shenandoah/TestStringDedupStress.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2017, 2018, Red Hat, Inc. All rights reserved.
2+
* Copyright (c) 2017, 2021, Red Hat, Inc. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* This code is free software; you can redistribute it and/or modify it
@@ -28,7 +28,6 @@
2828
* @key randomness
2929
* @requires vm.gc.Shenandoah
3030
* @library /test/lib
31-
* @modules java.base/jdk.internal.misc:open
3231
* @modules java.base/java.lang:open
3332
* java.management
3433
*
@@ -49,7 +48,6 @@
4948
* @key randomness
5049
* @requires vm.gc.Shenandoah
5150
* @library /test/lib
52-
* @modules java.base/jdk.internal.misc:open
5351
* @modules java.base/java.lang:open
5452
* java.management
5553
*
@@ -80,7 +78,6 @@
8078
* @key randomness
8179
* @requires vm.gc.Shenandoah
8280
* @library /test/lib
83-
* @modules java.base/jdk.internal.misc:open
8481
* @modules java.base/java.lang:open
8582
* java.management
8683
*
@@ -111,11 +108,8 @@
111108
import java.util.*;
112109
import jdk.test.lib.Utils;
113110

114-
import sun.misc.*;
115-
116111
public class TestStringDedupStress {
117112
private static Field valueField;
118-
private static Unsafe unsafe;
119113

120114
private static final int TARGET_STRINGS = Integer.getInteger("targetStrings", 2_500_000);
121115
private static final long MAX_REWRITE_GC_CYCLES = 6;
@@ -125,10 +119,6 @@ public class TestStringDedupStress {
125119

126120
static {
127121
try {
128-
Field field = Unsafe.class.getDeclaredField("theUnsafe");
129-
field.setAccessible(true);
130-
unsafe = (Unsafe) field.get(null);
131-
132122
valueField = String.class.getDeclaredField("value");
133123
valueField.setAccessible(true);
134124
} catch (Exception e) {

0 commit comments

Comments
 (0)