[8u60] (XS) RFR: 8071972: Minimal VM is broken

Jesper Wilhelmsson jesper.wilhelmsson at oracle.com
Wed Feb 4 01:44:15 UTC 2015


Unfortunately yes. We haven't decided to allow small changes to go in with only 
one reviewer yet. Also, I'm only committer so you need someone more experienced 
than me to study this change ;)
/Jesper

David Holmes skrev den 4/2/15 02:31:
> Thanks Jesper! Do I need a second GC reviewer for this?
>
> David
>
> On 4/02/2015 7:37 AM, Jesper Wilhelmsson wrote:
>> Looks good!
>> /Jesper
>>
>> David Holmes skrev den 2/2/15 05:51:
>>> http://cr.openjdk.java.net/~dholmes/8071972/webrev.8u60/
>>>
>>> Original bug is confidential but the backport issue is not:
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8072057
>>>
>>> Simple fix to ensure header inclusion is guarded based on all GCs
>>> being included.
>>>
>>> This fix is not applicable to 9 due to the changes there that have
>>> removed the
>>> parNew and defNew support. In fact inclusion of this header is no
>>> longer needed
>>> in 9 and I've filed
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8072056
>>>
>>> to clean that up.
>>>
>>> Thanks,
>>> David


More information about the hotspot-gc-dev mailing list