[PATCH] Fix for XDR Backend of NFS4ACL_XATTR module to get it working with NFS4.0 ACL Spec

Sandeep Nashikkar snashikkar at commvault.com
Mon Aug 27 06:59:34 UTC 2018


On Fri, 24 August, 2018 at 04:40 Jeremy Allison wrote:
> > On Thu, Aug 16, 2018 at 11:43:48AM +0000, Sandeep Nashikkar wrote:
> > On a side note, the nfs4acl_* functions do not comply with NFS 4.1 Spec. 

> Oh, that's a pain. Can you tell me how they differ ?

There is no field called iflag in the NFS ACE as per NFS 4.1 Spec

> So I'm OK with this code now. RB+ Jeremy Allison in terms of how it encodes and changes the nfs4acls.
> What I still need to understand is the effect of this on existing setups with nfs4 ACLs stored in xattrs.
> which means that selecting both NFS4ACL_ENCODING_XDR *and* NFS40ACL_ENCODING_XDR will use the same xattr name of

Jeremy, xattr name can be provided via xattr_name module parameter. It will not break the existing setup if they provide required parameters correctly. 
We can revert the default xattr name back to original if you insist. Let me know. 

Thanks
Sandeep

***************************Legal Disclaimer***************************
"This communication may contain confidential and privileged material for the
sole use of the intended recipient. Any unauthorized review, use or distribution
by others is strictly prohibited. If you have received the message by mistake,
please advise the sender by reply email and delete the message. Thank you."
**********************************************************************




More information about the samba-technical mailing list