Linux CIFS filesystem development
 help / color / mirror / Atom feed
From: Jeff Layton <jlayton-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Pavel Shilovsky <piastryyy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: linux-cifs <linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"Ronnie Sahlberg"
	<lsahlber-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	"Aurélien Aptel" <aaptel-IBi9RG/b67k@public.gmane.org>,
	"Pavel Shilovskiy"
	<pshilov-0li6OtcxBFHby3iVrkZq2A@public.gmane.org>
Subject: Re: [cifs-utils PATCH v2] manpage: update mount.cifs manpage with info about default version being mounted
Date: Sat, 04 Nov 2017 07:33:05 -0400	[thread overview]
Message-ID: <1509795185.29302.5.camel@kernel.org> (raw)
In-Reply-To: <CAKywueR4DpsM72EK6u=6PHRm+uD6m8vJWnmHfhYzUuYfFn1oNA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On Tue, 2017-10-31 at 10:45 -0700, Pavel Shilovsky wrote:
> 2017-10-31 9:02 GMT-07:00 Jeff Layton <jlayton-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>:
> > From: Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
> > 
> > Signed-off-by: Jeff Layton <jlayton-eUNUBHrolfbYtjvyW6yDsg@public.gmane.org>
> > ---
> >  mount.cifs.rst | 6 ++++--
> >  1 file changed, 4 insertions(+), 2 deletions(-)
> > 
> > Thanks Aurelien. I dropped Ronnie's Reviewed-by since this is a bit different.
> > Let me know if you'd like me to add it back.
> > 
> > diff --git a/mount.cifs.rst b/mount.cifs.rst
> > index 9714f795fe76..01305df60651 100644
> > --- a/mount.cifs.rst
> > +++ b/mount.cifs.rst
> > @@ -517,7 +517,7 @@ prefixpath=arg
> >  vers=arg
> >    SMB protocol version. Allowed values are:
> > 
> > -  - 1.0 - The classic CIFS/SMBv1 protocol. This is the default.
> > +  - 1.0 - The classic CIFS/SMBv1 protocol.
> >    - 2.0 - The SMBv2.002 protocol. This was initially introduced in
> >      Windows Vista Service Pack 1, and Windows Server 2008. Note that
> >      the initial release version of Windows Vista spoke a slightly
> > @@ -529,6 +529,9 @@ vers=arg
> >    Note too that while this option governs the protocol version used, not
> >    all features of each version are available.
> > 
> > +  In kernels v4.13 and later, the client and server negotiate a version greater
> > +  than or equal to ``2.1``. In kernels prior to v4.13, the default is ``1.0``.
> > +
> 
> The multi-dialect negotiation (version 2.1 and above) was added in
> v4.13.5 kernel. Should we mention the specific stable version in the
> doc?
> 

It would be very odd to make that sort of change in a stable series
kernel. What patch actually flipped the default to autonegotiate?

I was assuming that it was eef914a9eb5eb83e60eb498315a491cd1edc13a1:

    $ git tag --contains eef914a9eb5eb83e60eb498315a491cd1edc13a1 | grep v4.13$
    v4.13

git says that that commit went into v4.13. Was it a different one?
-- 
Jeff Layton <jlayton-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>

  parent reply	other threads:[~2017-11-04 11:33 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-31 16:02 [cifs-utils PATCH v2] manpage: update mount.cifs manpage with info about default version being mounted Jeff Layton
     [not found] ` <20171031160224.12523-1-jlayton-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-10-31 17:45   ` Pavel Shilovsky
     [not found]     ` <CAKywueR4DpsM72EK6u=6PHRm+uD6m8vJWnmHfhYzUuYfFn1oNA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-11-04 11:33       ` Jeff Layton [this message]
     [not found]         ` <1509795185.29302.5.camel-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2017-11-04 16:44           ` Pavel Shilovsky

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1509795185.29302.5.camel@kernel.org \
    --to=jlayton-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=aaptel-IBi9RG/b67k@public.gmane.org \
    --cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=lsahlber-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=piastryyy-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=pshilov-0li6OtcxBFHby3iVrkZq2A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox