RFR: JDK-8077265 Modify assert to help debug JDK-8068448

Eric Caspole eric.caspole at oracle.com
Thu Apr 9 14:26:08 UTC 2015


On 4/9/2015 10:01 AM, Stefan Karlsson wrote:
> Hi Eric,
>
> On 2015-04-09 15:19, Eric Caspole wrote:
>> HI everybody,
>> Here is a webrev to add more asserts related to debugging 
>> JDK-8068448. Beyond capturing more info in the original assert, after 
>> looking at another core I added more asserts to make sure there is no 
>> other place where old gen allocations would overrun the start array.
>
> Why didn't these two new asserts get the same, more informative, error 
> message as the first assert you changed? Maybe you could extract the 
> check out to a helper macro that prints the relevant information?

OK I will try that.

>
>
> Another point that Bengt mentioned yesterday, is that we don't really 
> need to print the old_gen part of the assert. It's already printed in 
> the hs_err file.

If it is a race as we discussed, I want to try to see the values that 
that thread saw, not what comes out in the hs_err file, similar to what 
is going on with the start array.

>
> Thanks,
> StefanK
>
>>
>>  http://cr.openjdk.java.net/~ecaspole/JDK-8077265/00/webrev/
>>
>> Passes JPRT.
>> Thanks,
>> Eric
>




More information about the hotspot-gc-dev mailing list