vfs: now it can be done - make mnt_parent point to struct mount
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
diff --git a/fs/namespace.c b/fs/namespace.c
index c6384bc..5e700c6 100644
--- a/fs/namespace.c
+++ b/fs/namespace.c
@@ -476,7 +476,7 @@
if (tmp == head)
break;
p = list_entry(tmp, struct mount, mnt_hash);
- if (p->mnt_parent == mnt && p->mnt.mnt_mountpoint == dentry) {
+ if (&p->mnt_parent->mnt == mnt && p->mnt.mnt_mountpoint == dentry) {
found = p;
break;
}
@@ -558,8 +558,8 @@
static void detach_mnt(struct mount *mnt, struct path *old_path)
{
old_path->dentry = mnt->mnt.mnt_mountpoint;
- old_path->mnt = mnt->mnt_parent;
- mnt->mnt_parent = &mnt->mnt;
+ old_path->mnt = &mnt->mnt_parent->mnt;
+ mnt->mnt_parent = mnt;
mnt->mnt.mnt_mountpoint = mnt->mnt.mnt_root;
list_del_init(&mnt->mnt.mnt_child);
list_del_init(&mnt->mnt_hash);
@@ -572,7 +572,7 @@
void mnt_set_mountpoint(struct vfsmount *mnt, struct dentry *dentry,
struct mount *child_mnt)
{
- child_mnt->mnt_parent = mntget(mnt);
+ child_mnt->mnt_parent = real_mount(mntget(mnt));
child_mnt->mnt.mnt_mountpoint = dget(dentry);
spin_lock(&dentry->d_lock);
dentry->d_flags |= DCACHE_MOUNTED;
@@ -610,12 +610,12 @@
*/
static void commit_tree(struct mount *mnt)
{
- struct vfsmount *parent = mnt->mnt_parent;
+ struct mount *parent = mnt->mnt_parent;
struct vfsmount *m;
LIST_HEAD(head);
- struct mnt_namespace *n = parent->mnt_ns;
+ struct mnt_namespace *n = parent->mnt.mnt_ns;
- BUG_ON(parent == &mnt->mnt);
+ BUG_ON(parent == mnt);
list_add_tail(&head, &mnt->mnt.mnt_list);
list_for_each_entry(m, &head, mnt_list) {
@@ -626,8 +626,8 @@
list_splice(&head, n->list.prev);
list_add_tail(&mnt->mnt_hash, mount_hashtable +
- hash(parent, mnt->mnt.mnt_mountpoint));
- list_add_tail(&mnt->mnt.mnt_child, &parent->mnt_mounts);
+ hash(&parent->mnt, mnt->mnt.mnt_mountpoint));
+ list_add_tail(&mnt->mnt.mnt_child, &parent->mnt.mnt_mounts);
touch_mnt_namespace(n);
}
@@ -639,9 +639,9 @@
if (&p->mnt == root)
return NULL;
next = p->mnt.mnt_child.next;
- if (next != &p->mnt_parent->mnt_mounts)
+ if (next != &p->mnt_parent->mnt.mnt_mounts)
break;
- p = real_mount(p->mnt_parent);
+ p = p->mnt_parent;
}
}
return list_entry(next, struct mount, mnt.mnt_child);
@@ -682,7 +682,7 @@
mnt->mnt.mnt_root = root;
mnt->mnt.mnt_sb = root->d_sb;
mnt->mnt.mnt_mountpoint = mnt->mnt.mnt_root;
- mnt->mnt_parent = &mnt->mnt;
+ mnt->mnt_parent = mnt;
return &mnt->mnt;
}
EXPORT_SYMBOL_GPL(vfs_kern_mount);
@@ -710,7 +710,7 @@
mnt->mnt.mnt_sb = sb;
mnt->mnt.mnt_root = dget(root);
mnt->mnt.mnt_mountpoint = mnt->mnt.mnt_root;
- mnt->mnt_parent = &mnt->mnt;
+ mnt->mnt_parent = mnt;
if (flag & CL_SLAVE) {
list_add(&mnt->mnt.mnt_slave, &old->mnt.mnt_slave_list);
@@ -1027,7 +1027,7 @@
struct path root = p->root;
int err = 0;
- seq_printf(m, "%i %i %u:%u ", mnt->mnt_id, r->mnt_parent->mnt_id,
+ seq_printf(m, "%i %i %u:%u ", mnt->mnt_id, r->mnt_parent->mnt.mnt_id,
MAJOR(sb->s_dev), MINOR(sb->s_dev));
if (sb->s_op->show_path)
err = sb->s_op->show_path(m, mnt);
@@ -1202,9 +1202,9 @@
br_write_lock(vfsmount_lock);
dentry = mnt->mnt.mnt_mountpoint;
- m = mnt->mnt_parent;
+ m = &mnt->mnt_parent->mnt;
mnt->mnt.mnt_mountpoint = mnt->mnt.mnt_root;
- mnt->mnt_parent = &mnt->mnt;
+ mnt->mnt_parent = mnt;
m->mnt_ghosts--;
br_write_unlock(vfsmount_lock);
dput(dentry);
@@ -1237,7 +1237,7 @@
__mnt_make_shortterm(&p->mnt);
list_del_init(&p->mnt.mnt_child);
if (mnt_has_parent(p)) {
- p->mnt_parent->mnt_ghosts++;
+ p->mnt_parent->mnt.mnt_ghosts++;
dentry_reset_mounted(p->mnt.mnt_mountpoint);
}
change_mnt_propagation(p, MS_PRIVATE);
@@ -1435,9 +1435,9 @@
s = skip_mnt_tree(s);
continue;
}
- while (p != real_mount(s->mnt_parent)) {
- p = real_mount(p->mnt_parent);
- q = real_mount(q->mnt_parent);
+ while (p != s->mnt_parent) {
+ p = p->mnt_parent;
+ q = q->mnt_parent;
}
p = s;
path.mnt = &q->mnt;
@@ -1899,7 +1899,7 @@
/*
* Don't move a mount residing in a shared parent.
*/
- if (IS_MNT_SHARED(old->mnt_parent))
+ if (IS_MNT_SHARED(&old->mnt_parent->mnt))
goto out1;
/*
* Don't move a mount tree containing unbindable mounts to a destination
@@ -1909,7 +1909,7 @@
tree_contains_unbindable(old))
goto out1;
err = -ELOOP;
- for (p = real_mount(path->mnt); mnt_has_parent(p); p = real_mount(p->mnt_parent))
+ for (p = real_mount(path->mnt); mnt_has_parent(p); p = p->mnt_parent)
if (p == old)
goto out1;
@@ -2159,7 +2159,7 @@
*/
if (this_parent != parent) {
next = this_parent->mnt.mnt_child.next;
- this_parent = real_mount(this_parent->mnt_parent);
+ this_parent = this_parent->mnt_parent;
goto resume;
}
return found;
@@ -2565,7 +2565,7 @@
{
while (&mnt->mnt != root->mnt && mnt_has_parent(mnt)) {
dentry = mnt->mnt.mnt_mountpoint;
- mnt = real_mount(mnt->mnt_parent);
+ mnt = mnt->mnt_parent;
}
return &mnt->mnt == root->mnt && is_subdir(dentry, root->dentry);
}
@@ -2636,8 +2636,8 @@
new_mnt = real_mount(new.mnt);
root_mnt = real_mount(root.mnt);
if (IS_MNT_SHARED(old.mnt) ||
- IS_MNT_SHARED(new_mnt->mnt_parent) ||
- IS_MNT_SHARED(root_mnt->mnt_parent))
+ IS_MNT_SHARED(&new_mnt->mnt_parent->mnt) ||
+ IS_MNT_SHARED(&root_mnt->mnt_parent->mnt))
goto out4;
if (!check_mnt(root.mnt) || !check_mnt(new.mnt))
goto out4;