Integrated: 8316304: (fs) Add support for BasicFileAttributes.creationTime() for Linux

Severin Gehwolf sgehwolf at openjdk.org
Mon Oct 16 07:44:17 UTC 2023


On Mon, 18 Sep 2023 17:05:11 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

> Please review this nio change which adds `BasicFileAttributes.creationTime()` support for Linux by means of the `statx` Linux specific system call. The patch does a dynamic lookup of the function and if it is available, uses it to set the support birth time capability in `UnixNativeDispatcher`. When `statx` is not available, it won't use it and fall back to the old behaviour on such systems. It should, however, compile fine on Linux systems not supporting the `statx` system call.
> 
> Testing:
> - [x] GHA (MacOS X failure seems unrelated)
> - [x] `java.nio` tests.
> - [x] Manual builds and tests on older glibc versions (2.17 => fallback, 2.28+ => works). Compiles fine and falls back to the not supported case as it was before this patch.
> 
> Thoughts?

This pull request has now been integrated.

Changeset: 0275efac
Author:    Severin Gehwolf <sgehwolf at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/0275efac882d263f85a723828a95d621f8dadbfb
Stats:     245 lines in 3 files changed: 229 ins; 0 del; 16 mod

8316304: (fs) Add support for BasicFileAttributes.creationTime() for Linux

Reviewed-by: stuefe, alanb, bpb, mli

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

PR: https://git.openjdk.org/jdk/pull/15792


More information about the nio-dev mailing list