Is this mutex really needed in cifs_fsync? (It got moved down from the vfs
with a global change to the vfs fsync for btrfs in
02c24a82187d5a628c68edfe71ae60dc135cd178). Don't see a problem with
sending multiple flushes in parallel on the wire.

        mutex_lock(&inode->i_mutex);

        xid = GetXid();

        cFYI(1, "Sync file - name: %s datasync: 0x%x",
                file->f_path.dentry->d_name.name, datasync);

        tcon = tlink_tcon(smbfile->tlink);
        if (!(cifs_sb->mnt_cifs_flags & CIFS_MOUNT_NOSSYNC))
                rc = CIFSSMBFlush(xid, tcon, smbfile->netfid);

        FreeXid(xid);
        mutex_unlock(&inode->i_mutex);


-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to