summaryrefslogtreecommitdiff
path: root/kernel/acct.c
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2014-05-07 05:12:09 -0400
committerAl Viro <viro@zeniv.linux.org.uk>2014-08-07 14:40:07 -0400
commite25ff11ff16aba000dfe9e568d867e5142c31f16 (patch)
tree9a0636f305f10efb8c927bf13b2e83c68c081c4b /kernel/acct.c
parentcdd37e23092c3c6fbbb2e611f8c3d18e676bf28f (diff)
downloadlwn-e25ff11ff16aba000dfe9e568d867e5142c31f16.tar.gz
lwn-e25ff11ff16aba000dfe9e568d867e5142c31f16.zip
split the slow path in acct_process() off
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'kernel/acct.c')
-rw-r--r--kernel/acct.c50
1 files changed, 28 insertions, 22 deletions
diff --git a/kernel/acct.c b/kernel/acct.c
index efa891beeaa3..51188603b258 100644
--- a/kernel/acct.c
+++ b/kernel/acct.c
@@ -599,34 +599,35 @@ void acct_collect(long exitcode, int group_dead)
spin_unlock_irq(&current->sighand->siglock);
}
-static void acct_process_in_ns(struct pid_namespace *ns)
+static void slow_acct_process(struct pid_namespace *ns)
{
- struct file *file = NULL;
- struct bsd_acct_struct *acct;
+ for ( ; ns; ns = ns->parent) {
+ struct file *file = NULL;
+ struct bsd_acct_struct *acct;
- acct = ns->bacct;
- /*
- * accelerate the common fastpath:
- */
- if (!acct || !acct->file)
- return;
+ acct = ns->bacct;
+ /*
+ * accelerate the common fastpath:
+ */
+ if (!acct || !acct->file)
+ continue;
- spin_lock(&acct_lock);
- file = acct->file;
- if (unlikely(!file)) {
+ spin_lock(&acct_lock);
+ file = acct->file;
+ if (unlikely(!file)) {
+ spin_unlock(&acct_lock);
+ continue;
+ }
+ get_file(file);
spin_unlock(&acct_lock);
- return;
- }
- get_file(file);
- spin_unlock(&acct_lock);
- do_acct_process(acct, ns, file);
- fput(file);
+ do_acct_process(acct, ns, file);
+ fput(file);
+ }
}
/**
- * acct_process - now just a wrapper around acct_process_in_ns,
- * which in turn is a wrapper around do_acct_process.
+ * acct_process
*
* handles process accounting for an exiting task
*/
@@ -639,6 +640,11 @@ void acct_process(void)
* alive and holds its namespace, which in turn holds
* its parent.
*/
- for (ns = task_active_pid_ns(current); ns != NULL; ns = ns->parent)
- acct_process_in_ns(ns);
+ for (ns = task_active_pid_ns(current); ns != NULL; ns = ns->parent) {
+ struct bsd_acct_struct *acct = ns->bacct;
+ if (acct && acct->file)
+ break;
+ }
+ if (unlikely(ns))
+ slow_acct_process(ns);
}