Integrated: 8265175: (fs) Files.copy(Path, Path, CopyOption...) should use sendfile on Linux
Brian Burkhalter
bpb at openjdk.java.net
Fri Apr 16 15:49:36 UTC 2021
On Tue, 13 Apr 2021 21:40:26 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:
> Please consider this request to change the underlying native implementation of `java.nio.file.Files.copy(Path,Path,CopyOption...)` on Linux only to perform zero-copy via `sendfile(2)`. The `sendfile()` system call is already used in `java.nio.channels.FIleChannel.transferTo(long,long,WritableByteChannel)`. It is intentionally _not_ proposed to use `sendfile()` in the native macOS implementation as on macOS the function requires that the destination file descriptor is for a socket.
>
> This change showed some performance improvement as measured by JMH.
>
> **Before: user-space buffers (read() + write())**
>
> Benchmark (size) Mode Cnt Score Error Units
> FilesCopy.copy 10240 thrpt 5 39167.400 ± 683.887 ops/s
> FilesCopy.copy 51200 thrpt 5 20782.622 ± 558.031 ops/s
> FilesCopy.copy 102400 thrpt 5 13260.709 ± 176.673 ops/s
> FilesCopy.copy 512000 thrpt 5 3171.837 ± 175.803 ops/s
> FilesCopy.copy 1048568 thrpt 5 1654.253 ± 39.419 ops/s
> FilesCopy.copy 10485760 thrpt 5 145.328 ± 7.192 ops/s
> FilesCopy.copy 104857600 thrpt 5 12.440 ± 2.275 ops/s
> FilesCopy.copy 1073741824 thrpt 5 1.073 ± 0.081 ops/s
>
> **After: zero-copy (sendfile())**
>
> Benchmark (size) Mode Cnt Score Error Units
> FilesCopy.copy 10240 thrpt 5 40571.516 ± 548.977 ops/s
> FilesCopy.copy 51200 thrpt 5 23993.334 ± 506.817 ops/s
> FilesCopy.copy 102400 thrpt 5 15927.485 ± 309.081 ops/s
> FilesCopy.copy 512000 thrpt 5 4207.129 ± 95.454 ops/s
> FilesCopy.copy 1048568 thrpt 5 2147.046 ± 33.446 ops/s
> FilesCopy.copy 10485760 thrpt 5 148.798 ± 1.329 ops/s
> FilesCopy.copy 104857600 thrpt 5 14.541 ± 0.675 ops/s
> FilesCopy.copy 1073741824 thrpt 5 1.270 ± 0.029 ops/s
This pull request has now been integrated.
Changeset: 1c3fd46d
Author: Brian Burkhalter <bpb at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/1c3fd46d
Stats: 29 lines in 1 file changed: 26 ins; 1 del; 2 mod
8265175: (fs) Files.copy(Path,Path,CopyOption...) should use sendfile on Linux
Reviewed-by: alanb
-------------
PR: https://git.openjdk.java.net/jdk/pull/3476
More information about the nio-dev
mailing list