Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0c18d7a5 authored by Pavel Emelyanov's avatar Pavel Emelyanov Committed by Linus Torvalds
Browse files

bsdacct: fix and add comments around acct_process() 



Fix the one describing what this function is and add one more - about
locking absence around pid namespaces loop.

Signed-off-by: default avatarPavel Emelyanov <xemul@openvz.org>
Cc: Randy Dunlap <randy.dunlap@oracle.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 7d1e1350

kernel/acct.c

+7 −1
Original line number Diff line number Diff line
@@ -657,7 +657,8 @@ static void acct_process_in_ns(struct pid_namespace *ns) 
}



/**

 * acct_process - now just a wrapper around do_acct_process

 * acct_process - now just a wrapper around acct_process_in_ns,

 * which in turn is a wrapper around do_acct_process.

 *

 * handles process accounting for an exiting task

 */
@@ -665,6 +666,11 @@ void acct_process(void) 
{

	struct pid_namespace *ns;



	/*

	 * This loop is safe lockless, since current is still

	 * 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);

}