| Message ID | 20181110172746.22254-1-kdsouza@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | Update mount.cifs with vers=default mount option. | expand |
> -----Original Message----- > From: linux-cifs-owner@vger.kernel.org <linux-cifs-owner@vger.kernel.org> On > Behalf Of Kenneth D'souza > Sent: Saturday, November 10, 2018 9:28 AM > To: linux-cifs@vger.kernel.org > Cc: piastryyy@gmail.com; smfrench@gmail.com > Subject: [PATCH] Update mount.cifs with vers=default mount option. > > Signed-off-by: Kenneth D'souza <kdsouza@redhat.com> > --- > mount.cifs.rst | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/mount.cifs.rst b/mount.cifs.rst > index 405c459..19f9956 100644 > --- a/mount.cifs.rst > +++ b/mount.cifs.rst > @@ -543,6 +543,10 @@ vers=arg > - 2.1 - The SMBv2.1 protocol that was introduced in Microsoft Windows 7 > and Windows Server 2008R2. > - 3.0 - The SMBv3.0 protocol that was introduced in Microsoft Windows 8 > and Windows Server 2012. Shouldn't the 3.0.2 dialect be mentioned here? > - 3.1.1 or 3.11 - The SMBv3.1.1 protocol that was introduced in Microsoft > Windows Server 2016. > + - default - Tries to negotiate the highest possible version supported by the > server. The "highest" version is not universally the case, where SMB1 is involved. It would be safer to say "highest SMB2+ version" perhaps. > + > + If no dialect is specified on mount vers=default is used. > + To check ``Dialect`` refer to /proc/fs/cifs/DebugData > > Note too that while this option governs the protocol version used, not > all features of each version are available. > -- > 2.17.2
On Mon, Nov 12, 2018 at 9:14 PM, Tom Talpey <ttalpey@microsoft.com> wrote: >> -----Original Message----- >> From: linux-cifs-owner@vger.kernel.org <linux-cifs-owner@vger.kernel.org> On >> Behalf Of Kenneth D'souza >> Sent: Saturday, November 10, 2018 9:28 AM >> To: linux-cifs@vger.kernel.org >> Cc: piastryyy@gmail.com; smfrench@gmail.com >> Subject: [PATCH] Update mount.cifs with vers=default mount option. >> >> Signed-off-by: Kenneth D'souza <kdsouza@redhat.com> >> --- >> mount.cifs.rst | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/mount.cifs.rst b/mount.cifs.rst >> index 405c459..19f9956 100644 >> --- a/mount.cifs.rst >> +++ b/mount.cifs.rst >> @@ -543,6 +543,10 @@ vers=arg >> - 2.1 - The SMBv2.1 protocol that was introduced in Microsoft Windows 7 >> and Windows Server 2008R2. >> - 3.0 - The SMBv3.0 protocol that was introduced in Microsoft Windows 8 >> and Windows Server 2012. > > Shouldn't the 3.0.2 dialect be mentioned here? > I will mention the same in V2. Something like the below ? 3.02 - The SMBv3.02 protocol that was introduced in Windows 8.1 (Windows 'Blue') and Windows Server 2012 R2. >> - 3.1.1 or 3.11 - The SMBv3.1.1 protocol that was introduced in Microsoft >> Windows Server 2016. >> + - default - Tries to negotiate the highest possible version supported by the >> server. > > The "highest" version is not universally the case, where SMB1 is involved. It would > be safer to say "highest SMB2+ version" perhaps. > Agreed, will update the same. >> + >> + If no dialect is specified on mount vers=default is used. >> + To check ``Dialect`` refer to /proc/fs/cifs/DebugData >> >> Note too that while this option governs the protocol version used, not >> all features of each version are available. >> -- >> 2.17.2 >
diff --git a/mount.cifs.rst b/mount.cifs.rst index 405c459..19f9956 100644 --- a/mount.cifs.rst +++ b/mount.cifs.rst @@ -543,6 +543,10 @@ vers=arg - 2.1 - The SMBv2.1 protocol that was introduced in Microsoft Windows 7 and Windows Server 2008R2. - 3.0 - The SMBv3.0 protocol that was introduced in Microsoft Windows 8 and Windows Server 2012. - 3.1.1 or 3.11 - The SMBv3.1.1 protocol that was introduced in Microsoft Windows Server 2016. + - default - Tries to negotiate the highest possible version supported by the server. + + If no dialect is specified on mount vers=default is used. + To check ``Dialect`` refer to /proc/fs/cifs/DebugData Note too that while this option governs the protocol version used, not all features of each version are available.
Signed-off-by: Kenneth D'souza <kdsouza@redhat.com> --- mount.cifs.rst | 4 ++++ 1 file changed, 4 insertions(+)