With the introduction of SMB3, the 'smb encrypt' setting has an effect also for protocol versions other than SMB1. This needs to be explained in the manual page.
Created attachment 11206 [details] proposed patch for 4.2
Created attachment 11207 [details] proposed patch for 4.1
Comment on attachment 11207 [details] proposed patch for 4.1 looks good but one thing i thought while reading it: to make it more clear, i would replace "hopefully soon Linux Cifsvfs..." by "not not supported yet (currently Linux cifs vfs 4.1)", so that the man page does not talk of "hopes" and so that it does not lie if the samba (man page) is older than the installed kernel which might support it already then.
Björn, thanks for the comments. That smb1 part I just copied from the original text. So I suggest to bring that into master and we can backport it separately. This patchset addresses the lack of addressing the effect of "smb encrypt" for smb3. Regarding your proposal, I'd say yeah, we can do it, but the current way has the advantage that by not specifying a version number of cifsfs, we don't need to adapt the manpage in that aspect as time proceeds (as long as cifsfs does not implement encryption).
(In reply to Michael Adam from comment #4) the current way has the disadvantage, that it is wrong as soon as the kernel situation changes. and you will have old less up-to-date samba installations on systems with more recent kernels. it is not neccessary to adopt the mentioned kernel version in the man page all the time. just dont't write things there that will for sure be misinforming at some point in the future. making clear that current kernel 4.1 does not support encrytption is clear. people with kernel 4.1+x will know that the sutuation might have changed.
(In reply to Björn Jacke from comment #5) Björn, I am happy to review a corresponding patch to the manpage (on mailing-list/master and also here in BZ for the releases). Could we get this separate patch into the releases?
Comment on attachment 11206 [details] proposed patch for 4.2 LGTM.
Karo, please add to v4.2 and v4.1. Thanks.
(In reply to Guenther Deschner from comment #8) Pushed to autobuild-v4-[1|2]-test.
(In reply to Karolin Seeger from comment #9) Pushed to both branches. Closing out bug report. Thanks!