]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
asmlinkage_protect replaces prevent_tail_call
authorRoland McGrath <roland@redhat.com>
Thu, 10 Apr 2008 22:37:38 +0000 (15:37 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 11 Apr 2008 00:28:26 +0000 (17:28 -0700)
commit54a015104136974262afa4b8ddd943ea70dec8a2
tree713f0c1f4d0afe62e5c568a424e309f70388cf7f
parent783e391b7b5b273cd20856d8f6f4878da8ec31b3
asmlinkage_protect replaces prevent_tail_call

The prevent_tail_call() macro works around the problem of the compiler
clobbering argument words on the stack, which for asmlinkage functions
is the caller's (user's) struct pt_regs.  The tail/sibling-call
optimization is not the only way that the compiler can decide to use
stack argument words as scratch space, which we have to prevent.
Other optimizations can do it too.

Until we have new compiler support to make "asmlinkage" binding on the
compiler's own use of the stack argument frame, we have work around all
the manifestations of this issue that crop up.

More cases seem to be prevented by also keeping the incoming argument
variables live at the end of the function.  This makes their original
stack slots attractive places to leave those variables, so the compiler
tends not clobber them for something else.  It's still no guarantee, but
it handles some observed cases that prevent_tail_call() did not.

Signed-off-by: Roland McGrath <roland@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/x86/kernel/tls.c
fs/open.c
include/asm-x86/linkage.h
include/linux/linkage.h
kernel/exit.c
kernel/uid16.c