diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-05-21 08:46:08 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-05-21 08:46:08 -0700 |
commit | 7e5cb5e151c5474b4a468f437f5038ba9f67ef4d (patch) | |
tree | 899f42e88f926fb4501402cf4ee4aa0903421ef5 /fs/stat.c | |
parent | 8c12fec90c54cfabdd57b2726a1defbc07c19a55 (diff) | |
parent | e994defb7b6813ba6fa7a2a36e86d2455ad1dc35 (diff) | |
download | lwn-7e5cb5e151c5474b4a468f437f5038ba9f67ef4d.tar.gz lwn-7e5cb5e151c5474b4a468f437f5038ba9f67ef4d.zip |
Merge branch 'vfs-cleanups' (random vfs cleanups)
This teaches vfs_fstat() to use the appropriate f[get|put]_light
functions, allowing it to avoid some unnecessary locking for the common
case.
More noticeably, it also cleans up and simplifies the "getname_flags()"
function, which now relies on the architecture strncpy_from_user() doing
all the user access checks properly, instead of hacking around the fact
that on x86 it didn't use to do it right (see commit 92ae03f2ef99: "x86:
merge 32/64-bit versions of 'strncpy_from_user()' and speed it up").
* vfs-cleanups:
VFS: make vfs_fstat() use f[get|put]_light()
VFS: clean up and simplify getname_flags()
x86: make word-at-a-time strncpy_from_user clear bytes at the end
Diffstat (limited to 'fs/stat.c')
-rw-r--r-- | fs/stat.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/fs/stat.c b/fs/stat.c index b30ac60291e2..0cef3366a919 100644 --- a/fs/stat.c +++ b/fs/stat.c @@ -57,12 +57,13 @@ EXPORT_SYMBOL(vfs_getattr); int vfs_fstat(unsigned int fd, struct kstat *stat) { - struct file *f = fget(fd); + int fput_needed; + struct file *f = fget_light(fd, &fput_needed); int error = -EBADF; if (f) { error = vfs_getattr(f->f_path.mnt, f->f_path.dentry, stat); - fput(f); + fput_light(f, fput_needed); } return error; } |