Commit 598af051 authored Apr 10, 2008 by Roland McGrath Committed by Linus Torvalds Apr 10, 2008

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>

parent 54a01510

fs/aio.c

+1 −0

Original line number	Diff line number	Diff line
		@@ -1790,6 +1790,7 @@ asmlinkage long sys_io_getevents(aio_context_t ctx_id,
		put_ioctx(ioctx);
		}

		asmlinkage_protect(5, ret, ctx_id, min_nr, nr, events, timeout);
		return ret;
		}

Admin message