diff options
| author | Andrew Morton <akpm@osdl.org> | 2003-07-02 08:47:59 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@home.osdl.org> | 2003-07-02 08:47:59 -0700 |
| commit | 3bd404cf66f024fac6e6071ead0f4032cf1f7c20 (patch) | |
| tree | 7253b8a4a711fed3e4cfe30bfed8687829542f99 /fs/attr.c | |
| parent | c94f7f38722f9d121186196f4b05ca934fccd9f9 (diff) | |
[PATCH] inode_change_ok(): remove lock_kernel()
`attr' is on the stack, and the inode's contents can change as soon as we
return from inode_change_ok() anyway. I can't see anything which is actually
being locked in there.
Diffstat (limited to 'fs/attr.c')
| -rw-r--r-- | fs/attr.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/fs/attr.c b/fs/attr.c index 0d9e778fb5d3..2048b99a112c 100644 --- a/fs/attr.c +++ b/fs/attr.c @@ -22,8 +22,6 @@ int inode_change_ok(struct inode *inode, struct iattr *attr) int retval = -EPERM; unsigned int ia_valid = attr->ia_valid; - lock_kernel(); - /* If force is set do it anyway. */ if (ia_valid & ATTR_FORCE) goto fine; @@ -58,7 +56,6 @@ int inode_change_ok(struct inode *inode, struct iattr *attr) fine: retval = 0; error: - unlock_kernel(); return retval; } |