[8u-dev] RFR + Request for approval: 8163969: Cyclic interface initialization causes JVM crash

[Coleen Phillimore]

Thank you Serguei.

On 10/12/16 7:35 PM, serguei.spitsyn at oracle.com wrote:
> Coleen,
>
> The backport looks good to me.
>
> Minor questions to the test.
>
>
> http://cr.openjdk.java.net/~coleenp/8163969.8.01/webrev/test/runtime/lambda-features/TestInterfaceInit.java.frames.html 
>
>
> 2 * Copyright (c) 2014, 2015, Oracle and/or its affiliates. All rights 
> reserved. 2015 => 2016
>

My commit script fixes copyrights, so I'll change that in the commit.
>
> 28 * @bug 8098557
>
>   Why the new bug number is not 8163969?
>

I fixed the bug number as you and David suggested.

Thanks again!
Coleen

>
> Thanks,
> Serguei
>
>
>
> On 10/12/16 15:51, Coleen Phillimore wrote:
>>
>> Note, this is also an RFR since the backport wasn't clean.
>> thanks,
>> Coleen
>>
>>
>> On 10/12/16 5:10 PM, Coleen Phillimore wrote:
>>> Summary: Backport change to correct interface initialization.
>>>
>>> There were too many changes to instanceKlass.cpp for a clean 
>>> backport.  Also in JDK8, this corrects interface initialization to 
>>> not initialize the whole interface hierarchy if an interface, not 
>>> class, initializes initialization.  This is to correctly follow JLS 
>>> 12.4.2 step 7.   I filed a compatibility request (in review) to 
>>> document the difference in behavior, which I believe will not be 
>>> noticed.
>>>
>>> Tested with JPRT, including runtime jtreg lambda-features tests, and 
>>> JCK tests.
>>>
>>> open webrev at http://cr.openjdk.java.net/~coleenp/8163969.8.01/webrev
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8163969.8
>>>
>>> Thanks,
>>> Coleen
>>
>