Skip to content

Commit c0dce25

Browse files
committed
8236917: TestInstanceKlassSize.java fails with "The size computed by SA for java.lang.Object does not match"
Use getTestJavaOpts() instead of getVmOptions() because of mach5 configuration settings. Reviewed-by: lmesnik, cjplummer, dholmes
1 parent 7707e32 commit c0dce25

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

test/hotspot/jtreg/serviceability/sa/TestInstanceKlassSize.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ private static void startMeWithArgs() throws Exception {
7575
try {
7676
List<String> vmArgs = new ArrayList<String>();
7777
vmArgs.add("-XX:+UsePerfData");
78-
vmArgs.addAll(Utils.getVmOptions());
78+
vmArgs.addAll(Arrays.asList(Utils.getTestJavaOpts()));
7979
app = LingeredApp.startApp(vmArgs);
8080
System.out.println ("Started LingeredApp with pid " + app.getPid());
8181
} catch (Exception ex) {

test/hotspot/jtreg/serviceability/sa/TestInstanceKlassSizeForInterface.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ private static void createAnotherToAttach(
116116
Integer.toString(lingeredAppPid)
117117
};
118118

119-
// Start a new process to attach to the LingeredApp process
119+
// Start a new process to attach to the LingeredApp process to get SA info
120120
ProcessBuilder processBuilder = ProcessTools
121121
.createJavaProcessBuilder(toolArgs);
122122
OutputAnalyzer SAOutput = ProcessTools.executeProcess(processBuilder);
@@ -150,9 +150,7 @@ public static void main (String... args) throws Exception {
150150

151151
if (args == null || args.length == 0) {
152152
try {
153-
List<String> vmArgs = new ArrayList<String>();
154-
vmArgs.addAll(Utils.getVmOptions());
155-
153+
List<String> vmArgs = Arrays.asList(Utils.getTestJavaOpts());
156154
theApp = new LingeredAppWithInterface();
157155
LingeredApp.startApp(vmArgs, theApp);
158156
createAnotherToAttach(instanceKlassNames,

0 commit comments

Comments
 (0)