On Wed, 13 Jul 2011 11:34:35 -0500
[email protected] wrote:

> From: Shirish Pargaonkar <[email protected]>
> 
> 
> Manpage contents for cifs mount option  cifsacl
> 
> 
> Signed-off-by: Shirish Pargaonkar <[email protected]>
> ---
>  mount.cifs.8 |   19 +++++++++++++++++++
>  1 files changed, 19 insertions(+), 0 deletions(-)
> 
> diff --git a/mount.cifs.8 b/mount.cifs.8
> index 7e0f117..6c67b4a 100644
> --- a/mount.cifs.8
> +++ b/mount.cifs.8
> @@ -272,6 +272,25 @@ Do not allow POSIX ACL operations even if server would 
> support them\&.
>  The CIFS client can get and set POSIX ACLs (getfacl, setfacl) to Samba 
> servers version 3\&.0\&.10 and later\&. Setting POSIX ACLs requires enabling 
> both XATTR and then POSIX support in the CIFS configuration options when 
> building the cifs module\&. POSIX ACL support can be disabled on a per mount 
> basis by specifying "noacl" on mount\&.
>  .RE
>  .PP
> +cifsacl
> +.RS 4
> +Allow operations with file objects which posses CIFS/NTFS ACL and Security 
> Descriptors\&.
> +.sp
> +This option is used to work with file objects which posses CIFS/NTFS ACL
> +instead of POSIX ACL and Security Descriptors on shares mounted from
> +servers such as Windows which do not support unix extensions.
> +

This seems rather vague. Maybe something like:

"This option allows the client to map CIFS/NTFS ACLs to permission mode bits."

Then later, you can specify that if the upcall is properly configured
that it will also map SIDs to UIDs for ownership purposes.

Note that the goal here is to describe this such that someone can
reasonably predict what happens when they turn this on, and make an
informed decision about whether they should do this. It should also
mention the drawbacks of turning it on (increased latency when fetching
metadata, etc). If it's not reasonable to fit this in a synopsis for
the mount option, consider adding a new section (like the "INODE
NUMBERS" section).


> +It is used in conjuction with files /etc/nsswitch.conf and smb.conf
> +and utilities cifs.idmap and winbindd.  Please refer to the manpages
> +for utilities cifs.idmap and winbindd for the usage.
> +

How is it used in conjunction with this? What happens if I enable this
option and don't set up cifs.idmap?

> +For commands which either display or modify unix mode and permissions
> +such as ls, stat, and chmod or commands which modify owner and group
> +such as chown and chgrp, to work on file system objects that belong
> +to shares mounted from servers such as Windows, this mount option
> +is necessary.

Holy run-on-sentences, Batman! Maybe break that up a little?


> +.RE
> +.PP
>  nocase
>  .RS 4
>  Request case insensitive path name matching (case sensitive is the default 
> if the server suports it)\&.


-- 
Jeff Layton <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to