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

Wang Zhuo wzhuo at openjdk.org
Fri Feb 2 03:28:01 UTC 2024


On Thu, 1 Feb 2024 11:26:25 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Wang Zhuo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update test
>
> test/hotspot/jtreg/gc/parallel/TestAlwaysPreTouchBehavior.java line 83:
> 
>> 81:         System.out.println("cannot get RSS, just skip");
>> 82:         return; // Did not get avaiable RSS, just ignore this test
>> 83:     }
> 
> The `EXCEPTION_VALUE` constant seems superfluous, and the marked code can be simplified to just
> 
> Suggestion:
> 
>     } catch (Exception e) {
>         System.out.println("cannot get RSS, just skip");
>         return; // Did not get avaiable RSS, just ignore this test
>     }
> 
> 
> imo.
> 
> Actually, thinking a bit further, I think there is no reason to actually try to catch anything here. If the `proc` filesystem isn't readable (or something changes in its contents) I would think it is worth looking at the environment or fixing the test.

Thanks, EXCEPTION_VALUE  related code were deleted. Test updated

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

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


More information about the hotspot-gc-dev mailing list