summaryrefslogtreecommitdiffstats
path: root/arch/i386/kernel/ioport.c
diff options
context:
space:
mode:
authorZachary Amsden2005-09-04 00:56:43 +0200
committerLinus Torvalds2005-09-05 09:06:12 +0200
commit0998e4228aca046fbd747c3fed909791d52e88eb (patch)
tree314cb04a6223100bf468cc420985bfe7e3680d44 /arch/i386/kernel/ioport.c
parent[PATCH] x86: more asm cleanups (diff)
downloadkernel-qcow2-linux-0998e4228aca046fbd747c3fed909791d52e88eb.tar.gz
kernel-qcow2-linux-0998e4228aca046fbd747c3fed909791d52e88eb.tar.xz
kernel-qcow2-linux-0998e4228aca046fbd747c3fed909791d52e88eb.zip
[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>
Diffstat (limited to 'arch/i386/kernel/ioport.c')
0 files changed, 0 insertions, 0 deletions