ACK: [bionic:linux-azure-4.15][PATCH 0/2] LP:#1882268 - [linux-azure] Request for two CIFS commits in 16.04

[Kleber Souza]

On 26.10.20 15:34, Marcelo Henrique Cerri wrote:
> BugLink: https://bugs.launchpad.net/bugs/1882268
> 
> The commit id 2084ed57167c ("CIFS: Only send SMB2_NEGOTIATE command on
> new TCP connections") is a clean cherry pick from upstream while
> 84a1f5b1cc6f ("cifs: Fix potential softlockups while refreshing DFS
> cache") had to be backported to 4.15.
> 
> Both changes are already included in the 5.4 linux-azure
> kernel. 2084ed57167c was included upstream in 5.1 and 84a1f5b1cc6f was
> applied via an upstream stable update (LP:#1860125). 2084ed57167c
> fixes commit id e99c63e4d86d ("SMB3: Fix deadlock in validate
> negotiate hits reconnect") that was included to our 4.15 kernel also
> via an upstream stable update.
> 
> The original author of 84a1f5b1cc6f reviewed and approved my backport
> for 4.15. Microsoft reported positive results with the test kernel and
> I also smoke tested it using dbench as described by LP:#1896642.
> 
> Since we don't have a direct reproducer but we have positive feedback
> from Microsoft I recommend that we include these fixes only to the
> 4.15 linux-azure kernels for now. Later we can evaluate if we should
> include them to the generic 4.15 kernel too.
> 
> ---
> Paulo Alcantara (SUSE) (1):
>    cifs: Fix potential softlockups while refreshing DFS cache
> 
> Pavel Shilovsky (1):
>    CIFS: Only send SMB2_NEGOTIATE command on new TCP connections
> 
>   fs/cifs/smb2pdu.c       | 38 ++++++++++++++++++++++++++++----------
>   fs/cifs/smb2transport.c |  8 ++++++++
>   2 files changed, 36 insertions(+), 10 deletions(-)
> 


Good test results from MS.

Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>