Skip to content

8373023: [REDO] Remove the default value of InitialRAMPercentage #28641

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jsikstro wants to merge 2 commits into from
+14 −4
Closed

8373023: [REDO] Remove the default value of InitialRAMPercentage #28641

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
d221b45
8373023: [REDO] Remove the default value of InitialRAMPercentage
jsikstro Nov 18, 2025
6a3510b
Remove unnecessary line
jsikstro Dec 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/hotspot/share/gc/shared/gc_globals.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -291,7 +291,7 @@
"size on systems with small physical memory size") \
range(0.0, 100.0) \
\
product(double, InitialRAMPercentage, 0.2, \
product(double, InitialRAMPercentage, 0.0, \
"Percentage of real memory used for initial heap size") \
range(0.0, 100.0) \
\
Expand Down
2 changes: 1 addition & 1 deletion src/hotspot/share/runtime/flags/jvmFlag.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,7 @@ void JVMFlag::print_on(outputStream* st, bool withComments, bool printRanges) co
// uintx ThresholdTolerance = 10 {product} {default}
// size_t TLABSize = 0 {product} {default}
// uintx SurvivorRatio = 8 {product} {default}
// double InitialRAMPercentage = 1.562500 {product} {default}
// double InitialRAMPercentage = 0.000000 {product} {default}
// ccstr CompileCommandFile = MyFile.cmd {product} {command line}
// ccstrlist CompileOnly = Method1
// CompileOnly += Method2 {product} {command line}
Expand Down
3 changes: 1 addition & 2 deletions src/java.base/share/man/java.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2455,8 +2455,7 @@ Java HotSpot VM.
`-XX:InitialRAMPercentage=`*percent*
: Sets the initial amount of memory that the JVM will use for the Java heap
before applying ergonomics heuristics as a percentage of the maximum amount
determined as described in the `-XX:MaxRAM` option. The default value is
0.2 percent.
determined as described in the `-XX:MaxRAM` option.

The following example shows how to set the percentage of the initial
amount of memory used for the Java heap:
Expand Down
10 changes: 10 additions & 0 deletions test/hotspot/jtreg/ProblemList.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ gc/shenandoah/TestEvilSyncBug.java#generational 8345501 generic-all
gc/shenandoah/TestRetainObjects.java#no-tlab 8361099 generic-all
gc/shenandoah/TestSieveObjects.java#no-tlab 8361099 generic-all
gc/shenandoah/TestSieveObjects.java#no-tlab-genshen 8361099 generic-all
gc/cslocker/TestCSLocker.java 8373025 generic-all

#############################################################################

Expand Down Expand Up @@ -139,6 +140,9 @@ serviceability/sa/ClhsdbPstack.java#core 8318754 macosx-aarch64
serviceability/sa/TestJmapCore.java 8318754 macosx-aarch64
serviceability/sa/TestJmapCoreMetaspace.java 8318754 macosx-aarch64

serviceability/sa/ClhsdbScanOops.java#parallel 8373022 generic-all
serviceability/sa/ClhsdbScanOops.java#serial 8373022 generic-all

serviceability/sa/ClhsdbThreadContext.java 8356704 windows-x64

serviceability/jvmti/stress/StackTrace/NotSuspended/GetStackTraceNotSuspendedStressTest.java 8315980 linux-all,windows-x64
Expand Down Expand Up @@ -183,3 +187,9 @@ vmTestbase/nsk/jdwp/ThreadReference/ForceEarlyReturn/forceEarlyReturn001/forceEa
vmTestbase/nsk/monitoring/ThreadMXBean/ThreadInfo/Multi/Multi005/TestDescription.java 8076494 windows-x64

vmTestbase/nsk/monitoring/ThreadMXBean/findMonitorDeadlockedThreads/find006/TestDescription.java 8310144 macosx-aarch64

vmTestbase/nsk/monitoring/MemoryPoolMBean/isUsageThresholdExceeded/isexceeded001/TestDescription.java 8373022 generic-all
vmTestbase/nsk/monitoring/MemoryPoolMBean/isUsageThresholdExceeded/isexceeded002/TestDescription.java 8373022 generic-all
vmTestbase/nsk/monitoring/MemoryPoolMBean/isUsageThresholdExceeded/isexceeded003/TestDescription.java 8373022 generic-all
vmTestbase/nsk/monitoring/MemoryPoolMBean/isUsageThresholdExceeded/isexceeded004/TestDescription.java 8373022 generic-all
vmTestbase/nsk/monitoring/MemoryPoolMBean/isUsageThresholdExceeded/isexceeded005/TestDescription.java 8373022 generic-all
1 change: 1 addition & 0 deletions test/jdk/ProblemList.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -715,6 +715,7 @@ javax/swing/plaf/synth/7158712/bug7158712.java 8324782 macosx-all
# jdk_jdi

com/sun/jdi/InvokeHangTest.java 8218463 linux-all
com/sun/jdi/MethodInvokeWithTraceOnTest.java 8373022 generic-all
Copy link
Contributor

@AlanBateman AlanBateman Dec 3, 2025

Choose a reason for hiding this comment

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

What about java/lang/management/MemoryPoolMXBean/ThresholdTest.java, that was the test that started failing for me yesterday.

Copy link
Member Author

@jsikstro jsikstro Dec 3, 2025

Choose a reason for hiding this comment

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

The test failure in java/lang/management/MemoryPoolMXBean/ThresholdTest.java is fixed in #28630

This was a pre-existing issue that is not directly tied to this change.

Copy link
Contributor

@AlanBateman AlanBateman Dec 3, 2025

Choose a reason for hiding this comment

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

Thanks, I didn't see that change go by.


############################################################################

Expand Down