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

Tue Feb 2 08:41:51 UTC 2021

On Thu, 28 Jan 2021 21:23:58 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: fix lseek64

There is an AIX 32bit but its not supported by the OpenJDK.

The change looks good. Thanks for this cleanup. I tested on AIX, seems to build and work fine.

..Thomas

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

Marked as reviewed by stuefe (Reviewer).

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