manpage: update mount.cifs manpage with info about echo_interval option.

Message ID CAA_-hQJ=DsbeVWU7mr-W0sTRiPXnCC3vznmznQwBedQLtE0s_g@mail.gmail.com
State New
Headers show
Series
  • manpage: update mount.cifs manpage with info about echo_interval option.
Related show

Commit Message

Kenneth Dsouza Jan. 23, 2018, 5:37 p.m.
Signed-off-by:  Kenneth D'souza <kdsouza@redhat.com>





Kind Regards,
Kenneth D'souza
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Steve French Jan. 23, 2018, 11:05 p.m. | #1
looks reasonable

On Tue, Jan 23, 2018 at 11:37 AM, Kenneth Dsouza <kdsouza@redhat.com> wrote:
> Signed-off-by:  Kenneth D'souza <kdsouza@redhat.com>
>
> diff --git a/mount.cifs.rst b/mount.cifs.rst
> index 023ff34..efd7055 100644
> --- a/mount.cifs.rst
> +++ b/mount.cifs.rst
> @@ -385,6 +385,14 @@ mfsymlinks
>    option. Minshall+French symlinks are used even if the server supports
>    the CIFS Unix Extensions.
>
> +echo_interval=n
> +  sets the interval at which echo requests are sent to the server on an
> +  idling connection. This setting also affects the time required
> +  for a connection to an unresponsive server to timeout.
> +  Here n is the echo interval in seconds. If this option is not given then
> +  the default value of 60 seconds is used.
> +  The minimum tunable value is 1 second and maximum can go up to 600 seconds.
> +
>  serverino
>    Use inode numbers (unique persistent file identifiers) returned by the
>    server instead of automatically generating temporary inode numbers on
>
>
>
>
> Kind Regards,
> Kenneth D'souza
> --
> To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
Aurélien Aptel Jan. 24, 2018, 10:40 a.m. | #2
In practice I've noticed that we actually wait for twice the
echo_interval value to reconnect (/me has been doing a lot of
reconnection tests lately).

Cheers,
Sachin Prabhu Jan. 24, 2018, 10:53 a.m. | #3
On Wed, 2018-01-24 at 11:40 +0100, Aurélien Aptel wrote:
> In practice I've noticed that we actually wait for twice the
> echo_interval value to reconnect (/me has been doing a lot of
> reconnection tests lately).
> 
> Cheers,
> 
I've documented it at
http://sprabhu.blogspot.in/2015/08/investigation-into-effects-of-server
.html
if you are interested.

Upstream patch which added this option.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/comm
it/?id=adfeb3e00e8e1b9fb4ad19eb7367e7c272d16003

--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Aurélien Aptel Jan. 24, 2018, 11:14 a.m. | #4
Sachin Prabhu <sprabhu@redhat.com> writes:
> I've documented it at
> http://sprabhu.blogspot.in/2015/08/investigation-into-effects-of-server
> .html
> if you are interested.

Great analysis! I remember reading through your blog some time ago but
forgot about this post. (Btw it would be even nicer if the code was
formated [1] ;)

Cheers,

1: https://stackoverflow.com/questions/679189/formatting-code-snippets-for-blogging-on-blogger
Sachin Prabhu Jan. 24, 2018, 12:35 p.m. | #5
On Wed, 2018-01-24 at 12:14 +0100, Aurélien Aptel wrote:
> Sachin Prabhu <sprabhu@redhat.com> writes:
> > I've documented it at
> > http://sprabhu.blogspot.in/2015/08/investigation-into-effects-of-se
> > rver
> > .html
> > if you are interested.
> 
> Great analysis! I remember reading through your blog some time ago
> but
> forgot about this post. (Btw it would be even nicer if the code was
> formated [1] ;)
> 
> Cheers,
> 
> 1: https://stackoverflow.com/questions/679189/formatting-code-snippet
> s-for-blogging-on-blogger
> 
Thanks,

I've used hilite.me to generate the html for the code and changed the
blog design. Hopefully will make it easier to follow. 

I still need to change my code walk through of smb3 encryption.

Sachin Prabhu
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/mount.cifs.rst b/mount.cifs.rst
index 023ff34..efd7055 100644
--- a/mount.cifs.rst
+++ b/mount.cifs.rst
@@ -385,6 +385,14 @@  mfsymlinks
   option. Minshall+French symlinks are used even if the server supports
   the CIFS Unix Extensions.

+echo_interval=n
+  sets the interval at which echo requests are sent to the server on an
+  idling connection. This setting also affects the time required
+  for a connection to an unresponsive server to timeout.
+  Here n is the echo interval in seconds. If this option is not given then
+  the default value of 60 seconds is used.
+  The minimum tunable value is 1 second and maximum can go up to 600 seconds.
+
 serverino
   Use inode numbers (unique persistent file identifiers) returned by the
   server instead of automatically generating temporary inode numbers on