mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-11-01 17:08:10 +00:00
Adjust cifssb maximum read size
[ Upstream commit 06a466565d
]
When session gets reconnected during mount then read size in super block fs context
gets set to zero and after negotiate, rsize is not modified which results in
incorrect read with requested bytes as zero. Fixes intermittent failure
of xfstest generic/240
Note that stable requires a different version of this patch which will be
sent to the stable mailing list.
Signed-off-by: Rohith Surabattula <rohiths@microsoft.com>
Acked-by: Paulo Alcantara (SUSE) <pc@cjr.nz>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
1a55c48bba
commit
9dd6bb11df
2 changed files with 13 additions and 0 deletions
|
@ -210,6 +210,9 @@ cifs_read_super(struct super_block *sb)
|
|||
if (rc)
|
||||
goto out_no_root;
|
||||
/* tune readahead according to rsize if readahead size not set on mount */
|
||||
if (cifs_sb->ctx->rsize == 0)
|
||||
cifs_sb->ctx->rsize =
|
||||
tcon->ses->server->ops->negotiate_rsize(tcon, cifs_sb->ctx);
|
||||
if (cifs_sb->ctx->rasize)
|
||||
sb->s_bdi->ra_pages = cifs_sb->ctx->rasize / PAGE_SIZE;
|
||||
else
|
||||
|
|
|
@ -3734,6 +3734,11 @@ cifs_send_async_read(loff_t offset, size_t len, struct cifsFileInfo *open_file,
|
|||
break;
|
||||
}
|
||||
|
||||
if (cifs_sb->ctx->rsize == 0)
|
||||
cifs_sb->ctx->rsize =
|
||||
server->ops->negotiate_rsize(tlink_tcon(open_file->tlink),
|
||||
cifs_sb->ctx);
|
||||
|
||||
rc = server->ops->wait_mtu_credits(server, cifs_sb->ctx->rsize,
|
||||
&rsize, credits);
|
||||
if (rc)
|
||||
|
@ -4512,6 +4517,11 @@ static int cifs_readpages(struct file *file, struct address_space *mapping,
|
|||
break;
|
||||
}
|
||||
|
||||
if (cifs_sb->ctx->rsize == 0)
|
||||
cifs_sb->ctx->rsize =
|
||||
server->ops->negotiate_rsize(tlink_tcon(open_file->tlink),
|
||||
cifs_sb->ctx);
|
||||
|
||||
rc = server->ops->wait_mtu_credits(server, cifs_sb->ctx->rsize,
|
||||
&rsize, credits);
|
||||
if (rc)
|
||||
|
|
Loading…
Reference in a new issue