fs: 9p: remove unneeded variable

Changcheng Deng · martinetd · commit 90d6cf349c56 · 2021-12-18T21:04:50.000+09:00
Fix the following coccicheck review: ./fs/9p/vfs_file.c: 117: 5-8: Unneeded variable Remove unneeded variable used to store return value. Link: https://lkml.kernel.org/r/20211109114343.132844-1-deng.changcheng@zte.com.cn Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Changcheng Deng <deng.changcheng@zte.com.cn> Signed-off-by: Dominique Martinet <asmadeus@codewreck.org>
diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
@@ -114,7 +114,6 @@ int v9fs_file_open(struct inode *inode, struct file *file)
 
 static int v9fs_file_lock(struct file *filp, int cmd, struct file_lock *fl)
 {
-	int res = 0;
 	struct inode *inode = file_inode(filp);
 
 	p9_debug(P9_DEBUG_VFS, "filp: %p lock: %p\n", filp, fl);
@@ -124,7 +123,7 @@ static int v9fs_file_lock(struct file *filp, int cmd, struct file_lock *fl)
 		invalidate_mapping_pages(&inode->i_data, 0, -1);
 	}
 
-	return res;
+	return 0;
 }
 
 static int v9fs_file_do_lock(struct file *filp, int cmd, struct file_lock *fl)

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,6 @@ int v9fs_file_open(struct inode inode, struct file file)`
`114`	`114`
`115`	`115`	`static int v9fs_file_lock(struct file filp, int cmd, struct file_lock fl)`
`116`	`116`	`{`
`117`		`- int res = 0;`
`118`	`117`	`struct inode *inode = file_inode(filp);`
`119`	`118`
`120`	`119`	`p9_debug(P9_DEBUG_VFS, "filp: %p lock: %p\n", filp, fl);`
`@@ -124,7 +123,7 @@ static int v9fs_file_lock(struct file filp, int cmd, struct file_lock fl)`
`124`	`123`	`invalidate_mapping_pages(&inode->i_data, 0, -1);`
`125`	`124`	`}`
`126`	`125`
`127`		`- return res;`
	`126`	`+ return 0;`
`128`	`127`	`}`
`129`	`128`
`130`	`129`	`static int v9fs_file_do_lock(struct file filp, int cmd, struct file_lock fl)`