diff options
| author | Oleg Nesterov <[email protected]> | 2014-04-07 15:38:46 -0700 | 
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2014-04-07 16:36:06 -0700 | 
| commit | ad86622b478eaafdc25b74237df82b10fce6326d (patch) | |
| tree | 5cec0d87c13edce3c73a087903801492362310cd /fs/proc/array.c | |
| parent | b3ab03160dfaf8ab78d476b670de319f4c1a5685 (diff) | |
wait: swap EXIT_ZOMBIE and EXIT_DEAD to hide EXIT_TRACE from user-space
get_task_state() uses the most significant bit to report the state to
user-space, this means that EXIT_ZOMBIE->EXIT_TRACE->EXIT_DEAD transition
can be noticed via /proc as Z -> X -> Z change.  Note that this was
possible even before EXIT_TRACE was introduced.
This is not really bad but imho it make sense to hide EXIT_TRACE from
user-space completely.  So the patch simply swaps EXIT_ZOMBIE and
EXIT_DEAD, this way EXIT_TRACE will be seen as EXIT_ZOMBIE by user-space.
Signed-off-by: Oleg Nesterov <[email protected]>
Cc: Jan Kratochvil <[email protected]>
Cc: Michal Schmidt <[email protected]>
Cc: Al Viro <[email protected]>
Cc: Lennart Poettering <[email protected]>
Cc: Roland McGrath <[email protected]>
Cc: Tejun Heo <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'fs/proc/array.c')
| -rw-r--r-- | fs/proc/array.c | 4 | 
1 files changed, 2 insertions, 2 deletions
| diff --git a/fs/proc/array.c b/fs/proc/array.c index 656e401794de..64db2bceac59 100644 --- a/fs/proc/array.c +++ b/fs/proc/array.c @@ -138,8 +138,8 @@ static const char * const task_state_array[] = {  	"D (disk sleep)",	/*   2 */  	"T (stopped)",		/*   4 */  	"t (tracing stop)",	/*   8 */ -	"Z (zombie)",		/*  16 */ -	"X (dead)",		/*  32 */ +	"X (dead)",		/*  16 */ +	"Z (zombie)",		/*  32 */  };  static inline const char *get_task_state(struct task_struct *tsk) |