RFR: Forward port of 8055006 - Store original value of Min/MaxHeapFreeRatio

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Tue Sep 23 08:13:25 UTC 2014


Thanks Magnus!
/Jesper

Magnus Ihse Bursie skrev 23/9/14 09:13:
> On 2014-09-20 19:16, Jesper Wilhelmsson wrote:
>> Hi all,
>>
>> I got approvals for the HotSpot changes and they have now been pushed to 
>> jdk9/hs-gc. For the JDK makefile change I would prefer if someone that feels 
>> comfortable with the JDK makefiles would have a look at it before I push that 
>> part.
> The jdk makefile changes looks good to me.
>
> /Magnus
>>
>> Thanks,
>> /Jesper
>>
>> Jesper Wilhelmsson skrev 16/9/14 20:32:
>>> Hi,
>>>
>>> The fix for JDK-8055006 was reviewed by several engineers and was pushed
>>> directly to 8u40 due to time constraints. This is a forward port to get the 
>>> same
>>> changes into JDK 9.
>>>
>>> There are two webrevs, one for HotSpot and one for the JDK.
>>>
>>> The 8u40 HotSpot change applied cleanly to 9 so if this was a traditional
>>> backport it wouldn't require another review. But since this is a weird 
>>> situation
>>> and I'm pushing to 9 I'll ask for reviews just to be on the safe side.
>>> Also, the original 8u40 push contained some unnecessary changes that was later
>>> cleaned up by JDK-8056056. In this port to 9 I have merged these two changes
>>> into one to avoid introducing a known issue only to remove it again.
>>>
>>> The JDK change is new. The makefiles differ between 8u40 and 9 and this new
>>> change makes use of functionality not present in 8u40. This patch was provided
>>> by Erik Joelsson and I have reviewed it myself, but it needs two reviews so
>>> another one is welcome.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8055006
>>>
>>> Webrevs: http://cr.openjdk.java.net/~jwilhelm/8055006/jdk9/
>>>
>>>
>>> 8u40 Webrevs: http://cr.openjdk.java.net/~jwilhelm/8055006/
>>>
>>> 8u40 changes:
>>> http://hg.openjdk.java.net/jdk8u/jdk8u-dev/hotspot/rev/f933a15469d4
>>> http://hg.openjdk.java.net/jdk8u/jdk8u-dev/jdk/rev/312152328471
>>>
>>> Bug and change for the second 8u40 fix:
>>> https://bugs.openjdk.java.net/browse/JDK-8056056
>>> http://hg.openjdk.java.net/jdk8u/hs-dev/hotspot/rev/9be4ca335650
>>>
>>> Thanks!
>>> /Jesper
>



More information about the hotspot-dev mailing list