diff options
| author |   Steve French <stfrench@microsoft.com> | 2019-12-02 21:46:54 -0600 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2019-12-03 15:48:02 -0600 |
| commit | 43f8a6a74ee2442b9410ed297f5d4c77e7cb5ace (patch) | |
| tree | 0eabdc6fb77016e544ea32569722114e67b8a6b9 /fs/cifs/smb2inode.c | |
| parent | smb3: remove unused flag passed into close functions (diff) | |
| download | linux-dev-43f8a6a74ee2442b9410ed297f5d4c77e7cb5ace.tar.xz linux-dev-43f8a6a74ee2442b9410ed297f5d4c77e7cb5ace.zip | |
smb3: query attributes on file close
Since timestamps on files on most servers can be updated at
close, and since timestamps on our dentries default to one
second we can have stale timestamps in some common cases
(e.g. open, write, close, stat, wait one second, stat - will
show different mtime for the first and second stat).
The SMB2/SMB3 protocol allows querying timestamps at close
so add the code to request timestamp and attr information
(which is cheap for the server to provide) to be returned
when a file is closed (it is not needed for the many
paths that call SMB2_close that are from compounded
query infos and close nor is it needed for some of
the cases where a directory close immediately follows a
directory open.
Signed-off-by: Steve French <stfrench@microsoft.com>
Acked-by: Ronnie Sahlberg <lsahlber@redhat.com>
Reviewed-by: Aurelien Aptel <aaptel@suse.com>
Reviewed-by: Pavel Shilovsky <pshilov@microsoft.com>
Diffstat (limited to 'fs/cifs/smb2inode.c')
| -rw-r--r-- | fs/cifs/smb2inode.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/smb2inode.c b/fs/cifs/smb2inode.c index 4121ac1163ca..18c7a33adceb 100644 --- a/fs/cifs/smb2inode.c +++ b/fs/cifs/smb2inode.c @@ -313,7 +313,7 @@ smb2_compound_op(const unsigned int xid, struct cifs_tcon *tcon, rqst[num_rqst].rq_iov = close_iov; rqst[num_rqst].rq_nvec = 1; rc = SMB2_close_init(tcon, &rqst[num_rqst], COMPOUND_FID, - COMPOUND_FID); + COMPOUND_FID, false); smb2_set_related(&rqst[num_rqst]); if (rc) goto finished; |