RFR: 8324817: Parallel GC does not pre-touch all heap pages when AlwaysPreTouch enabled and large page disabled [v4]

Wang Zhuo wzhuo at openjdk.org
Thu Feb 1 03:37:05 UTC 2024


On Wed, 31 Jan 2024 12:38:55 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Wang Zhuo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   remove printings and move necessary information into assertion.
>
> test/hotspot/jtreg/gc/parallel/TestAlwaysPreTouchBehavior.java line 72:
> 
>> 70:         } catch (Exception e) {
>> 71:            return EXCEPTION_VALUE;
>> 72:         }
> 
> This is identical to what caller does. I believe try-catch can be omitted in this method (callee), and keep the only the try-catch in the caller.

Thanks. Redundant try-catch removed.

> test/hotspot/jtreg/gc/parallel/TestAlwaysPreTouchBehavior.java line 78:
> 
>> 76:     long rss = 0;
>> 77:     Runtime runtime = Runtime.getRuntime();
>> 78:     long committedMemory = (runtime.totalMemory()) / 1024; // in kb
> 
> The `()` seems not needed. Also, one doesn't need introduce a tmp `runtime` var.

Redundant () removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17610#discussion_r1473755417
PR Review Comment: https://git.openjdk.org/jdk/pull/17610#discussion_r1473755593


More information about the hotspot-gc-dev mailing list