Skip to content

Commit

Permalink
Use LDR_CNTRL on AIX to use 64KB pages and modify removeAixExpectedVars
Browse files Browse the repository at this point in the history
Signed-off-by: Abdulrahman Alattas <rmnattas@gmail.com>
  • Loading branch information
rmnattas committed Mar 1, 2024
1 parent 687a145 commit d658f57
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
7 changes: 6 additions & 1 deletion src/java.base/unix/native/libjli/java_md.c
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@

/*
* ===========================================================================
* (c) Copyright IBM Corp. 2020, 2023 All Rights Reserved
* (c) Copyright IBM Corp. 2020, 2024 All Rights Reserved
* ===========================================================================
*/

Expand Down Expand Up @@ -312,6 +312,11 @@ CreateExecutionEnvironment(int *pargc, char ***pargv,
if (setenv(mallocOptionsName, mallocOptionsValue, 0) != 0) {
fprintf(stderr, "setenv('MALLOCOPTIONS=multiheap,considersize') failed: performance may be affected\n");
}
const char * ldrCntrlName = "LDR_CNTRL";
const char *ldrCntrlValue = "TEXTPSIZE=64K@DATAPSIZE=64K@STACKPSIZE=64K@SHMPSIZE=64K";
if (setenv(ldrCntrlName, ldrCntrlValue, 0) != 0) {
fprintf(stderr, "setenv('LDR_CNTRL=TEXTPSIZE=64K@DATAPSIZE=64K@STACKPSIZE=64K@SHMPSIZE=64K') failed: performance may be affected\n");
}
#endif

#ifdef SETENV_REQUIRED
Expand Down
5 changes: 3 additions & 2 deletions test/jdk/java/lang/ProcessBuilder/Basic.java
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@

/*
* ===========================================================================
* (c) Copyright IBM Corp. 2020, 2022 All Rights Reserved
* (c) Copyright IBM Corp. 2020, 2024 All Rights Reserved
* ===========================================================================
*/

Expand Down Expand Up @@ -811,8 +811,9 @@ private static String removeMacExpectedVars(String vars) {
*/
private static String removeAixExpectedVars(String vars) {
String cleanedVars = vars.replace("AIXTHREAD_GUARDPAGES=0,", "");
// OpenJ9 adds MALLOCOPTIONS
// OpenJ9 adds MALLOCOPTIONS and LDR_CNTRL
cleanedVars = cleanedVars.replace("MALLOCOPTIONS=multiheap,considersize,", "");
cleanedVars = cleanedVars.replace("LDR_CNTRL=TEXTPSIZE=64K@DATAPSIZE=64K@STACKPSIZE=64K@SHMPSIZE=64K,", "");
return cleanedVars;
}

Expand Down

0 comments on commit d658f57

Please sign in to comment.