Unverified Commit f61c906a authored by Eric Van Hensbergen's avatar Eric Van Hensbergen
Browse files

fs/9p: Eliminate now unused v9fs_get_inode 



Now with all inode allocation going through get_from_fid
functions we can remove v9fs_get_inode and reduce us down
to a single inode allocation path.

Signed-off-by: default avatarEric Van Hensbergen <ericvh@kernel.org>
parent 2dc92e59

fs/9p/v9fs_vfs.h

+0 −2
Original line number Diff line number Diff line
@@ -40,8 +40,6 @@ extern struct kmem_cache *v9fs_inode_cache; 


struct inode *v9fs_alloc_inode(struct super_block *sb);

void v9fs_free_inode(struct inode *inode);

struct inode *v9fs_get_inode(struct super_block *sb, umode_t mode,

			     dev_t rdev);

void v9fs_set_netfs_context(struct inode *inode);

int v9fs_init_inode(struct v9fs_session_info *v9ses,

		    struct inode *inode, umode_t mode, dev_t rdev);

fs/9p/vfs_inode.c

+0 −30
Original line number Diff line number Diff line
@@ -331,36 +331,6 @@ int v9fs_init_inode(struct v9fs_session_info *v9ses, 


}



/**

 * v9fs_get_inode - helper function to setup an inode

 * @sb: superblock

 * @mode: mode to setup inode with

 * @rdev: The device numbers to set

 */



struct inode *v9fs_get_inode(struct super_block *sb, umode_t mode, dev_t rdev)

{

	int err;

	struct inode *inode;

	struct v9fs_session_info *v9ses = sb->s_fs_info;



	p9_debug(P9_DEBUG_VFS, "super block: %p mode: %ho\n", sb, mode);



	inode = new_inode(sb);

	if (!inode) {

		pr_warn("%s (%d): Problem allocating inode\n",

			__func__, task_pid_nr(current));

		return ERR_PTR(-ENOMEM);

	}

	err = v9fs_init_inode(v9ses, inode, mode, rdev);

	if (err) {

		iput(inode);

		return ERR_PTR(err);

	}

	v9fs_set_netfs_context(inode);

	return inode;

}



/**

 * v9fs_evict_inode - Remove an inode from the inode cache

 * @inode: inode to release