RFR: 8281455: Change JVM options with small ranges from 64 to 32 bits, for gc_globals.hpp [v3]

Coleen Phillimore coleenp at openjdk.org
Fri Jun 9 17:41:47 UTC 2023


On Fri, 9 Jun 2023 14:36:29 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/gc/shared/gc_globals.hpp line 395:
>> 
>>> 393:   product(uint, PausePadding, 1,                                            \
>>> 394:           "How much buffer to keep for pause time")                         \
>>> 395:           range(0, UINT_MAX)                                                \
>> 
>> Why did you change from max_juint -> UINT_MAX ?  max_juint should have compile without integer size mismatch.
>
> Or, not in this change, but should we prefer UINT_MAX (found in /usr/include/limits.h) to our globalDefinitions.hpp definitions?

Now I see Thomas's comments about this.  I guess changing a couple at a time would be ok.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14259#discussion_r1224583583


More information about the hotspot-gc-dev mailing list