diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2008-06-11 13:26:23 -0400 |
---|---|---|
committer | Trond Myklebust <Trond.Myklebust@netapp.com> | 2008-07-15 18:10:53 -0400 |
commit | bba67e0e3f4caba2b2b90b48ed798fb0461bcb86 (patch) | |
tree | c13781f1265ccd73b4f71a33e9bb22931e954c1e /fs/nfs/dir.c | |
parent | b6a2e569e2157509951c9f3f58dfa18b44ce91b3 (diff) |
NFS: Remove BKL usage from open()
All the NFSv4 stateful operations are already protected by other locks (in
particular by the rpc_sequence locks.
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Diffstat (limited to 'fs/nfs/dir.c')
-rw-r--r-- | fs/nfs/dir.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c index 73e0f9740dd1..c68ec447ace1 100644 --- a/fs/nfs/dir.c +++ b/fs/nfs/dir.c | |||
@@ -139,10 +139,8 @@ nfs_opendir(struct inode *inode, struct file *filp) | |||
139 | 139 | ||
140 | nfs_inc_stats(inode, NFSIOS_VFSOPEN); | 140 | nfs_inc_stats(inode, NFSIOS_VFSOPEN); |
141 | 141 | ||
142 | lock_kernel(); | ||
143 | /* Call generic open code in order to cache credentials */ | 142 | /* Call generic open code in order to cache credentials */ |
144 | res = nfs_open(inode, filp); | 143 | res = nfs_open(inode, filp); |
145 | unlock_kernel(); | ||
146 | return res; | 144 | return res; |
147 | } | 145 | } |
148 | 146 | ||
@@ -1019,9 +1017,7 @@ static struct dentry *nfs_atomic_lookup(struct inode *dir, struct dentry *dentry | |||
1019 | } | 1017 | } |
1020 | 1018 | ||
1021 | /* Open the file on the server */ | 1019 | /* Open the file on the server */ |
1022 | lock_kernel(); | ||
1023 | res = nfs4_atomic_open(dir, dentry, nd); | 1020 | res = nfs4_atomic_open(dir, dentry, nd); |
1024 | unlock_kernel(); | ||
1025 | if (IS_ERR(res)) { | 1021 | if (IS_ERR(res)) { |
1026 | error = PTR_ERR(res); | 1022 | error = PTR_ERR(res); |
1027 | switch (error) { | 1023 | switch (error) { |
@@ -1083,9 +1079,7 @@ static int nfs_open_revalidate(struct dentry *dentry, struct nameidata *nd) | |||
1083 | * operations that change the directory. We therefore save the | 1079 | * operations that change the directory. We therefore save the |
1084 | * change attribute *before* we do the RPC call. | 1080 | * change attribute *before* we do the RPC call. |
1085 | */ | 1081 | */ |
1086 | lock_kernel(); | ||
1087 | ret = nfs4_open_revalidate(dir, dentry, openflags, nd); | 1082 | ret = nfs4_open_revalidate(dir, dentry, openflags, nd); |
1088 | unlock_kernel(); | ||
1089 | out: | 1083 | out: |
1090 | dput(parent); | 1084 | dput(parent); |
1091 | if (!ret) | 1085 | if (!ret) |