8337493: [JVMCI] Number of libgraal threads might be too low

rmosaner · Doug Simon · commit 7cc7c080b5db · 2024-10-01T11:46:13.000Z
Reviewed-by: dnsimon
diff --git a/src/hotspot/share/jvmci/jvmci_globals.hpp b/src/hotspot/share/jvmci/jvmci_globals.hpp
@@ -143,9 +143,11 @@ class fileStream;
           "on the HotSpot heap. Defaults to true if UseJVMCICompiler or "   \
           "EnableJVMCI is true and a JVMCI native library is available.")   \
                                                                             \
-  product(double, JVMCINativeLibraryThreadFraction, 0.33, EXPERIMENTAL,     \
+  product(double, JVMCINativeLibraryThreadFraction, 0.66, EXPERIMENTAL,     \
           "The fraction of compiler threads used by libjvmci. "             \
-          "The remaining compiler threads are used by C1.")                 \
+          "The remaining compiler threads are used by C1. "                 \
+          "Reducing this value could reduce the max RSS but "               \
+          "also increase the warmup time.")                                 \
           range(0.0, 1.0)                                                   \
                                                                             \
   product(ccstr, JVMCINativeLibraryErrorFile, nullptr, EXPERIMENTAL,        \

Original file line number	Diff line number	Diff line change
`@@ -143,9 +143,11 @@ class fileStream;`
`143`	`143`	`"on the HotSpot heap. Defaults to true if UseJVMCICompiler or " \`
`144`	`144`	`"EnableJVMCI is true and a JVMCI native library is available.") \`
`145`	`145`	`\`
`146`		`- product(double, JVMCINativeLibraryThreadFraction, 0.33, EXPERIMENTAL, \`
	`146`	`+ product(double, JVMCINativeLibraryThreadFraction, 0.66, EXPERIMENTAL, \`
`147`	`147`	`"The fraction of compiler threads used by libjvmci. " \`
`148`		`- "The remaining compiler threads are used by C1.") \`
	`148`	`+ "The remaining compiler threads are used by C1. " \`
	`149`	`+ "Reducing this value could reduce the max RSS but " \`
	`150`	`+ "also increase the warmup time.") \`
`149`	`151`	`range(0.0, 1.0) \`
`150`	`152`	`\`
`151`	`153`	`product(ccstr, JVMCINativeLibraryErrorFile, nullptr, EXPERIMENTAL, \`