Unverified Commit 10211b4a authored by Colin Ian King's avatar Colin Ian King Committed by Eric Van Hensbergen
Browse files

fs/9p: remove redundant pointer v9ses 



Pointer v9ses is being assigned the value from the return of inlined
function v9fs_inode2v9ses (which just returns inode->i_sb->s_fs_info).
The pointer is not used after the assignment, so the variable is
redundant and can be removed.

Cleans up clang scan warnings such as:
fs/9p/vfs_inode_dotl.c:300:28: warning: variable 'v9ses' set but not
used [-Wunused-but-set-variable]

Signed-off-by: default avatarColin Ian King <colin.i.king@gmail.com>
Reviewed-by: default avatarDominique Martinet <asmadeus@codewreck.org>
Signed-off-by: default avatarEric Van Hensbergen <ericvh@kernel.org>
parent 11763a85

fs/9p/vfs_inode_dotl.c

+0 −4
Original line number Diff line number Diff line
@@ -297,7 +297,6 @@ static int v9fs_vfs_mkdir_dotl(struct mnt_idmap *idmap, 
			       umode_t omode)

{

	int err;

	struct v9fs_session_info *v9ses;

	struct p9_fid *fid = NULL, *dfid = NULL;

	kgid_t gid;

	const unsigned char *name;
@@ -307,7 +306,6 @@ static int v9fs_vfs_mkdir_dotl(struct mnt_idmap *idmap, 
	struct posix_acl *dacl = NULL, *pacl = NULL;



	p9_debug(P9_DEBUG_VFS, "name %pd\n", dentry);

	v9ses = v9fs_inode2v9ses(dir);



	omode |= S_IFDIR;

	if (dir->i_mode & S_ISGID)
@@ -739,7 +737,6 @@ v9fs_vfs_mknod_dotl(struct mnt_idmap *idmap, struct inode *dir, 
	kgid_t gid;

	const unsigned char *name;

	umode_t mode;

	struct v9fs_session_info *v9ses;

	struct p9_fid *fid = NULL, *dfid = NULL;

	struct inode *inode;

	struct p9_qid qid;
@@ -749,7 +746,6 @@ v9fs_vfs_mknod_dotl(struct mnt_idmap *idmap, struct inode *dir, 
		 dir->i_ino, dentry, omode,

		 MAJOR(rdev), MINOR(rdev));



	v9ses = v9fs_inode2v9ses(dir);

	dfid = v9fs_parent_fid(dentry);

	if (IS_ERR(dfid)) {

		err = PTR_ERR(dfid);