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

David Holmes david.holmes at oracle.com
Wed Feb 4 08:15:28 UTC 2015


On 4/02/2015 11:44 AM, Jesper Wilhelmsson wrote:
> 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 ;)

Yes although we have already applied to this to a non-OpenJDK repo this 
still needs a Reviewer for 8u. Can someone from GC please Review.

Thanks,
David

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