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

[David Holmes]

Hi Coleen,

On 13/10/2016 8:51 AM, Coleen Phillimore wrote:
>
> Note, this is also an RFR since the backport wasn't clean.
> thanks,
> Coleen

Backport of fix itself is good.

I'm assuming you simply copied across the existing test from the JDK9 
repo. The reference in the test to 8098557 is confusing because 8098557 
was never backported and 8163969 effectively replaces it. So as Serguei 
alluded to I'd replace the @bug 8098557 with 8163969.

Thanks,
David

>
> 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
>