Skip to content

Commit

Permalink
make fs/{namespace,super}.c forget about acct.h
Browse files Browse the repository at this point in the history
These externs belong in fs/internal.h.  Rename (they are not acct-specific
anymore) and move them over there.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
  • Loading branch information
Al Viro committed Aug 7, 2014
1 parent efb170c commit 8fa1f1c
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 10 deletions.
9 changes: 5 additions & 4 deletions fs/fs_pin.c
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#include <linux/fs.h>
#include <linux/slab.h>
#include <linux/fs_pin.h>
#include "internal.h"
#include "mount.h"

static void pin_free_rcu(struct rcu_head *head)
Expand Down Expand Up @@ -32,13 +33,13 @@ void pin_insert(struct fs_pin *pin, struct vfsmount *m)
spin_unlock(&pin_lock);
}

void acct_auto_close_mnt(struct hlist_head *list)
void mnt_pin_kill(struct mount *m)
{
while (1) {
struct hlist_node *p;
struct fs_pin *pin;
rcu_read_lock();
p = ACCESS_ONCE(list->first);
p = ACCESS_ONCE(m->mnt_pins.first);
if (!p) {
rcu_read_unlock();
break;
Expand All @@ -54,13 +55,13 @@ void acct_auto_close_mnt(struct hlist_head *list)
}
}

void acct_auto_close(struct hlist_head *list)
void sb_pin_kill(struct super_block *sb)
{
while (1) {
struct hlist_node *p;
struct fs_pin *pin;
rcu_read_lock();
p = ACCESS_ONCE(list->first);
p = ACCESS_ONCE(sb->s_pins.first);
if (!p) {
rcu_read_unlock();
break;
Expand Down
6 changes: 6 additions & 0 deletions fs/internal.h
Original file line number Diff line number Diff line change
Expand Up @@ -143,3 +143,9 @@ extern long do_splice_direct(struct file *in, loff_t *ppos, struct file *out,
* pipe.c
*/
extern const struct file_operations pipefifo_fops;

/*
* fs_pin.c
*/
extern void sb_pin_kill(struct super_block *sb);
extern void mnt_pin_kill(struct mount *m);
3 changes: 1 addition & 2 deletions fs/namespace.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
#include <linux/namei.h>
#include <linux/security.h>
#include <linux/idr.h>
#include <linux/acct.h> /* acct_auto_close_mnt */
#include <linux/init.h> /* init_rootfs */
#include <linux/fs_struct.h> /* get_fs_root et.al. */
#include <linux/fsnotify.h> /* fsnotify_vfsmount_delete */
Expand Down Expand Up @@ -956,7 +955,7 @@ static void mntput_no_expire(struct mount *mnt)
mnt->mnt_pinned = 0;
rcu_read_unlock();
unlock_mount_hash();
acct_auto_close_mnt(&mnt->mnt_pins);
mnt_pin_kill(mnt);
goto put_again;
}
if (unlikely(mnt->mnt.mnt_flags & MNT_DOOMED)) {
Expand Down
3 changes: 1 addition & 2 deletions fs/super.c
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@

#include <linux/export.h>
#include <linux/slab.h>
#include <linux/acct.h>
#include <linux/blkdev.h>
#include <linux/mount.h>
#include <linux/security.h>
Expand Down Expand Up @@ -707,7 +706,7 @@ int do_remount_sb(struct super_block *sb, int flags, void *data, int force)
if (remount_ro) {
if (sb->s_pins.first) {
up_write(&sb->s_umount);
acct_auto_close(&sb->s_pins);
sb_pin_kill(sb);
down_write(&sb->s_umount);
if (!sb->s_root)
return 0;
Expand Down
2 changes: 0 additions & 2 deletions include/linux/acct.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,8 +32,6 @@ extern void acct_exit_ns(struct pid_namespace *);
#define acct_process() do { } while (0)
#define acct_exit_ns(ns) do { } while (0)
#endif
extern void acct_auto_close(struct hlist_head *);
extern void acct_auto_close_mnt(struct hlist_head *);

/*
* ACCT_VERSION numbers as yet defined:
Expand Down

0 comments on commit 8fa1f1c

Please sign in to comment.