From: Dan Carpenter <dan.carpenter@linaro.org>
Date: Fri, 24 Oct 2025 11:26:00 +0000 (+0300)
Subject: fs/9p: delete unnnecessary condition
X-Git-Tag: v6.19-rc1~60^2~6
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=52df783f33ec6df2d592dd12d0afc3265d9e37eb;p=thirdparty%2Fkernel%2Flinux.git

fs/9p: delete unnnecessary condition

We already know that "retval" is negative, so there is no need to check
again.  Also the statement is not indented far enough.  Delete it.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Fixes: 43c36a56ccf6 ("Revert "fs/9p: Refresh metadata in d_revalidate for uncached mode too"")
Reviewed-by: Christian Schoenebeck <linux_oss@crudebyte.com>
Message-ID: <aPtiSJl8EwSfVvqN@stanley.mountain>
Signed-off-by: Dominique Martinet <asmadeus@codewreck.org>
---

diff --git a/fs/9p/vfs_dentry.c b/fs/9p/vfs_dentry.c
index c1acbc98465de..c5bf74d547e84 100644
--- a/fs/9p/vfs_dentry.c
+++ b/fs/9p/vfs_dentry.c
@@ -109,7 +109,6 @@ static int __v9fs_lookup_revalidate(struct dentry *dentry, unsigned int flags)
 			p9_debug(P9_DEBUG_VFS,
 				"refresh inode: dentry = %pd (%p), got error %pe\n",
 				dentry, dentry, ERR_PTR(retval));
-		if (retval < 0)
 			return retval;
 		}
 	}