Code review request: JDK-8000692,Remove old KERNEL code
Chris Plummer
chris.plummer at oracle.com
Thu Jan 24 09:48:56 PST 2013
We stopped building kernel when minimalvm support was added. At the time
it was only being built on Windows. As for sun.jkernel.DownloadManager,
I see remnants of it in jdk/.hg/store/data, so it looks like it was
deleted. install/src/windows still contains DownloadManager.c and
DownloadManager.h. These should probably be removed.
Chris
On 1/24/13 7:46 AM, David Holmes wrote:
> I am not sure. CC-ed Bill from deployment.
>
> Bill, could you please shed some light on status of Kernel VM on JDK
> side? Has it already been removed?
>
> Thanks,
>
> -Zhengyu
>
> On 1/23/2013 9:32 PM, David Holmes wrote:
>> >Looks good to me.
>> >
>> >Once question: I can not find sun.jkernel.DownloadManager anywhere
>> >(open or closed repos). Has this already been removed?
>> >
>> >David
>> >
>> >On 24/01/2013 1:02 AM, Zhengyu Gu wrote:
>>> >>I updated webrev, based on the comments from David, Chris and Karen,
>>> >>etc.
>>> >>
>>> >>Webrev:http://cr.openjdk.java.net/~zgu/8000692/webrev.01/
>>> >>
>>> >>I also filed JBS bug 8006691
>>> >>(https://jbs.oracle.com/bugs/browse/JDK-8006691) to track the removal of
>>> >>is_kernel_vm field in jvm_version_info structure.
>>> >>
>>> >>Thanks,
>>> >>
>>> >>-Zhengyu
>>> >>
>>> >>On 1/18/2013 1:22 PM, Zhengyu Gu wrote:
>>>> >>>Hi,
>>>> >>>
>>>> >>>This is a cleanup of kernel code, since it is already obsoleted - JDK7
>>>> >>>and JDK8 are no longer building kernel.
>>>> >>>
>>>> >>>JBS:https://jbs.oracle.com/bugs/browse/JDK-8000692
>>>> >>>Webrev:http://cr.openjdk.java.net/~zgu/8000692/webrev.00/
>>>> >>>
>>>> >>>Tests:
>>>> >>>- JPRT
>>>> >>>- ute vm.quick.testlist on Linux x32.
>>>> >>>
>>>> >>>Thanks,
>>>> >>>
>>>> >>>-Zhengyu
More information about the hotspot-dev
mailing list