[PATCH] vfs_acl_xattr: avoid setting POSIX acls if "ignore system acls" is set

[Uri Simchoni]

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.

Uri.

>> Passes local make test.
>>
>> Please review,
>> Uri.
>
>
>