RFR: 8346916: [REDO] align_up has potential overflow [v3]

[Casper Norrbin]

On Fri, 28 Feb 2025 19:29:00 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Casper Norrbin has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   changed max size of MinHeapDeltaBytes
>
> src/hotspot/share/gc/parallel/psOldGen.cpp line 193:
> 
>> 191: #endif
>> 192:   const size_t alignment = virtual_space()->alignment();
>> 193:   size_t aligned_bytes  = align_up_or_min(bytes, alignment);
> 
> How about using `bytes = MIN2(bytes, virtual_space()->uncommitted_size())` to dodge the potential overflow? I find it more intuitive to provide a proper arg to `align_up` and expect the result to be >= arg.

Changed it to this now

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23711#discussion_r1977194293