8326590: Improve description of MarkStackSize[Max] flags

Thomas Schatzl · Thomas Schatzl · commit eb4b6fa62123 · 2024-02-28T10:42:44.000Z
Reviewed-by: iwalulya
diff --git a/src/hotspot/share/gc/shared/gc_globals.hpp b/src/hotspot/share/gc/shared/gc_globals.hpp
@@ -213,13 +213,13 @@
                                                                             \
   /* where does the range max value of (max_jint - 1) come from? */         \
   product(size_t, MarkStackSizeMax, NOT_LP64(4*M) LP64_ONLY(512*M),         \
-          "Maximum size of marking stack")                                  \
-          range(1, (INT_MAX - 1))                                          \
+          "Maximum size of marking stack in bytes.")                        \
+          range(1, (INT_MAX - 1))                                           \
                                                                             \
   product(size_t, MarkStackSize, NOT_LP64(64*K) LP64_ONLY(4*M),             \
-          "Size of marking stack")                                          \
+          "Size of marking stack in bytes.")                                \
           constraint(MarkStackSizeConstraintFunc,AfterErgo)                 \
-          range(1, (INT_MAX - 1))                                          \
+          range(1, (INT_MAX - 1))                                           \
                                                                             \
   product(bool, ParallelRefProcEnabled, false,                              \
           "Enable parallel reference processing whenever possible")         \

Original file line number	Diff line number	Diff line change
`@@ -213,13 +213,13 @@`
`213`	`213`	`\`
`214`	`214`	`/* where does the range max value of (max_jint - 1) come from? */ \`
`215`	`215`	`product(size_t, MarkStackSizeMax, NOT_LP64(4M) LP64_ONLY(512M), \`
`216`		`- "Maximum size of marking stack") \`
`217`		`- range(1, (INT_MAX - 1)) \`
	`216`	`+ "Maximum size of marking stack in bytes.") \`
	`217`	`+ range(1, (INT_MAX - 1)) \`
`218`	`218`	`\`
`219`	`219`	`product(size_t, MarkStackSize, NOT_LP64(64K) LP64_ONLY(4M), \`
`220`		`- "Size of marking stack") \`
	`220`	`+ "Size of marking stack in bytes.") \`
`221`	`221`	`constraint(MarkStackSizeConstraintFunc,AfterErgo) \`
`222`		`- range(1, (INT_MAX - 1)) \`
	`222`	`+ range(1, (INT_MAX - 1)) \`
`223`	`223`	`\`
`224`	`224`	`product(bool, ParallelRefProcEnabled, false, \`
`225`	`225`	`"Enable parallel reference processing whenever possible") \`