RFR: 8260264: Move common os_<unix> inline methods to a common posix source file [v2]

Coleen Phillimore coleenp at openjdk.java.net
Thu Jan 28 21:23:59 UTC 2021


On Thu, 28 Jan 2021 19:38:01 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Please review this fix for JDK-8260264 to move common functions from os_<unix>.inline.hpp files to os_posix.inline.hpp.  Functions ftruncate() and the two numa* functions could also be moved to os_posix.inline.hpp but would require BSD_ONLY and AIX_ONLY macros because of minor differences.
>> 
>> The change was tested by running Mach5 tiers 1,2 on Linux, Mac OS, and Windows, and tiers 3-5 on Linux x64.  Additionally, builds were done on Linux-s390x, Linux-ppc, Linux-arm32, and linux-x86.
>> 
>> Thanks, Harold
>
> Harold Seigel has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8260264: restore stack APIs, move ftruncate and numa APIs

Looks good to me.

-------------

Marked as reviewed by coleenp (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2267


More information about the hotspot-runtime-dev mailing list