RFR: 8318696: Do not use LFS64 symbols on Linux [v3]

Sam James duke at openjdk.org
Fri Jan 19 10:37:46 UTC 2024


On Fri, 19 Jan 2024 10:06:33 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Sam James has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add message for assert
>>   
>>   Not all C++ stds implement it w/o.
>
> src/hotspot/os/linux/os_linux.cpp line 4252:
> 
>> 4250: // otherwise, returns -1 that implies an error
>> 4251: jlong os::Linux::sendfile(int out_fd, int in_fd, jlong* offset, jlong count) {
>> 4252:   return ::sendfile64(out_fd, in_fd, (off_t*)offset, (size_t)count);
> 
> Why is this continuing to use sendfile64, rather than sendfile?

oh, great spot! I'll take you up on the recommendation for banning the *64 functions in one of the followups.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16329#discussion_r1458756249


More information about the build-dev mailing list