Review request (S): 7132311 G1: assert((s == klass->oop_size(this)) || (Universe::heap()->is_gc_active() && ((is_typeArray()...

[Bengt Rutisson]

Forgot to mention something about testing:

I was able to reproduce the assert using UTE and the Juggle12 test. It 
hits the assert every time I run it on my laptop. With the fix in the 
webrev the assert is gone.

I also tested with the reproducer I had for "7131791: G1: Asserts in 
nightly testing due to 6976060" and it passes that too. Finally I also 
tried out the small test app that I wrote to see that we get concurrent 
collections. That works fine as well. No full GCs.

Bengt

On 2012-01-23 13:21, Bengt Rutisson wrote:
>
> Hi all,
>
> Can I have one more review for this change, please? Stefan has already 
> looked at it.
>
> http://cr.openjdk.java.net/~brutisso/7132311/webrev.02/
>
> The idea is that we move the check for whether or not we should 
> initiate a marking cycle to before we allocate a humongous object. 
> This way we can ignore the issue with uninitialized memory.
>
> Thanks,
> Bengt
>