mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-11-23 23:30:50 +07:00
get rid of hostfs_read_inode()
There are only two call sites; in one (hostfs_iget()) it's actually a no-op and in another (fill_super()) it's easier to expand the damn thing and use what we know about its arguments to simplify it. Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
This commit is contained in:
parent
601d2c38b9
commit
52b209f7b8
@ -204,50 +204,11 @@ static char *follow_link(char *link)
|
||||
return ERR_PTR(n);
|
||||
}
|
||||
|
||||
static int hostfs_read_inode(struct inode *ino)
|
||||
{
|
||||
char *name;
|
||||
int err = 0;
|
||||
|
||||
/*
|
||||
* Unfortunately, we are called from iget() when we don't have a dentry
|
||||
* allocated yet.
|
||||
*/
|
||||
if (list_empty(&ino->i_dentry))
|
||||
goto out;
|
||||
|
||||
err = -ENOMEM;
|
||||
name = inode_name(ino, 0);
|
||||
if (name == NULL)
|
||||
goto out;
|
||||
|
||||
if (file_type(name, NULL, NULL) == OS_TYPE_SYMLINK) {
|
||||
name = follow_link(name);
|
||||
if (IS_ERR(name)) {
|
||||
err = PTR_ERR(name);
|
||||
goto out;
|
||||
}
|
||||
}
|
||||
|
||||
err = read_name(ino, name);
|
||||
kfree(name);
|
||||
out:
|
||||
return err;
|
||||
}
|
||||
|
||||
static struct inode *hostfs_iget(struct super_block *sb)
|
||||
{
|
||||
struct inode *inode;
|
||||
long ret;
|
||||
|
||||
inode = new_inode(sb);
|
||||
struct inode *inode = new_inode(sb);
|
||||
if (!inode)
|
||||
return ERR_PTR(-ENOMEM);
|
||||
ret = hostfs_read_inode(inode);
|
||||
if (ret < 0) {
|
||||
iput(inode);
|
||||
return ERR_PTR(ret);
|
||||
}
|
||||
return inode;
|
||||
}
|
||||
|
||||
@ -979,13 +940,23 @@ static int hostfs_fill_sb_common(struct super_block *sb, void *d, int silent)
|
||||
|
||||
sprintf(host_root_path, "%s/%s", root_ino, req_root);
|
||||
|
||||
root_inode = hostfs_iget(sb);
|
||||
if (IS_ERR(root_inode)) {
|
||||
err = PTR_ERR(root_inode);
|
||||
root_inode = new_inode(sb);
|
||||
if (!root_inode)
|
||||
goto out;
|
||||
}
|
||||
|
||||
err = init_inode(root_inode, NULL);
|
||||
root_inode->i_op = &hostfs_dir_iops;
|
||||
root_inode->i_fop = &hostfs_dir_fops;
|
||||
|
||||
if (file_type(host_root_path, NULL, NULL) == OS_TYPE_SYMLINK) {
|
||||
char *name = follow_link(host_root_path);
|
||||
if (IS_ERR(name))
|
||||
err = PTR_ERR(name);
|
||||
else
|
||||
err = read_name(root_inode, name);
|
||||
kfree(name);
|
||||
} else {
|
||||
err = read_name(root_inode, host_root_path);
|
||||
}
|
||||
if (err)
|
||||
goto out_put;
|
||||
|
||||
@ -994,14 +965,6 @@ static int hostfs_fill_sb_common(struct super_block *sb, void *d, int silent)
|
||||
if (sb->s_root == NULL)
|
||||
goto out_put;
|
||||
|
||||
err = hostfs_read_inode(root_inode);
|
||||
if (err) {
|
||||
/* No iput in this case because the dput does that for us */
|
||||
dput(sb->s_root);
|
||||
sb->s_root = NULL;
|
||||
goto out;
|
||||
}
|
||||
|
||||
return 0;
|
||||
|
||||
out_put:
|
||||
|
Loading…
Reference in New Issue
Block a user