diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2024-01-20 14:20:36 +0300 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2024-04-15 23:03:24 +0300 |
commit | 613aee94dddf07de9fde8dd4fcb6e4579cde8a65 (patch) | |
tree | 183454f99e1471628ae57fa500e326825ec6e5c1 /fs/file.c | |
parent | c4aab26253cd1f302279b8d6b5b66ccf1b120520 (diff) | |
download | linux-613aee94dddf07de9fde8dd4fcb6e4579cde8a65.tar.xz |
get_file_rcu(): no need to check for NULL separately
IS_ERR(NULL) is false and IS_ERR() already comes with unlikely()...
Reviewed-by: Christian Brauner <brauner@kernel.org>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/file.c')
-rw-r--r-- | fs/file.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/fs/file.c b/fs/file.c index ab38b005633c..8076aef9c210 100644 --- a/fs/file.c +++ b/fs/file.c @@ -920,13 +920,8 @@ struct file *get_file_rcu(struct file __rcu **f) struct file __rcu *file; file = __get_file_rcu(f); - if (unlikely(!file)) - return NULL; - - if (unlikely(IS_ERR(file))) - continue; - - return file; + if (!IS_ERR(file)) + return file; } } EXPORT_SYMBOL_GPL(get_file_rcu); |