Code review request: 8006691: Remove jvm_version_info.is_kernel_jvm field
Zhengyu Gu
zhengyu.gu at oracle.com
Fri Feb 8 13:44:44 PST 2013
Mandy,
Thanks for reviewing.
What's the process to update jvm.h in the jdk repo? it should go to both
jdk7 and jdk8, which repos should I make the changes?
Thanks,
-Zhengyu
On 2/8/2013 4:18 PM, Mandy Chung wrote:
> Zhengyu,
>
> This looks good to me too. Please also update the jvm.h in the jdk
> repo (src/share/javavm/export/jvm.h) - for historical reason, this is
> not an exact copy in the hotspot repo and we tend to apply the same
> patch rather than replacing it with the hotspot copy.
>
> BTW here is the public accessible link to the bug report:
> http://bugs.sun.com/view_bug.do?bug_id=8006691
>
> Mandy
>
> On 2/8/2013 8:00 AM, Zhengyu Gu wrote:
>> This is a simple change to remove is_kernel_jvm field in
>> jvm_version_info structure. The CCC request has been approved, as
>> KERNEL VM has been deprecated since JDK7.
>>
>> JBS: https://jbs.oracle.com/bugs/browse/JDK-8006691
>> Webrev: http://cr.openjdk.java.net/~zgu/8006691/webrev.00/
>>
>> Thanks,
>>
>> -Zhengyu
>>
>>
More information about the hotspot-dev
mailing list