RFR (XXS): 8134856: Incorrect use of PLAB::min_size() in MaxPLABSizeBounds

[sangheon.kim]

Hi Thomas,

On 09/01/2015 09:52 AM, Thomas Schatzl wrote:
> Hi Sangheon,
>
> On Tue, 2015-09-01 at 09:47 -0700, sangheon.kim wrote:
>> Hi Thomas,
>>
>> Thank you for fixing this!
>>
>> One question for the test.
>> If we set 'G1HeapRegionSize=1M' and 'YoungPLABSize=2M', error message
>> comes from the constructor of G1CollectorPolicy.
>>       jio_snprintf(buffer, sizeof(buffer), "%sPLABSize should be at most
>> " SIZE_FORMAT,
>>                    OldPLABSize > region_size ? "Old" : "Young", region_size);
> No. This "validation" has already been removed in an earlier RFR
> (JDK-8134858).
Okay, so this RFR is based on other RFR(JDK-8134858) which is not yet 
committed.

Thanks,
Sangheon

>
>> So VM exits before going to the command-line validation (i.e.
>> MaxPLABSizeBounds() ) routines.
> No, it ends up in the new command line validation that has this bug.
>
>> Am I missing something?
> Thanks,
>    Thomas
>
>