* [PATCH V1 Resend 07/10] fs: Drop unlikely before IS_ERR(_OR_NULL)
[not found] <cover.1439375087.git.viresh.kumar@linaro.org>
@ 2015-08-12 10:29 ` Viresh Kumar
[not found] ` <20150812142815.27968b32@tlielax.poochiereds.net>
0 siblings, 1 reply; 3+ messages in thread
From: Viresh Kumar @ 2015-08-12 10:29 UTC (permalink / raw)
To: trivial
Cc: linaro-kernel, linux-kernel, Viresh Kumar, David Howells,
Alexander Viro, Andreas Dilger, Andrew Morton, Anna Schumaker,
Boaz Harrosh, Christoph Hellwig, open list:ECRYPT FILE SYSTEM,
Eric W. Biederman, Fabian Frederick,
open list:COMMON INTERNET FILE SYSTEM (CIFS),
open list:EXT4 FILE SYSTEM,
open list:FILESYSTEMS (VFS and infrastructure),
open list:NFS, SUNRPC, AND LOCKD CLIENTS, Peng Tao,
Petr Vandrovec, moderated list:COMMON INTERNET FILE SYSTEM (CIFS),
Steve French, Theodore Ts'o, Tom Haynes, Trond Myklebust,
Tyler Hicks, Weston Andros Adamson
IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag and there
is no need to do that again from its callers. Drop it.
Reviewed-by: David Howells <dhowells@redhat.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
fs/cifs/readdir.c | 2 +-
fs/ecryptfs/inode.c | 2 +-
fs/ext4/extents.c | 6 +++---
fs/ext4/namei.c | 2 +-
fs/namei.c | 4 ++--
fs/ncpfs/dir.c | 2 +-
fs/nfs/objlayout/objio_osd.c | 2 +-
fs/proc/proc_sysctl.c | 2 +-
8 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/fs/cifs/readdir.c b/fs/cifs/readdir.c
index b1eede3678a9..0557c45e9c33 100644
--- a/fs/cifs/readdir.c
+++ b/fs/cifs/readdir.c
@@ -84,7 +84,7 @@ cifs_prime_dcache(struct dentry *parent, struct qstr *name,
cifs_dbg(FYI, "%s: for %s\n", __func__, name->name);
dentry = d_hash_and_lookup(parent, name);
- if (unlikely(IS_ERR(dentry)))
+ if (IS_ERR(dentry))
return;
if (dentry) {
diff --git a/fs/ecryptfs/inode.c b/fs/ecryptfs/inode.c
index 3c4db1172d22..e2e47ba5d313 100644
--- a/fs/ecryptfs/inode.c
+++ b/fs/ecryptfs/inode.c
@@ -270,7 +270,7 @@ ecryptfs_create(struct inode *directory_inode, struct dentry *ecryptfs_dentry,
ecryptfs_inode = ecryptfs_do_create(directory_inode, ecryptfs_dentry,
mode);
- if (unlikely(IS_ERR(ecryptfs_inode))) {
+ if (IS_ERR(ecryptfs_inode)) {
ecryptfs_printk(KERN_WARNING, "Failed to create file in"
"lower filesystem\n");
rc = PTR_ERR(ecryptfs_inode);
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
index 2553aa8b608d..799f01714767 100644
--- a/fs/ext4/extents.c
+++ b/fs/ext4/extents.c
@@ -899,7 +899,7 @@ ext4_find_extent(struct inode *inode, ext4_lblk_t block,
bh = read_extent_tree_block(inode, path[ppos].p_block, --i,
flags);
- if (unlikely(IS_ERR(bh))) {
+ if (IS_ERR(bh)) {
ret = PTR_ERR(bh);
goto err;
}
@@ -5792,7 +5792,7 @@ ext4_swap_extents(handle_t *handle, struct inode *inode1,
int split = 0;
path1 = ext4_find_extent(inode1, lblk1, NULL, EXT4_EX_NOCACHE);
- if (unlikely(IS_ERR(path1))) {
+ if (IS_ERR(path1)) {
*erp = PTR_ERR(path1);
path1 = NULL;
finish:
@@ -5800,7 +5800,7 @@ ext4_swap_extents(handle_t *handle, struct inode *inode1,
goto repeat;
}
path2 = ext4_find_extent(inode2, lblk2, NULL, EXT4_EX_NOCACHE);
- if (unlikely(IS_ERR(path2))) {
+ if (IS_ERR(path2)) {
*erp = PTR_ERR(path2);
path2 = NULL;
goto finish;
diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
index 9f61e7679a6d..f12b277ad2af 100644
--- a/fs/ext4/namei.c
+++ b/fs/ext4/namei.c
@@ -1429,7 +1429,7 @@ static struct buffer_head * ext4_find_entry (struct inode *dir,
}
num++;
bh = ext4_getblk(NULL, dir, b++, 0);
- if (unlikely(IS_ERR(bh))) {
+ if (IS_ERR(bh)) {
if (ra_max == 0) {
ret = bh;
goto cleanup_and_exit;
diff --git a/fs/namei.c b/fs/namei.c
index cacc51ddcf54..67d419e55547 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -1942,7 +1942,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
if (err) {
const char *s = get_link(nd);
- if (unlikely(IS_ERR(s)))
+ if (IS_ERR(s))
return PTR_ERR(s);
err = 0;
if (unlikely(!s)) {
@@ -3430,7 +3430,7 @@ struct file *do_file_open_root(struct dentry *dentry, struct vfsmount *mnt,
return ERR_PTR(-ELOOP);
filename = getname_kernel(name);
- if (unlikely(IS_ERR(filename)))
+ if (IS_ERR(filename))
return ERR_CAST(filename);
set_nameidata(&nd, -1, filename);
diff --git a/fs/ncpfs/dir.c b/fs/ncpfs/dir.c
index 93575e91a7aa..356816e7bc90 100644
--- a/fs/ncpfs/dir.c
+++ b/fs/ncpfs/dir.c
@@ -597,7 +597,7 @@ ncp_fill_cache(struct file *file, struct dir_context *ctx,
qname.name = __name;
newdent = d_hash_and_lookup(dentry, &qname);
- if (unlikely(IS_ERR(newdent)))
+ if (IS_ERR(newdent))
goto end_advance;
if (!newdent) {
newdent = d_alloc(dentry, &qname);
diff --git a/fs/nfs/objlayout/objio_osd.c b/fs/nfs/objlayout/objio_osd.c
index 5aaed363556a..5c0c6b58157f 100644
--- a/fs/nfs/objlayout/objio_osd.c
+++ b/fs/nfs/objlayout/objio_osd.c
@@ -124,7 +124,7 @@ objio_alloc_deviceid_node(struct nfs_server *server, struct pnfs_device *pdev,
retry_lookup:
od = osduld_info_lookup(&odi);
- if (unlikely(IS_ERR(od))) {
+ if (IS_ERR(od)) {
err = PTR_ERR(od);
dprintk("%s: osduld_info_lookup => %d\n", __func__, err);
if (err == -ENODEV && retry_flag) {
diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index fdda62e6115e..fe5b6e6c4671 100644
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -948,7 +948,7 @@ static struct ctl_dir *get_subdir(struct ctl_dir *dir,
found:
subdir->header.nreg++;
failed:
- if (unlikely(IS_ERR(subdir))) {
+ if (IS_ERR(subdir)) {
pr_err("sysctl could not get directory: ");
sysctl_print_dir(dir);
pr_cont("/%*.*s %ld\n",
--
2.4.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V1 Resend 07/10] fs: Drop unlikely before IS_ERR(_OR_NULL)
[not found] ` <20150812142815.27968b32@tlielax.poochiereds.net>
@ 2015-08-20 16:08 ` Steve French
2015-08-20 16:14 ` Viresh Kumar
0 siblings, 1 reply; 3+ messages in thread
From: Steve French @ 2015-08-20 16:08 UTC (permalink / raw)
To: Jeff Layton
Cc: Viresh Kumar, Boaz Harrosh, linux-nfs, linaro-kernel, Jiri Kosina,
open list:COMMON INTERNET FILE SYSTEM CIFS,
open list:ECRYPT FILE SYSTEM, LKML, Christoph Hellwig,
David Howells, open list:FILESYSTEMS VFS and infrastructure,
Andreas Dilger, Alexander Viro, Fabian Frederick, Andrew Morton,
open list:EXT4 FILE SYSTEM, Anna Schumaker, Eric W. Biederman
On Wed, Aug 12, 2015 at 1:28 PM, Jeff Layton <jlayton@poochiereds.net> wrote:
> On Wed, 12 Aug 2015 15:59:44 +0530
> Viresh Kumar <viresh.kumar@linaro.org> wrote:
>
>> IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag and there
>> is no need to do that again from its callers. Drop it.
>>
>> Reviewed-by: David Howells <dhowells@redhat.com>
>> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
>> ---
>> fs/cifs/readdir.c | 2 +-
>> fs/ecryptfs/inode.c | 2 +-
>> fs/ext4/extents.c | 6 +++---
>> fs/ext4/namei.c | 2 +-
>> fs/namei.c | 4 ++--
>> fs/ncpfs/dir.c | 2 +-
>> fs/nfs/objlayout/objio_osd.c | 2 +-
>> fs/proc/proc_sysctl.c | 2 +-
>> 8 files changed, 11 insertions(+), 11 deletions(-)
>>
>> diff --git a/fs/cifs/readdir.c b/fs/cifs/readdir.c
>> index b1eede3678a9..0557c45e9c33 100644
>> --- a/fs/cifs/readdir.c
>> +++ b/fs/cifs/readdir.c
>> @@ -84,7 +84,7 @@ cifs_prime_dcache(struct dentry *parent, struct qstr *name,
>> cifs_dbg(FYI, "%s: for %s\n", __func__, name->name);
>>
>> dentry = d_hash_and_lookup(parent, name);
>> - if (unlikely(IS_ERR(dentry)))
>> + if (IS_ERR(dentry))
>> return;
>>
>> if (dentry) {
>> diff --git a/fs/ecryptfs/inode.c b/fs/ecryptfs/inode.c
>> index 3c4db1172d22..e2e47ba5d313 100644
>> --- a/fs/ecryptfs/inode.c
>> +++ b/fs/ecryptfs/inode.c
>> @@ -270,7 +270,7 @@ ecryptfs_create(struct inode *directory_inode, struct dentry *ecryptfs_dentry,
>>
>> ecryptfs_inode = ecryptfs_do_create(directory_inode, ecryptfs_dentry,
>> mode);
>> - if (unlikely(IS_ERR(ecryptfs_inode))) {
>> + if (IS_ERR(ecryptfs_inode)) {
>> ecryptfs_printk(KERN_WARNING, "Failed to create file in"
>> "lower filesystem\n");
>> rc = PTR_ERR(ecryptfs_inode);
>> diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
>> index 2553aa8b608d..799f01714767 100644
>> --- a/fs/ext4/extents.c
>> +++ b/fs/ext4/extents.c
>> @@ -899,7 +899,7 @@ ext4_find_extent(struct inode *inode, ext4_lblk_t block,
>>
>> bh = read_extent_tree_block(inode, path[ppos].p_block, --i,
>> flags);
>> - if (unlikely(IS_ERR(bh))) {
>> + if (IS_ERR(bh)) {
>> ret = PTR_ERR(bh);
>> goto err;
>> }
>> @@ -5792,7 +5792,7 @@ ext4_swap_extents(handle_t *handle, struct inode *inode1,
>> int split = 0;
>>
>> path1 = ext4_find_extent(inode1, lblk1, NULL, EXT4_EX_NOCACHE);
>> - if (unlikely(IS_ERR(path1))) {
>> + if (IS_ERR(path1)) {
>> *erp = PTR_ERR(path1);
>> path1 = NULL;
>> finish:
>> @@ -5800,7 +5800,7 @@ ext4_swap_extents(handle_t *handle, struct inode *inode1,
>> goto repeat;
>> }
>> path2 = ext4_find_extent(inode2, lblk2, NULL, EXT4_EX_NOCACHE);
>> - if (unlikely(IS_ERR(path2))) {
>> + if (IS_ERR(path2)) {
>> *erp = PTR_ERR(path2);
>> path2 = NULL;
>> goto finish;
>> diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c
>> index 9f61e7679a6d..f12b277ad2af 100644
>> --- a/fs/ext4/namei.c
>> +++ b/fs/ext4/namei.c
>> @@ -1429,7 +1429,7 @@ static struct buffer_head * ext4_find_entry (struct inode *dir,
>> }
>> num++;
>> bh = ext4_getblk(NULL, dir, b++, 0);
>> - if (unlikely(IS_ERR(bh))) {
>> + if (IS_ERR(bh)) {
>> if (ra_max == 0) {
>> ret = bh;
>> goto cleanup_and_exit;
>> diff --git a/fs/namei.c b/fs/namei.c
>> index cacc51ddcf54..67d419e55547 100644
>> --- a/fs/namei.c
>> +++ b/fs/namei.c
>> @@ -1942,7 +1942,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
>> if (err) {
>> const char *s = get_link(nd);
>>
>> - if (unlikely(IS_ERR(s)))
>> + if (IS_ERR(s))
>> return PTR_ERR(s);
>> err = 0;
>> if (unlikely(!s)) {
>> @@ -3430,7 +3430,7 @@ struct file *do_file_open_root(struct dentry *dentry, struct vfsmount *mnt,
>> return ERR_PTR(-ELOOP);
>>
>> filename = getname_kernel(name);
>> - if (unlikely(IS_ERR(filename)))
>> + if (IS_ERR(filename))
>> return ERR_CAST(filename);
>>
>> set_nameidata(&nd, -1, filename);
>> diff --git a/fs/ncpfs/dir.c b/fs/ncpfs/dir.c
>> index 93575e91a7aa..356816e7bc90 100644
>> --- a/fs/ncpfs/dir.c
>> +++ b/fs/ncpfs/dir.c
>> @@ -597,7 +597,7 @@ ncp_fill_cache(struct file *file, struct dir_context *ctx,
>> qname.name = __name;
>>
>> newdent = d_hash_and_lookup(dentry, &qname);
>> - if (unlikely(IS_ERR(newdent)))
>> + if (IS_ERR(newdent))
>> goto end_advance;
>> if (!newdent) {
>> newdent = d_alloc(dentry, &qname);
>> diff --git a/fs/nfs/objlayout/objio_osd.c b/fs/nfs/objlayout/objio_osd.c
>> index 5aaed363556a..5c0c6b58157f 100644
>> --- a/fs/nfs/objlayout/objio_osd.c
>> +++ b/fs/nfs/objlayout/objio_osd.c
>> @@ -124,7 +124,7 @@ objio_alloc_deviceid_node(struct nfs_server *server, struct pnfs_device *pdev,
>>
>> retry_lookup:
>> od = osduld_info_lookup(&odi);
>> - if (unlikely(IS_ERR(od))) {
>> + if (IS_ERR(od)) {
>> err = PTR_ERR(od);
>> dprintk("%s: osduld_info_lookup => %d\n", __func__, err);
>> if (err == -ENODEV && retry_flag) {
>> diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
>> index fdda62e6115e..fe5b6e6c4671 100644
>> --- a/fs/proc/proc_sysctl.c
>> +++ b/fs/proc/proc_sysctl.c
>> @@ -948,7 +948,7 @@ static struct ctl_dir *get_subdir(struct ctl_dir *dir,
>> found:
>> subdir->header.nreg++;
>> failed:
>> - if (unlikely(IS_ERR(subdir))) {
>> + if (IS_ERR(subdir)) {
>> pr_err("sysctl could not get directory: ");
>> sysctl_print_dir(dir);
>> pr_cont("/%*.*s %ld\n",
>
> Looks fine to me. You can add:
>
> Reviewed-by: Jeff Layton <jlayton@poochiereds.net>
>
> ...but the problem is that you have a patch here that touches several
> different filesystems. You'll have to find someone willing to take it
> into some overarching tree (Al's perhaps, or maybe Andrew's?), or break
> it up into several per-fs patches and shepherd it into different
> maintainers' trees.
Reviewed-by: Steve French <smfrench@gmail.com>
Do you want me to merge the fs/cifs part of this independently (into
cifs-2.6.git staging tree) or are you going to merge all of this
(which changes multiple fs) at one time through someone else's tree?
--
Thanks,
Steve
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH V1 Resend 07/10] fs: Drop unlikely before IS_ERR(_OR_NULL)
2015-08-20 16:08 ` Steve French
@ 2015-08-20 16:14 ` Viresh Kumar
0 siblings, 0 replies; 3+ messages in thread
From: Viresh Kumar @ 2015-08-20 16:14 UTC (permalink / raw)
To: Steve French
Cc: Jeff Layton, Boaz Harrosh,
"open list:NFS, Peng Tao <tao.peng@primarydata.com>, Petr Vandrovec <petr@vandrovec.name>, samba-technical@lists.samba.org moderated list:COMMON INTERNET FILE SYSTEM CIFS, Steve French <sfrench@samba.org>, Theodore Ts'o <tytso@mit.edu>, Tom Haynes <loghyr@primarydata.com>, Trond Myklebust <trond.myklebust@primarydata.com>, Tyler Hicks <tyhicks@canonical.com>, Weston Andros Adamson <dros@primarydata.com>",
linux-nfs, linaro-kernel, Jiri Kosina,
open list:COMMON INTERNET FILE SYSTEM CIFS,
open list:ECRYPT FILE SYSTEM, LKML, Christoph Hellwig,
David Howells, open list:FILESYSTEMS VFS and infrastructure,
Andreas Dilger, Alexander Viro, Fabian Frederick, Andrew Morton,
open list:EXT4 FILE SYSTEM, Anna Schumaker, Eric W. Biederman
On 20-08-15, 11:08, Steve French wrote:
> Reviewed-by: Steve French <smfrench@gmail.com>
>
> Do you want me to merge the fs/cifs part of this independently (into
> cifs-2.6.git staging tree) or are you going to merge all of this
> (which changes multiple fs) at one time through someone else's tree?
The plan is to merge via Trivial tree, but the Maintainer isn't
responding yet :)
--
viresh
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-08-20 16:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1439375087.git.viresh.kumar@linaro.org>
2015-08-12 10:29 ` [PATCH V1 Resend 07/10] fs: Drop unlikely before IS_ERR(_OR_NULL) Viresh Kumar
[not found] ` <20150812142815.27968b32@tlielax.poochiereds.net>
2015-08-20 16:08 ` Steve French
2015-08-20 16:14 ` Viresh Kumar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).