RFR: 8026848: -XX:+G1SummarizeRSetStats can result in wrong exit code and crash

[Stefan Johansson]

Thanks Erik!

On 2013-10-22 10:37, Erik Helin wrote:
> Looks good!
>
> Thanks,
> Erik
>
> On 2013-10-22, Stefan Johansson wrote:
>> Thanks for the review Stefan.
>>
>> Cheers,
>> Stefan
>>
>> On 2013-10-21 17:18, Stefan Karlsson wrote:
>>> On 2013-10-21 17:05, Stefan Johansson wrote:
>>>> Hi,
>>>>
>>>> Please review this fix for:
>>>> https://bugs.openjdk.java.net/browse/JDK-8026848
>>>>
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~sjohanss/8026848/webrev.00/
>>> Looks good.
>>>
>>> StefanK
>>>
>>>> Summary:
>>>> Some of the log-messages in g1RemSetSummary.cpp used %d and %zu
>>>> to print size_t values. This might lead to the unexpected
>>>> behavior described in the bug report. This patch changes those
>>>> messages to use SIZE_FORMAT and SIZE_FORMAT_W instead.
>>>>
>>>> Testing:
>>>> I was able to reproduce the crash on my laptop with a clean
>>>> build, after applying my change the problem goes away. Also ran
>>>> the tests mentioned in the bug-report through JPRT and results
>>>> were clean.
>>>>
>>>> Thanks,
>>>> StefanJ