Skip to content

Commit

Permalink
8318964: Fix build failures caused by 8315097
Browse files Browse the repository at this point in the history
Reviewed-by: aboldtch, rcastanedalo
  • Loading branch information
lkorinth committed Oct 27, 2023
1 parent d52a995 commit b9dcd4b
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public class CompressedClassPointersEncodingScheme {
private static void test(long forceAddress, long classSpaceSize, long expectedEncodingBase, int expectedEncodingShift) throws IOException {
String forceAddressString = String.format("0x%016X", forceAddress).toLowerCase();
String expectedEncodingBaseString = String.format("0x%016X", expectedEncodingBase).toLowerCase();
ProcessBuilder pb = ProcessTools.createJavaProcessBuilder(
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(
"-Xshare:off", // to make CompressedClassSpaceBaseAddress work
"-XX:+UnlockDiagnosticVMOptions",
"-XX:-UseCompressedOops", // keep VM from optimizing heap location
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ public class TestSymbolsInHsErrFile {
public static void main(String[] args) throws Exception {

// Start a jvm and cause a SIGSEGV / ACCESS_VIOLATION
ProcessBuilder pb = ProcessTools.createJavaProcessBuilder(
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(
"-XX:+UnlockDiagnosticVMOptions",
"-Xmx100M",
"-XX:-CreateCoredumpOnCrash",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,7 @@ public static void main(String[] extraOptions) throws Exception {
boolean useTHP = allOptions.contains("-XX:+UseTransparentHugePages");
System.out.println("useLP: " + useLP + " useTHP: " + useTHP);

ProcessBuilder pb = ProcessTools.createJavaProcessBuilder(allOptions.toArray(new String[0]));
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(allOptions.toArray(new String[0]));
OutputAnalyzer output = new OutputAnalyzer(pb.start());
output.reportDiagnosticSummary();
HugePageConfiguration configuration = HugePageConfiguration.readFromOS();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ public class AppendToClassPathModuleTest {
public static void main(String... args) throws Throwable {
String modulepath = System.getProperty("test.module.path");

// can't use ProcessTools.createJavaProcessBuilder as it always adds -cp
// can't use ProcessTools.createLimitedTestJavaProcessBuilder as it always adds -cp
ProcessBuilder pb = new ProcessBuilder(
JDKToolFinder.getTestJDKTool("java"),
"-javaagent:Agent.jar",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ public void testClassPathAttribute() throws Throwable {
}

private OutputAnalyzer execute(List<String> options) throws Throwable {
// can't use ProcessTools.createJavaProcessBuilder as it always adds -cp
// can't use ProcessTools.createLimitedTestJavaProcessBuilder as it always adds -cp
ProcessBuilder pb = new ProcessBuilder(
Stream.concat(Stream.of(JDKToolFinder.getTestJDKTool("java")),
options.stream()
Expand Down

1 comment on commit b9dcd4b

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.