RFR 8134995(M): [REDO] GC: implement ranges (optionally constraints) for those flags that have them missing
sangheon.kim
sangheon.kim at oracle.com
Fri Sep 25 16:01:06 UTC 2015
Hi Zoltan,
Thank you for reviewing this!
On 09/24/2015 11:36 PM, Zoltán Majó wrote:
> Hi Sangheon,
>
>
> I spotted a small issue in TestG1HeapRegionSize.java: You've left a
> println statement in the code (although you've commented it out).
>
> 58 //System.out.println(stdout);
Nice!
I'll include this at next webrev.
Thanks,
Sangheon
>
>
> Otherwise it looks good to me.
>
> Thank you and best regards,
>
>
> Zoltan
>
>
> On 09/11/2015 02:01 AM, sangheon.kim wrote:
>> Hi all,
>>
>> Please review this patch for command-line validation for GC flags.
>> This REDO patch is adding ranges and implementing constraint
>> functions for GC flags.
>>
>> Original CR of JDK-8078555 was backout as it made a test failure from
>> 'TestOptionsWithRanges.java'.
>> And also there were some discussion of OOM handling.
>>
>> Most parts are same as JDK-8078555 except below:
>> 1. Changed 'range' for some flags.
>> 2. Excluded 3 flags for TestOptionsWithRanges.java test. These flags
>> make this test unstable as it tries to allocate huge amount of memory.
>>
>> And below are the suggestion note for JDK-8078555:
>> 1. Exponential notation for 'double' type variable parse: Previously
>> there were some discussion for maximum value for double type flags
>> from code review of JDK-8059557 and JDK-8112746. And Kim and I
>> decided not to add upper limit unless there are problems with
>> DBL_MAX. And as 255 is the maximum length that can be passed via
>> command-line, we introduced exponential notation to avoid this limit.
>> ( arguments.cpp )
>> 2. These GC flags ranges are not ideal ranges but ranges which don't
>> make problem with current source code.
>> If one flag makes some problem but hard to find good range, I
>> added some ranges.
>> 3. There are some constraint functions to avoid overflow.
>> 4. Test applications are changed: as some of them assumed to be
>> ParallelGC or to check it's output messages.
>> 5. Includes cleanup of JDK-8133565: GC -2nd followup to JDK-8059557.
>>
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8134995
>>
>> Webrev:
>> http://cr.openjdk.java.net/~sangheki/8134995/webrev.00/
>> http://cr.openjdk.java.net/~sangheki/8134995/webrev.00_to_8078555
>>
>> Testing:
>> JPRT, UTE(vm.quick-pcl) and
>> test/runtime/CommandLine/OptionsValidation/TestOptionsWithRanges.java.
>>
>> Thanks,
>> Sangheon
>>
>
More information about the hotspot-dev
mailing list