RFR: 8232211: Remove dead code from os.hpp|cpp

Daniel D. Daugherty daniel.daugherty at oracle.com
Tue Oct 15 14:33:12 UTC 2019


On 10/15/19 10:00 AM, Claes Redestad wrote:
> Hi,
>
> Robbin pointed out a few additional things that seemed to be unused,
> including an unused createThread_lock Mutex and that _os_version is
> never read, which means initialize_os_info can go - too.
>
> New webrev: http://cr.openjdk.java.net/~redestad/8232211/open.01/
> Incremental: http://cr.openjdk.java.net/~redestad/8232211/open.00_01.inc/

src/hotspot/os/aix/os_aix.hpp
     No comments.

src/hotspot/os/linux/os_linux.cpp
     No comments.

src/hotspot/os/linux/os_linux.hpp
     No comments.

Thumbs up.

Dan

>
> Testing: I'll get some done...
>
> Thanks!
>
> /Claes
>
> On 2019-10-15 14:28, Claes Redestad wrote:
>> Hi,
>>
>> feel free to review this patch which simply removes a number of
>> dead and/or effectively unused methods from our os interface
>>
>> Webrev: http://cr.openjdk.java.net/~redestad/8232211/open.00/
>> Bug:    https://bugs.openjdk.java.net/browse/JDK-8232211
>>
>> Testing: tier1-2,5 (includes minimal VM builds)
>>
>> Thanks!
>>
>> /Claes



More information about the hotspot-runtime-dev mailing list