Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f2a42c6b authored by Steve French's avatar Steve French Committed by Greg Kroah-Hartman
Browse files

smb3: directory sync should not return an error 



commit 6e70c267e68d77679534dcf4aaf84e66f2cf1425 upstream.

As with NFS, which ignores sync on directory handles,
fsync on a directory handle is a noop for CIFS/SMB3.
Do not return an error on it.  It breaks some database
apps otherwise.

Signed-off-by: default avatarSteve French <smfrench@gmail.com>
CC: Stable <stable@vger.kernel.org>
Reviewed-by: default avatarRonnie Sahlberg <lsahlber@redhat.com>
Reviewed-by: default avatarPavel Shilovsky <pshilov@microsoft.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 57e2ce8b

fs/cifs/cifsfs.c

+13 −0
Original line number Diff line number Diff line
@@ -1045,6 +1045,18 @@ ssize_t cifs_file_copychunk_range(unsigned int xid, 
	return rc;

}



/*

 * Directory operations under CIFS/SMB2/SMB3 are synchronous, so fsync()

 * is a dummy operation.

 */

static int cifs_dir_fsync(struct file *file, loff_t start, loff_t end, int datasync)

{

	cifs_dbg(FYI, "Sync directory - name: %pD datasync: 0x%x\n",

		 file, datasync);



	return 0;

}



static ssize_t cifs_copy_file_range(struct file *src_file, loff_t off,

				struct file *dst_file, loff_t destoff,

				size_t len, unsigned int flags)
@@ -1173,6 +1185,7 @@ const struct file_operations cifs_dir_ops = { 
	.copy_file_range = cifs_copy_file_range,

	.clone_file_range = cifs_clone_file_range,

	.llseek = generic_file_llseek,

	.fsync = cifs_dir_fsync,

};



static void