[PATCH] vfs_acl_xattr: avoid setting POSIX acls if "ignore system acls" is set
uri at samba.org
Tue Mar 22 19:03:04 UTC 2016
On 03/22/2016 05:14 PM, Richard Sharpe wrote:
> Can you also remove that stupid time value that was added? It screws
> up ACL deduplication if you have a file system that can do that.
> At least it should be configurable.
From brief look into the code I can't understand the purpose of the
time stamp (only idea that comes to mind is for extra-debugging info).
I'll look through the history but if someone can tell me please do so.
Same goes for "description" which seems (again, brief look) to be a
place holder for something.
> Actually, it should also be possible to configure V3 vs V4 formats as well.
Cannot see the use case for that -
v4 seems to be more efficient for write-once-read-many which is the
common case. If there's a file system in which fetching the NT ACL is
faster than fetching the POSIX ACL (and hence v3 would be faster for
read), then I suppose it is not a POSIX-like file system and should not
be using vfs_acl_xattr in the first place.
>> Passes local make test.
>> Please review,
More information about the samba-technical