RFR: JDK-8262957 (fs) Fail fast in UnixFileStore.isExtendedAttributesEnabled
Aleksey Shipilev
shade at openjdk.java.net
Mon Apr 12 06:24:30 UTC 2021
On Wed, 7 Apr 2021 16:51:23 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Added new capability flag `UnixNativeDispatcher.SUPPORTS_XATTR`, which gets set `#ifdef _SYS_XATTR_H_`.
>>
>> Note that `_SYS_XATTR_H_` is defined in `xattr.h` in both [macOS/Darwin](https://github.com/apple/darwin-xnu/blob/main/bsd/sys/xattr.h), [Linux and GNU-based BSD systems using glibc](https://github.com/bminor/glibc/blob/master/misc/sys/xattr.h). It might not be defined for other operating systems that still support xattr. So if OpenJDK eventually adds support for further platforms, this might need to be adjusted as well.
>>
>> If xattr capabilities are missing, `UnixFileStore.isExtendedAttributesEnabled` will return false immediately, avoiding any I/O.
>>
>> As a side effect of this change, I redefined some other (private) capabilities. Strictly speaking, this change is not required but keeps the code consistent, as [previously discussed in the mailing list](https://mail.openjdk.java.net/pipermail/nio-dev/2021-February/008133.html).
>
> Marked as reviewed by alanb (Reviewer).
Alan, are you good with integrating this?
-------------
PR: https://git.openjdk.java.net/jdk/pull/2816
More information about the nio-dev
mailing list