diff options
| author | Steve French <sfrench@us.ibm.com> | 2008-03-14 22:37:16 +0000 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2008-03-14 22:37:16 +0000 |
| commit | 8b1327f6ed957030a64ccdb17131955bfea2d3fe (patch) | |
| tree | 075ff00b4c333ef333aff5927eac45bde16d8d53 /fs/cifs/link.c | |
| parent | ebe8912be214662c8289977fb416c1f015df4a0b (diff) | |
| download | kernel_samsung_smdk4412-8b1327f6ed957030a64ccdb17131955bfea2d3fe.zip kernel_samsung_smdk4412-8b1327f6ed957030a64ccdb17131955bfea2d3fe.tar.gz kernel_samsung_smdk4412-8b1327f6ed957030a64ccdb17131955bfea2d3fe.tar.bz2 | |
[CIFS] file create with acl support enabled is slow
Shirish Pargaonkar noted:
With cifsacl mount option, when a file is created on the Windows server,
exclusive oplock is broken right away because the get cifs acl code
again opens the file to obtain security descriptor.
The client does not have the newly created file handle or inode in any
of its lists yet so it does not respond to oplock break and server waits for
its duration and then responds to the second open. This slows down file
creation signficantly. The fix is to pass the file descriptor to the get
cifsacl code wherever available so that get cifs acl code does not send
second open (NT Create ANDX) and oplock is not broken.
CC: Shirish Pargaonkar <shirishp@us.ibm.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/link.c')
| -rw-r--r-- | fs/cifs/link.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/cifs/link.c b/fs/cifs/link.c index 1d6fb01..d4e7ec9 100644 --- a/fs/cifs/link.c +++ b/fs/cifs/link.c @@ -205,7 +205,7 @@ cifs_symlink(struct inode *inode, struct dentry *direntry, const char *symname) inode->i_sb, xid); else rc = cifs_get_inode_info(&newinode, full_path, NULL, - inode->i_sb, xid); + inode->i_sb, xid, NULL); if (rc != 0) { cFYI(1, ("Create symlink ok, getinodeinfo fail rc = %d", |