aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJonathan Lim <jlim@sgi.com>2007-08-31 02:56:23 -0400
committerLinus Torvalds <torvalds@woody.linux-foundation.org>2007-08-31 04:42:22 -0400
commitf2ab6d8889422c1f5354f014e8bef337b1d1bade (patch)
treeab7de371c86ce9e72015363fc06563709d823354
parentc24c55c75a0d799be07ad6c1a7ff4ccfe25da75b (diff)
Assign task_struct.exit_code before taskstats_exit()
taskstats.ac_exitcode is assigned to task_struct.exit_code in bacct_add_tsk() through the following kernel function calls: do_exit() taskstats_exit() fill_pid() bacct_add_tsk() The problem is that in do_exit(), task_struct.exit_code is set to 'code' only after taskstats_exit() has been called. So we need to move the assignment before taskstats_exit(). Signed-off-by: Jonathan Lim <jlim@sgi.com> Cc: Balbir Singh <balbir@in.ibm.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--kernel/exit.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/exit.c b/kernel/exit.c
index 9578c1ae19ca..06b24b3aa370 100644
--- a/kernel/exit.c
+++ b/kernel/exit.c
@@ -975,6 +975,7 @@ fastcall NORET_TYPE void do_exit(long code)
975 if (unlikely(tsk->audit_context)) 975 if (unlikely(tsk->audit_context))
976 audit_free(tsk); 976 audit_free(tsk);
977 977
978 tsk->exit_code = code;
978 taskstats_exit(tsk, group_dead); 979 taskstats_exit(tsk, group_dead);
979 980
980 exit_mm(tsk); 981 exit_mm(tsk);
@@ -996,7 +997,6 @@ fastcall NORET_TYPE void do_exit(long code)
996 if (tsk->binfmt) 997 if (tsk->binfmt)
997 module_put(tsk->binfmt->module); 998 module_put(tsk->binfmt->module);
998 999
999 tsk->exit_code = code;
1000 proc_exit_connector(tsk); 1000 proc_exit_connector(tsk);
1001 exit_task_namespaces(tsk); 1001 exit_task_namespaces(tsk);
1002 exit_notify(tsk); 1002 exit_notify(tsk);