Commit 863d684f authored by Al Viro's avatar Al Viro

vfs: move the rest of int fields to struct mount

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 15169fe7
...@@ -28,6 +28,9 @@ struct mount { ...@@ -28,6 +28,9 @@ struct mount {
struct mnt_namespace *mnt_ns; /* containing namespace */ struct mnt_namespace *mnt_ns; /* containing namespace */
int mnt_id; /* mount identifier */ int mnt_id; /* mount identifier */
int mnt_group_id; /* peer group identifier */ int mnt_group_id; /* peer group identifier */
int mnt_expiry_mark; /* true if marked for expiry */
int mnt_pinned;
int mnt_ghosts;
}; };
static inline struct mount *real_mount(struct vfsmount *mnt) static inline struct mount *real_mount(struct vfsmount *mnt)
......
...@@ -787,9 +787,9 @@ static void mntput_no_expire(struct mount *mnt) ...@@ -787,9 +787,9 @@ static void mntput_no_expire(struct mount *mnt)
return; return;
br_write_lock(vfsmount_lock); br_write_lock(vfsmount_lock);
#endif #endif
if (unlikely(mnt->mnt.mnt_pinned)) { if (unlikely(mnt->mnt_pinned)) {
mnt_add_count(mnt, mnt->mnt.mnt_pinned + 1); mnt_add_count(mnt, mnt->mnt_pinned + 1);
mnt->mnt.mnt_pinned = 0; mnt->mnt_pinned = 0;
br_write_unlock(vfsmount_lock); br_write_unlock(vfsmount_lock);
acct_auto_close_mnt(&mnt->mnt); acct_auto_close_mnt(&mnt->mnt);
goto put_again; goto put_again;
...@@ -801,10 +801,11 @@ static void mntput_no_expire(struct mount *mnt) ...@@ -801,10 +801,11 @@ static void mntput_no_expire(struct mount *mnt)
void mntput(struct vfsmount *mnt) void mntput(struct vfsmount *mnt)
{ {
if (mnt) { if (mnt) {
struct mount *m = real_mount(mnt);
/* avoid cacheline pingpong, hope gcc doesn't get "smart" */ /* avoid cacheline pingpong, hope gcc doesn't get "smart" */
if (unlikely(mnt->mnt_expiry_mark)) if (unlikely(m->mnt_expiry_mark))
mnt->mnt_expiry_mark = 0; m->mnt_expiry_mark = 0;
mntput_no_expire(real_mount(mnt)); mntput_no_expire(m);
} }
} }
EXPORT_SYMBOL(mntput); EXPORT_SYMBOL(mntput);
...@@ -820,16 +821,17 @@ EXPORT_SYMBOL(mntget); ...@@ -820,16 +821,17 @@ EXPORT_SYMBOL(mntget);
void mnt_pin(struct vfsmount *mnt) void mnt_pin(struct vfsmount *mnt)
{ {
br_write_lock(vfsmount_lock); br_write_lock(vfsmount_lock);
mnt->mnt_pinned++; real_mount(mnt)->mnt_pinned++;
br_write_unlock(vfsmount_lock); br_write_unlock(vfsmount_lock);
} }
EXPORT_SYMBOL(mnt_pin); EXPORT_SYMBOL(mnt_pin);
void mnt_unpin(struct vfsmount *mnt) void mnt_unpin(struct vfsmount *m)
{ {
struct mount *mnt = real_mount(m);
br_write_lock(vfsmount_lock); br_write_lock(vfsmount_lock);
if (mnt->mnt_pinned) { if (mnt->mnt_pinned) {
mnt_add_count(real_mount(mnt), 1); mnt_add_count(mnt, 1);
mnt->mnt_pinned--; mnt->mnt_pinned--;
} }
br_write_unlock(vfsmount_lock); br_write_unlock(vfsmount_lock);
...@@ -1200,17 +1202,17 @@ void release_mounts(struct list_head *head) ...@@ -1200,17 +1202,17 @@ void release_mounts(struct list_head *head)
list_del_init(&mnt->mnt_hash); list_del_init(&mnt->mnt_hash);
if (mnt_has_parent(mnt)) { if (mnt_has_parent(mnt)) {
struct dentry *dentry; struct dentry *dentry;
struct vfsmount *m; struct mount *m;
br_write_lock(vfsmount_lock); br_write_lock(vfsmount_lock);
dentry = mnt->mnt_mountpoint; dentry = mnt->mnt_mountpoint;
m = &mnt->mnt_parent->mnt; m = mnt->mnt_parent;
mnt->mnt_mountpoint = mnt->mnt.mnt_root; mnt->mnt_mountpoint = mnt->mnt.mnt_root;
mnt->mnt_parent = mnt; mnt->mnt_parent = mnt;
m->mnt_ghosts--; m->mnt_ghosts--;
br_write_unlock(vfsmount_lock); br_write_unlock(vfsmount_lock);
dput(dentry); dput(dentry);
mntput(m); mntput(&m->mnt);
} }
mntput(&mnt->mnt); mntput(&mnt->mnt);
} }
...@@ -1239,7 +1241,7 @@ void umount_tree(struct mount *mnt, int propagate, struct list_head *kill) ...@@ -1239,7 +1241,7 @@ void umount_tree(struct mount *mnt, int propagate, struct list_head *kill)
__mnt_make_shortterm(p); __mnt_make_shortterm(p);
list_del_init(&p->mnt_child); list_del_init(&p->mnt_child);
if (mnt_has_parent(p)) { if (mnt_has_parent(p)) {
p->mnt_parent->mnt.mnt_ghosts++; p->mnt_parent->mnt_ghosts++;
dentry_reset_mounted(p->mnt_mountpoint); dentry_reset_mounted(p->mnt_mountpoint);
} }
change_mnt_propagation(p, MS_PRIVATE); change_mnt_propagation(p, MS_PRIVATE);
...@@ -1281,7 +1283,7 @@ static int do_umount(struct mount *mnt, int flags) ...@@ -1281,7 +1283,7 @@ static int do_umount(struct mount *mnt, int flags)
} }
br_write_unlock(vfsmount_lock); br_write_unlock(vfsmount_lock);
if (!xchg(&mnt->mnt.mnt_expiry_mark, 1)) if (!xchg(&mnt->mnt_expiry_mark, 1))
return -EAGAIN; return -EAGAIN;
} }
...@@ -2106,7 +2108,7 @@ void mark_mounts_for_expiry(struct list_head *mounts) ...@@ -2106,7 +2108,7 @@ void mark_mounts_for_expiry(struct list_head *mounts)
* cleared by mntput()) * cleared by mntput())
*/ */
list_for_each_entry_safe(mnt, next, mounts, mnt_expire) { list_for_each_entry_safe(mnt, next, mounts, mnt_expire) {
if (!xchg(&mnt->mnt.mnt_expiry_mark, 1) || if (!xchg(&mnt->mnt_expiry_mark, 1) ||
propagate_mount_busy(mnt, 1)) propagate_mount_busy(mnt, 1))
continue; continue;
list_move(&mnt->mnt_expire, &graveyard); list_move(&mnt->mnt_expire, &graveyard);
......
...@@ -272,7 +272,7 @@ int propagate_mnt(struct mount *dest_mnt, struct dentry *dest_dentry, ...@@ -272,7 +272,7 @@ int propagate_mnt(struct mount *dest_mnt, struct dentry *dest_dentry,
*/ */
static inline int do_refcount_check(struct mount *mnt, int count) static inline int do_refcount_check(struct mount *mnt, int count)
{ {
int mycount = mnt_get_count(mnt) - mnt->mnt.mnt_ghosts; int mycount = mnt_get_count(mnt) - mnt->mnt_ghosts;
return (mycount > count); return (mycount > count);
} }
......
...@@ -58,9 +58,6 @@ struct vfsmount { ...@@ -58,9 +58,6 @@ struct vfsmount {
#endif #endif
const char *mnt_devname; /* Name of device e.g. /dev/dsk/hda1 */ const char *mnt_devname; /* Name of device e.g. /dev/dsk/hda1 */
struct list_head mnt_list; struct list_head mnt_list;
int mnt_expiry_mark; /* true if marked for expiry */
int mnt_pinned;
int mnt_ghosts;
}; };
struct file; /* forward dec */ struct file; /* forward dec */
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment