login
Header Space

 
 

Re: add parsing to cifs mount option for mounting to shares which support transport encryption

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <smfrench@...>
Cc: <linux-fsdevel@...>, <linux-cifs-client@...>, <jlayton@...>
Date: Thursday, May 15, 2008 - 1:30 pm

> Samba now supports transport encryption on particular shares (exports)

Please also add it to cifs_show_options().

BTW, cifs is the last filesystem in mainline which is missing a lot of
mount options from its show_options() method.

cifs_parse_mount_options() is also ripe for some cleanups: it's a >500
line function and hacks like the "sep=" option should really be
exterminated.  There are _much_ better ways to escape the separator
charater than to completely replace it.  Like surrounding it with
quotes or prefixing with a backslash.

Miklos
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: add parsing to cifs mount option for mounting to shares ..., Miklos Szeredi, (Thu May 15, 1:30 pm)
speck-geostationary