RE: [PATCH 4.4 06/28] cifs: Do not send echoes before Negotiate is complete

From	Pavel Shilovskiy <pshilov@microsoft.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH 4.4 06/28] cifs: Do not send echoes before Negotiate is complete
Date	2017-05-25 19:10 +0200
Message-ID	<tL4qR-2vP-7@gated-at.bofh.it> (permalink)
References	<tAagh-6jY-5@gated-at.bofh.it> <tAaq0-6oY-73@gated-at.bofh.it> <tFchs-1AL-25@gated-at.bofh.it> <tFoiC-1gH-5@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tu, May 9, 2017 6:01 PM, Pavel Shilovskiy wrote:
> On Tue, May 9, 2017 5:13 AM, Ben Hutchings wrote:
> > > commit 4fcd1813e640 ("Fix reconnect to not defer smb3 session 
> > > reconnect long after socket reconnect") added support for Negotiate 
> > > requests to be initiated by echo calls.
> > > 
> > > To avoid delays in calling echo after a reconnect, I added the patch 
> > > introduced by the commit b8c600120fc8 ("Call echo service 
> > > immediately after socket reconnect").
> > 
> > The second commit hasn't actually been applied to any stable branches (so this one didn't need to be).  Should it be?
> 
> Yes, 2nd commit hasn't been applied to stable branches, but the proposed fix does the right thing anyway since it
> doesn't allow to call echo on connections before negotiate phase. The commit b8c600120fc8 ("Call echo service 
> immediately after socket reconnect") just allows to trigger the wrong behavior easily. So, I think it is better to have the 
> fix in stable branches.

The commit b8c600120fc8 ("Call echo service immediately after socket reconnect") does the right thing and fixes the problem with persistent handles reconnect, so, I suggest to take it to stable as well. I've just sent the backported version of the patch to stable@.

Best regards,
Pavel Shilovsky

Back to linux.kernel | Previous | Next | Find similar | Unroll thread

Thread

RE: [PATCH 4.4 06/28] cifs: Do not send echoes before Negotiate is  complete Pavel Shilovskiy <pshilov@microsoft.com> - 2017-05-25 19:10 +0200

csiph-web