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

[Coleen Phillimore]

On 10/12/16 9:47 PM, David Holmes wrote:
> 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, yes, I fixed it.  Thanks for you and Serguei noticing it.

Coleen

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