On 10/12/18 12:04 PM, Soumya Koduri wrote:
1. Cache it separately as posix ACLs (a new option maybe
"cache-glusterfs-acl" is added);
And make sure _posix_xattr_get_set fills them when lookup requests.
I am not sure if posix layer can handle it. Virtual xattrs are
in-memory and not stored on disk. They are converted to/from posix-acl
in posix-acl xlator. So FWIU, posix-acl xlator should handle setting
these attributes as part of LOOKUP response if needed. Same shall
apply for any virtual xattr cached in md-cache. Request Poornima to
comment.
Posix-acl can hand it correctly now.
Okay.
At a time, any gfapi consumer would use either posix-acl or virtual
glusterfs ACLs. So having two options to selectively choose which one
of them to cache sounds better to me instead of unnecessarily storing
two different representations of the same ACL.
Make sense.
I will add another option for virtual glusterfs ACLs in md-cache.
Cool. thanks!
-Soumya
thanks,
Kinglong Mee
_______________________________________________
Gluster-devel mailing list
Gluster-devel@xxxxxxxxxxx
https://lists.gluster.org/mailman/listinfo/gluster-devel