[8u] RFR for backport of JDK-8169931: 8k class metaspace chunks misallocated from 4k chunk freelist

[Muthusamy Chinnathambi]

Hi Kevin,

> Yes looks good to me, let me know if you need help with the push,
Thanks. 
Yes, it would be great if you could help with the push.

Regards,
Muthusamy C

-----Original Message-----
From: Kevin Walls 
Sent: Tuesday, January 2, 2018 5:14 PM
To: Muthusamy Chinnathambi <muthusamy.chinnathambi at oracle.com>; David Holmes <david.holmes at oracle.com>; HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>
Cc: Stephen Fitch <stephen.fitch at oracle.com>
Subject: Re: [8u] RFR for backport of JDK-8169931: 8k class metaspace chunks misallocated from 4k chunk freelist

Hi Muthu,

Yes looks good to me, let me know if you need help with the push,

Thanks
Kevin


On 02/01/2018 11:04, Muthusamy Chinnathambi wrote:
> Hi David,
>
> Thanks for the review.
>
>> Copyright dates will now need to be updated to 2018.
> Yes. Please find the updated webrev link below.
> http://cr.openjdk.java.net/~mchinnathamb/JDK-8169931/webrev.02/
>
> Regards,
> Muthusamy C
>
> -----Original Message-----
> From: David Holmes
> Sent: Tuesday, January 2, 2018 12:21 PM
> To: Muthusamy Chinnathambi <muthusamy.chinnathambi at oracle.com>; HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>
> Cc: Stephen Fitch <stephen.fitch at oracle.com>
> Subject: Re: [8u] RFR for backport of JDK-8169931: 8k class metaspace chunks misallocated from 4k chunk freelist
>
> Hi Muthusamy,
>
> On 2/01/2018 4:08 PM, Muthusamy Chinnathambi wrote:
>> Ping..
>>
>> May I please get a review for this?
>>
>> Regards,
>> Muthusamy C
>>
>> -----Original Message-----
>> From: Muthusamy Chinnathambi
>> Sent: Monday, December 18, 2017 3:06 PM
>> To: HotSpot Open Source Developers <hotspot-dev at openjdk.java.net>
>> Subject: [8u] RFR for backport of JDK-8169931: 8k class metaspace chunks misallocated from 4k chunk freelist
>>
>> Hi,
>> 		
>> Please review the backport of bug: "JDK-8169931: 8k class metaspace chunks misallocated from 4k chunk freelist" to jdk8u-dev
>>
>> Please note that this is not a clean backport due to new entries in jni.cpp.
>>
>>
>> Webrev:  http://cr.openjdk.java.net/~mchinnathamb/JDK-8169931/webrev.01/
>> jdk9 bug:  https://bugs.openjdk.java.net/browse/JDK-8169931
>> Original patch pushed to jdk9:  http://hg.openjdk.java.net/jdk9/hs/hotspot/rev/8d26435a4c69
> The functional backport appears to be a correct copy of the JDK 9 fix.
>
> The use of the test code in jni.cpp also appears correct.
>
> Copyright dates will now need to be updated to 2018.
>
> Thanks,
> David
>
>> Test:  Had run jtreg, jprt hotspot testset and with -XX:+ExecuteInternalVMTests.
>>
>> Regards,
>> Muthusamy C
>>