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

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


On 10/15/19 8:28 AM, 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/

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

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

src/hotspot/os/bsd/os_bsd.cpp
     No comments.

src/hotspot/os/bsd/os_bsd.inline.hpp
     No comments.

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

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

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

src/hotspot/os/posix/os_posix.cpp
     No comments.

src/hotspot/os/solaris/os_solaris.cpp
     Wow. Haven't seen that code in years.

src/hotspot/os/solaris/os_solaris.hpp
     No comments.

src/hotspot/os/solaris/os_solaris.inline.hpp
     No comments.

src/hotspot/os/windows/os_windows.cpp
     No comments.

src/hotspot/os/windows/os_windows.inline.hpp
     No comments.

src/hotspot/os_cpu/solaris_sparc/os_solaris_sparc.cpp
     No comments.

src/hotspot/share/gc/g1/g1CollectedHeap.cpp
     No comments.

src/hotspot/share/runtime/os.hpp
     is_suspend_request() is a bit of a surprise, but okay.

Thumbs up.

Dan


> 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