aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoland McGrath <roland@redhat.com>2008-04-10 18:38:45 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2008-04-10 20:28:26 -0400
commit598af051a79d05b751fe793f1fe09fcf74763e02 (patch)
treec9adafd96ee316c3d05f2978697d3624547f9e84
parent54a015104136974262afa4b8ddd943ea70dec8a2 (diff)
asmlinkage_protect sys_io_getevents
Use asmlinkage_protect in sys_io_getevents, because GCC for i386 with CONFIG_FRAME_POINTER=n can decide to clobber an argument word on the stack, i.e. the user struct pt_regs. Here the problem is not a tail call, but just the compiler's use of the stack when it inlines and optimizes the body of the called function. This seems to avoid it. Signed-off-by: Roland McGrath <roland@redhat.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--fs/aio.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/aio.c b/fs/aio.c
index 6af921940622..60a4cd4d1137 100644
--- a/fs/aio.c
+++ b/fs/aio.c
@@ -1790,6 +1790,7 @@ asmlinkage long sys_io_getevents(aio_context_t ctx_id,
1790 put_ioctx(ioctx); 1790 put_ioctx(ioctx);
1791 } 1791 }
1792 1792
1793 asmlinkage_protect(5, ret, ctx_id, min_nr, nr, events, timeout);
1793 return ret; 1794 return ret;
1794} 1795}
1795 1796