]> nv-tegra.nvidia Code Review - linux-3.10.git/commit
[PATCH] x86: privilege cleanup
authorZachary Amsden <zach@vmware.com>
Sat, 3 Sep 2005 22:56:43 +0000 (15:56 -0700)
committerLinus Torvalds <torvalds@evo.osdl.org>
Mon, 5 Sep 2005 07:06:12 +0000 (00:06 -0700)
commit0998e4228aca046fbd747c3fed909791d52e88eb
tree314cb04a6223100bf468cc420985bfe7e3680d44
parentf2ab4461249df85b20930a7a57b54f39c5ae291a
[PATCH] x86: privilege cleanup

Privilege checking cleanup.  Originally, these diffs were much greater, but
recent cleanups in Linux have already done much of the cleanup.  I added
some explanatory comments in places where the reasoning behind certain
tests is rather subtle.

Also, in traps.c, we can skip the user_mode check in handle_BUG().  The
reason is, there are only two call chains - one via die_if_kernel() and one
via do_page_fault(), both entering from die().  Both of these paths already
ensure that a kernel mode failure has happened.  Also, the original check
here, if (user_mode(regs)) was insufficient anyways, since it would not
rule out BUG faults from V8086 mode execution.

Saving the %ss segment in show_regs() rather than assuming a fixed value
also gives better information about the current kernel state in the
register dump.

Signed-off-by: Zachary Amsden <zach@vmware.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
arch/i386/kernel/signal.c
arch/i386/kernel/traps.c
include/asm-i386/ptrace.h