diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2021-06-28 14:52:01 -0500 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2021-12-13 12:04:45 -0600 |
commit | 0e25498f8cd43c1b5aa327f373dd094e9a006da7 (patch) | |
tree | f9f4cf6d7c1deccaa34ad6f7af345bff52b63c12 /arch/sparc/kernel/traps_64.c | |
parent | 5e354747b2c91f64544b97760d38e2d3280307b2 (diff) | |
download | lwn-0e25498f8cd43c1b5aa327f373dd094e9a006da7.tar.gz lwn-0e25498f8cd43c1b5aa327f373dd094e9a006da7.zip |
exit: Add and use make_task_dead.
There are two big uses of do_exit. The first is it's design use to be
the guts of the exit(2) system call. The second use is to terminate
a task after something catastrophic has happened like a NULL pointer
in kernel code.
Add a function make_task_dead that is initialy exactly the same as
do_exit to cover the cases where do_exit is called to handle
catastrophic failure. In time this can probably be reduced to just a
light wrapper around do_task_dead. For now keep it exactly the same so
that there will be no behavioral differences introducing this new
concept.
Replace all of the uses of do_exit that use it for catastraphic
task cleanup with make_task_dead to make it clear what the code
is doing.
As part of this rename rewind_stack_do_exit
rewind_stack_and_make_dead.
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'arch/sparc/kernel/traps_64.c')
-rw-r--r-- | arch/sparc/kernel/traps_64.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/arch/sparc/kernel/traps_64.c b/arch/sparc/kernel/traps_64.c index 6863025ed56d..21077821f427 100644 --- a/arch/sparc/kernel/traps_64.c +++ b/arch/sparc/kernel/traps_64.c @@ -2559,9 +2559,7 @@ void __noreturn die_if_kernel(char *str, struct pt_regs *regs) } if (panic_on_oops) panic("Fatal exception"); - if (regs->tstate & TSTATE_PRIV) - do_exit(SIGKILL); - do_exit(SIGSEGV); + make_task_dead((regs->tstate & TSTATE_PRIV)? SIGKILL : SIGSEGV); } EXPORT_SYMBOL(die_if_kernel); |