[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <DM5PR03MB25539E2E1489E6C11D7063A5B6FF0@DM5PR03MB2553.namprd03.prod.outlook.com>
Date: Thu, 25 May 2017 16:59:52 +0000
From: Pavel Shilovskiy <pshilov@...rosoft.com>
To: Ben Hutchings <ben.hutchings@...ethink.co.uk>,
Sachin Prabhu <sprabhu@...hat.com>,
Steve French <smfrench@...il.com>
CC: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"stable@...r.kernel.org" <stable@...r.kernel.org>,
Jonathan Liu <net147@...il.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: RE: [PATCH 4.4 06/28] cifs: Do not send echoes before Negotiate is
complete
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
Powered by blists - more mailing lists